Remove remnants of codecov-opentelemetry (#883)

Author: Swatinem

Diff for: celery_config.py

@@ -2,18 +2,11 @@
 import gc
 import logging
 import logging.config
-import os
 from datetime import timedelta
 
 from celery import signals
 from celery.beat import BeatLazyFunc
 from celery.schedules import crontab
-from celery.signals import worker_process_init
-from codecovopentelem import (
-    CoverageSpanFilter,
-    UnableToStartProcessorException,
-    get_codecov_opentelemetry_instances,
-)
 from shared.celery_config import (
     BaseCeleryConfig,
     brolly_stats_rollup_task_name,
@@ -27,9 +20,7 @@
 from celery_task_router import route_task
 from helpers.cache import cache
 from helpers.clock import get_utc_now_as_iso_format
-from helpers.environment import is_enterprise
 from helpers.health_check import get_health_check_interval_seconds
-from helpers.version import get_current_version
 from services.redis import get_redis_connection
 
 log = logging.getLogger(__name__)
@@ -59,49 +50,6 @@ def initialize_cache(**kwargs):
     cache.configure(redis_cache_backend)
 
 
-@worker_process_init.connect(weak=False)
-def init_celery_tracing(*args, **kwargs):
-    if (
-        os.getenv("OPENTELEMETRY_ENDPOINT")
-        and os.getenv("OPENTELEMETRY_TOKEN")
-        and os.getenv("OPENTELEMETRY_CODECOV_RATE")
-        and not is_enterprise()
-    ):
-        from opentelemetry import trace
-        from opentelemetry.instrumentation.celery import CeleryInstrumentor
-        from opentelemetry.sdk.trace import TracerProvider
-        from opentelemetry.sdk.trace.export import BatchSpanProcessor
-
-        log.info("Configuring opentelemetry exporter")
-        provider = TracerProvider()
-        trace.set_tracer_provider(provider)
-        export_rate = float(os.getenv("OPENTELEMETRY_CODECOV_RATE"))
-        current_version = get_current_version()
-        current_env = "production"
-        try:
-            generator, exporter = get_codecov_opentelemetry_instances(
-                repository_token=os.getenv("OPENTELEMETRY_TOKEN"),
-                version_identifier=current_version,
-                sample_rate=export_rate,
-                filters={
-                    CoverageSpanFilter.regex_name_filter: None,
-                    CoverageSpanFilter.span_kind_filter: [
-                        trace.SpanKind.SERVER,
-                        trace.SpanKind.CONSUMER,
-                    ],
-                },
-                code=f"{current_version}:{current_env}",
-                untracked_export_rate=export_rate,
-                codecov_endpoint=os.getenv("OPENTELEMETRY_ENDPOINT"),
-                environment=current_env,
-            )
-            provider.add_span_processor(generator)
-            provider.add_span_processor(BatchSpanProcessor(exporter))
-            CeleryInstrumentor().instrument()
-        except UnableToStartProcessorException:
-            log.warning("Unable to instrument opentelemetry on worker")
-
-
 hourly_check_task_name = "app.cron.hourly_check.HourlyCheckTask"
 daily_plan_manager_task_name = "app.cron.daily.PlanManagerTask"
 

Diff for: enterprise/hooks/hook-opentelemetry.py

@@ -31,7 +31,6 @@ print(' --hiddenimport celery.worker.consumer')
 print(' --hiddenimport sqlalchemy.ext.baked')
 print(' --hiddenimport tasks')
 print(' --hiddenimport tornado.curl_httpclient')
-print(' --hiddenimport opentelemetry-sdk')
 print(' --hiddenimport asyncore')
 print(' --hiddenimport imaplib')
 print(' --hiddenimport poplib')

Diff for: enterprise/package.sh

@@ -1,4 +1,3 @@
-https://github.com/codecov/opentelem-python/archive/refs/tags/v0.0.4a1.tar.gz#egg=codecovopentelem
 https://github.com/codecov/test-results-parser/archive/94ff26cb083a02ff726dd2497c2129e4aaf5f7cd.tar.gz#egg=test-results-parser
 https://github.com/codecov/shared/archive/4db8f12c1a4ad43cedb1a965d2fa8cd8138fc2b3.tar.gz#egg=shared
 https://github.com/codecov/timestring/archive/d37ceacc5954dff3b5bd2f887936a98a668dda42.tar.gz#egg=timestring
@@ -19,8 +18,6 @@ jinja2>=3.1.3
 lxml>=4.9.1
 mock
 openai
-opentelemetry-instrumentation-celery>=0.45b0
-opentelemetry-sdk>=1.24.0
 orjson
 pre-commit
 polars==1.12.0

Diff for: requirements.in

@@ -1,9 +1,5 @@
-#
-# This file is autogenerated by pip-compile with Python 3.12
-# by the following command:
-#
-#    pip-compile
-#
+# This file was autogenerated by uv via the following command:
+#    uv pip compile requirements.in -o requirements.txt
 amqp==5.2.0
     # via kombu
 analytics-python==1.3.0b1
@@ -76,19 +72,14 @@ codecov-ribs==0.1.18
     # via
     #   -r requirements.in
     #   shared
-codecovopentelem @ https://github.com/codecov/opentelem-python/archive/refs/tags/v0.0.4a1.tar.gz
-    # via -r requirements.in
 colour==0.1.5
     # via shared
-coverage[toml]==7.5.0
+coverage==7.5.0
     # via
     #   -r requirements.in
-    #   codecovopentelem
     #   pytest-cov
 cryptography==43.0.1
     # via shared
-deprecated==1.2.12
-    # via opentelemetry-api
 distlib==0.3.7
     # via virtualenv
 distro==1.8.0
@@ -119,7 +110,7 @@ filelock==3.12.4
     # via virtualenv
 freezegun==1.5.0
     # via pytest-freezegun
-google-api-core[grpc]==2.11.1
+google-api-core==2.11.1
     # via
     #   google-cloud-core
     #   google-cloud-pubsub
@@ -144,7 +135,7 @@ google-crc32c==1.1.2
     # via google-resumable-media
 google-resumable-media==2.5.0
     # via google-cloud-storage
-googleapis-common-protos[grpc]==1.59.1
+googleapis-common-protos==1.59.1
     # via
     #   google-api-core
     #   grpc-google-iam-v1
@@ -184,8 +175,6 @@ idna==3.7
     #   yarl
 ijson==3.2.3
     # via shared
-importlib-metadata==6.8.0
-    # via opentelemetry-api
 iniconfig==1.1.1
     # via pytest
 jinja2==3.1.4
@@ -218,23 +207,6 @@ oauthlib==3.1.0
     # via shared
 openai==1.2.4
     # via -r requirements.in
-opentelemetry-api==1.24.0
-    # via
-    #   opentelemetry-instrumentation
-    #   opentelemetry-instrumentation-celery
-    #   opentelemetry-sdk
-opentelemetry-instrumentation==0.45b0
-    # via opentelemetry-instrumentation-celery
-opentelemetry-instrumentation-celery==0.45b0
-    # via -r requirements.in
-opentelemetry-sdk==1.24.0
-    # via
-    #   -r requirements.in
-    #   codecovopentelem
-opentelemetry-semantic-conventions==0.45b0
-    # via
-    #   opentelemetry-instrumentation-celery
-    #   opentelemetry-sdk
 orjson==3.10.7
     # via
     #   -r requirements.in
@@ -347,24 +319,27 @@ requests==2.32.3
     # via
     #   -r requirements.in
     #   analytics-python
-    #   codecovopentelem
     #   google-api-core
     #   google-cloud-storage
     #   shared
     #   stripe
 respx==0.20.2
     # via -r requirements.in
-rfc3986[idna2008]==1.4.0
+rfc3986==1.4.0
     # via httpx
 rsa==4.7.2
     # via google-auth
 s3transfer==0.10.1
     # via boto3
-sentry-sdk[celery]==2.13.0
+sentry-sdk==2.13.0
     # via
     #   -r requirements.in
     #   shared
-shared @ https://github.com/codecov/shared/archive/4db8f12c1a4ad43cedb1a965d2fa8cd8138fc2b3.tar.gz
+setuptools==75.4.0
+    # via
+    #   cerberus
+    #   nodeenv
+shared @ https://github.com/codecov/shared/archive/4db8f12c1a4ad43cedb1a965d2fa8cd8138fc2b3.tar.gz#egg=shared
     # via -r requirements.in
 six==1.16.0
     # via
@@ -396,13 +371,13 @@ statsd==3.3.0
     # via -r requirements.in
 stripe==9.6.0
     # via -r requirements.in
-test-results-parser @ https://github.com/codecov/test-results-parser/archive/94ff26cb083a02ff726dd2497c2129e4aaf5f7cd.tar.gz
+test-results-parser @ https://github.com/codecov/test-results-parser/archive/94ff26cb083a02ff726dd2497c2129e4aaf5f7cd.tar.gz#egg=test-results-parser
     # via -r requirements.in
 text-unidecode==1.3
     # via faker
 time-machine==2.14.1
     # via -r requirements.in
-timestring @ https://github.com/codecov/timestring/archive/d37ceacc5954dff3b5bd2f887936a98a668dda42.tar.gz
+timestring @ https://github.com/codecov/timestring/archive/d37ceacc5954dff3b5bd2f887936a98a668dda42.tar.gz#egg=timestring
     # via -r requirements.in
 tlslite-ng==0.8.0b1
     # via shared
@@ -413,7 +388,6 @@ typing==3.7.4.3
 typing-extensions==4.6.3
     # via
     #   openai
-    #   opentelemetry-sdk
     #   pydantic
     #   pydantic-core
     #   shared
@@ -441,16 +415,8 @@ virtualenv==20.24.5
 wcwidth==0.2.5
     # via prompt-toolkit
 wrapt==1.16.0
-    # via
-    #   deprecated
-    #   opentelemetry-instrumentation
-    #   vcrpy
+    # via vcrpy
 yarl==1.9.4
     # via vcrpy
-zipp==3.19.2
-    # via importlib-metadata
 zstandard==0.23.0
     # via -r requirements.in
-
-# The following packages are considered to be unsafe in a requirements file:
-# setuptools