Skip to content

Commit

Permalink
fix: remove celery&redis instrumentor (#1399)
Browse files Browse the repository at this point in the history
  • Loading branch information
nannan00 authored Nov 18, 2023
1 parent b8788c2 commit 1407a29
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/bk-login/bklogin/monitoring/tracing/instrumentor.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,9 @@

from django.conf import settings
from opentelemetry.instrumentation import dbapi
from opentelemetry.instrumentation.celery import CeleryInstrumentor
from opentelemetry.instrumentation.django import DjangoInstrumentor
from opentelemetry.instrumentation.instrumentor import BaseInstrumentor # type: ignore
from opentelemetry.instrumentation.logging import LoggingInstrumentor
from opentelemetry.instrumentation.redis import RedisInstrumentor
from opentelemetry.instrumentation.requests import RequestsInstrumentor

from .hooks import django_request_hook, django_response_hook, requests_response_hook
Expand All @@ -36,10 +34,6 @@ def _instrument(self, **kwargs):
logger.info("otel instructment: requests")
DjangoInstrumentor().instrument(request_hook=django_request_hook, response_hook=django_response_hook)
logger.info("otel instructment: django")
RedisInstrumentor().instrument()
logger.info("otel instructment: redis")
CeleryInstrumentor().instrument()
logger.info("otel instructment: celery")

if getattr(settings, "OTEL_INSTRUMENT_DB_API", False):
import MySQLdb # noqa
Expand Down

0 comments on commit 1407a29

Please sign in to comment.