add confluent kafka instrumentation

Author: oxeye-dorkolog

.github/component_owners.yml

@@ -11,6 +11,10 @@ components:
     - oxeye-nikolay
     - nikosokolik
 
+  instrumentation/opentelemetry-instrumentation-confluent-kafka:
+    - oxeye-dorkolog
+    - dorkolog
+
   propagator/opentelemetry-propagator-aws-xray:
     - NathanielRN
 

CHANGELOG.md

@@ -13,6 +13,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - cleanup type hints for textmap `Getter` and `Setter` classes
   ([1106](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/1106))
 
+### Added
+- Added `opentelemetry-instrumention-confluent-kafka`
+  ([#1111](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/1111))
+
+
 ## [1.12.0rc1-0.31b0](https://github.com/open-telemetry/opentelemetry-python/releases/tag/v1.12.0rc1-0.31b0) - 2022-05-17
 
 

instrumentation/README.md

@@ -10,6 +10,7 @@
 | [opentelemetry-instrumentation-boto3sqs](./opentelemetry-instrumentation-boto3sqs) | boto3 ~= 1.0 |
 | [opentelemetry-instrumentation-botocore](./opentelemetry-instrumentation-botocore) | botocore ~= 1.0 |
 | [opentelemetry-instrumentation-celery](./opentelemetry-instrumentation-celery) | celery >= 4.0, < 6.0 |
+| [opentelemetry-instrumentation-confluent-kafka](./opentelemetry-instrumentation-confluent-kafka) | confluent-kafka ~= 1.8.2 |
 | [opentelemetry-instrumentation-dbapi](./opentelemetry-instrumentation-dbapi) | dbapi |
 | [opentelemetry-instrumentation-django](./opentelemetry-instrumentation-django) | django >= 1.10 |
 | [opentelemetry-instrumentation-elasticsearch](./opentelemetry-instrumentation-elasticsearch) | elasticsearch >= 2.0 |

instrumentation/opentelemetry-instrumentation-confluent-kafka/README.rst

@@ -0,0 +1,23 @@
+OpenTelemetry confluent-kafka Instrumentation
+===========================
+
+|pypi|
+
+.. |pypi| image:: https://badge.fury.io/py/opentelemetry-instrumentation-confluent-kafka.svg
+   :target: https://pypi.org/project/opentelemetry-instrumentation-confluent-kafka/
+
+This library allows tracing requests made by the confluent-kafka library.
+
+Installation
+------------
+
+::
+
+    pip install opentelemetry-instrumentation-confluent-kafka
+
+
+References
+----------
+
+* `OpenTelemetry confluent-kafka/ Tracing <https://opentelemetry-python-contrib.readthedocs.io/en/latest/instrumentation/confluent-kafka/confluent-kafka.html>`_
+* `OpenTelemetry Project <https://opentelemetry.io/>`_

instrumentation/opentelemetry-instrumentation-confluent-kafka/setup.cfg

@@ -0,0 +1,57 @@
+# Copyright The OpenTelemetry Authors
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+[metadata]
+name = opentelemetry-instrumentation-confluent-kafka
+description = OpenTelemetry Confluent Kafka instrumentation
+long_description = file: README.rst
+long_description_content_type = text/x-rst
+author = OpenTelemetry Authors
+author_email = cncf-opentelemetry-contributors@lists.cncf.io
+url = https://github.com/open-telemetry/opentelemetry-python-contrib/tree/main/instrumentation/opentelemetry-instrumentation-confluent-kafka
+platforms = any
+license = Apache-2.0
+classifiers =
+    Development Status :: 4 - Beta
+    Intended Audience :: Developers
+    License :: OSI Approved :: Apache Software License
+    Programming Language :: Python
+    Programming Language :: Python :: 3
+    Programming Language :: Python :: 3.6
+    Programming Language :: Python :: 3.7
+    Programming Language :: Python :: 3.8
+    Programming Language :: Python :: 3.9
+    Programming Language :: Python :: 3.10
+
+[options]
+python_requires = >=3.6
+package_dir=
+    =src
+packages=find_namespace:
+
+install_requires =
+    opentelemetry-api ~= 1.3
+    wrapt >= 1.0.0, < 2.0.0
+
+[options.extras_require]
+test =
+    # add any test dependencies here
+    confluent-kafka ~= 1.8.2
+
+[options.packages.find]
+where = src
+
+[options.entry_points]
+opentelemetry_instrumentor =
+    confluent_kafka = opentelemetry.instrumentation.confluent_kafka:ConfluentKafkaInstrumentor

instrumentation/opentelemetry-instrumentation-confluent-kafka/setup.py

@@ -0,0 +1,99 @@
+# Copyright The OpenTelemetry Authors
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+
+# DO NOT EDIT. THIS FILE WAS AUTOGENERATED FROM templates/instrumentation_setup.py.txt.
+# RUN `python scripts/generate_setup.py` TO REGENERATE.
+
+
+import distutils.cmd
+import json
+import os
+from configparser import ConfigParser
+
+import setuptools
+
+config = ConfigParser()
+config.read("setup.cfg")
+
+# We provide extras_require parameter to setuptools.setup later which
+# overwrites the extras_require section from setup.cfg. To support extras_require
+# section in setup.cfg, we load it here and merge it with the extras_require param.
+extras_require = {}
+if "options.extras_require" in config:
+    for key, value in config["options.extras_require"].items():
+        extras_require[key] = [v for v in value.split("\n") if v.strip()]
+
+BASE_DIR = os.path.dirname(__file__)
+PACKAGE_INFO = {}
+
+VERSION_FILENAME = os.path.join(
+    BASE_DIR,
+    "src",
+    "opentelemetry",
+    "instrumentation",
+    "confluent_kafka",
+    "version.py",
+)
+with open(VERSION_FILENAME, encoding="utf-8") as f:
+    exec(f.read(), PACKAGE_INFO)
+
+PACKAGE_FILENAME = os.path.join(
+    BASE_DIR,
+    "src",
+    "opentelemetry",
+    "instrumentation",
+    "confluent_kafka",
+    "package.py",
+)
+with open(PACKAGE_FILENAME, encoding="utf-8") as f:
+    exec(f.read(), PACKAGE_INFO)
+
+# Mark any instruments/runtime dependencies as test dependencies as well.
+extras_require["instruments"] = PACKAGE_INFO["_instruments"]
+test_deps = extras_require.get("test", [])
+for dep in extras_require["instruments"]:
+    test_deps.append(dep)
+
+extras_require["test"] = test_deps
+
+
+class JSONMetadataCommand(distutils.cmd.Command):
+
+    description = (
+        "print out package metadata as JSON. This is used by OpenTelemetry dev scripts to ",
+        "auto-generate code in other places",
+    )
+    user_options = []
+
+    def initialize_options(self):
+        pass
+
+    def finalize_options(self):
+        pass
+
+    def run(self):
+        metadata = {
+            "name": config["metadata"]["name"],
+            "version": PACKAGE_INFO["__version__"],
+            "instruments": PACKAGE_INFO["_instruments"],
+        }
+        print(json.dumps(metadata))
+
+
+setuptools.setup(
+    cmdclass={"meta": JSONMetadataCommand},
+    version=PACKAGE_INFO["__version__"],
+    extras_require=extras_require,
+)

instrumentation/opentelemetry-instrumentation-confluent-kafka/src/opentelemetry/instrumentation/confluent_kafka/__init__.py

@@ -101,24 +101,30 @@ def instrument_consumer(consumer: Consumer, tracer_provider=None)
 
 import confluent_kafka
 import wrapt
-from confluent_kafka import Producer, Consumer
-from opentelemetry import trace, propagate, context
+from confluent_kafka import Consumer, Producer
+
+from opentelemetry import context, propagate, trace
+from opentelemetry.instrumentation.confluent_kafka.package import _instruments
+from opentelemetry.instrumentation.confluent_kafka.utils import (
+    KafkaPropertiesExtractor,
+    _enrich_span,
+    _get_span_name,
+    _kafka_getter,
+    _kafka_setter,
+)
+from opentelemetry.instrumentation.confluent_kafka.version import __version__
 from opentelemetry.instrumentation.instrumentor import BaseInstrumentor
-from opentelemetry.semconv.trace import MessagingOperationValues
-from opentelemetry.trace import Tracer, Link, SpanKind
 from opentelemetry.instrumentation.utils import unwrap
-
-from kafka_instrumentation.package import _instruments
-from kafka_instrumentation.utils import KafkaPropertiesExtractor, _get_span_name, \
-    _kafka_setter, _enrich_span, _kafka_getter
-from kafka_instrumentation.version import __version__
+from opentelemetry.semconv.trace import MessagingOperationValues
+from opentelemetry.trace import Link, SpanKind, Tracer
 
 
 class AutoInstrumentedProducer(Producer):
-    def __init__(self, config):
-        super().__init__(config)
 
-    def produce(self, topic, value=None, *args, **kwargs):
+    # This method is deliberately implemented in order to allow wrapt to wrap this function
+    def produce(
+        self, topic, value=None, *args, **kwargs
+    ):  # pylint: disable=keyword-arg-before-vararg,useless-super-delegation
         super().produce(topic, value, *args, **kwargs)
 
 
@@ -127,12 +133,12 @@ def __init__(self, config):
         super().__init__(config)
         self._current_consume_span = None
 
-    def poll(self, timeout=-1):
+    # This method is deliberately implemented in order to allow wrapt to wrap this function
+    def poll(self, timeout=-1):  # pylint: disable=useless-super-delegation
         return super().poll(timeout)
 
 
 class ProxiedProducer(Producer):
-
     def __init__(self, producer: Producer, tracer: Tracer):
         self._producer = producer
         self._tracer = tracer
@@ -143,19 +149,22 @@ def flush(self, timeout=-1):
     def poll(self, timeout=-1):
         self._producer.poll(timeout)
 
-    def produce(self, topic, value=None, *args, **kwargs):
+    def produce(
+        self, topic, value=None, *args, **kwargs
+    ):  # pylint: disable=keyword-arg-before-vararg
         new_kwargs = kwargs.copy()
-        new_kwargs['topic'] = topic
-        new_kwargs['value'] = value
+        new_kwargs["topic"] = topic
+        new_kwargs["value"] = value
 
-        return ConfluentKafkaInstrumentor.wrap_produce(self._producer.produce, self, self._tracer, args, new_kwargs)
+        return ConfluentKafkaInstrumentor.wrap_produce(
+            self._producer.produce, self, self._tracer, args, new_kwargs
+        )
 
     def original_producer(self):
         return self._producer
 
 
 class ProxiedConsumer(Consumer):
-
     def __init__(self, consumer: Consumer, tracer: Tracer):
         self._consumer = consumer
         self._tracer = tracer
@@ -165,19 +174,29 @@ def __init__(self, consumer: Consumer, tracer: Tracer):
     def committed(self, partitions, timeout=-1):
         return self._consumer.committed(partitions, timeout)
 
-    def consume(self, num_messages=1, *args, **kwargs):
+    def consume(
+        self, num_messages=1, *args, **kwargs
+    ):  # pylint: disable=keyword-arg-before-vararg
         return self._consumer.consume(num_messages, *args, **kwargs)
 
-    def get_watermark_offsets(self, partition, timeout=-1, *args, **kwargs):
-        return self._consumer.get_watermark_offsets(partition, timeout, *args, **kwargs)
+    def get_watermark_offsets(
+        self, partition, timeout=-1, *args, **kwargs
+    ):  # pylint: disable=keyword-arg-before-vararg
+        return self._consumer.get_watermark_offsets(
+            partition, timeout, *args, **kwargs
+        )
 
     def offsets_for_times(self, partitions, timeout=-1):
         return self._consumer.offsets_for_times(partitions, timeout)
 
     def poll(self, timeout=-1):
-        return ConfluentKafkaInstrumentor.wrap_poll(self._consumer.poll, self, self._tracer, [timeout], {})
+        return ConfluentKafkaInstrumentor.wrap_poll(
+            self._consumer.poll, self, self._tracer, [timeout], {}
+        )
 
-    def subscribe(self, topics, on_assign=lambda *args: None, *args, **kwargs):
+    def subscribe(
+        self, topics, on_assign=lambda *args: None, *args, **kwargs
+    ):  # pylint: disable=keyword-arg-before-vararg
         self._consumer.subscribe(topics, on_assign, *args, **kwargs)
 
     def original_consumer(self):
@@ -189,8 +208,11 @@ class ConfluentKafkaInstrumentor(BaseInstrumentor):
     See `BaseInstrumentor`
     """
 
+    # pylint: disable=attribute-defined-outside-init
     @staticmethod
-    def instrument_producer(producer: Producer, tracer_provider=None) -> ProxiedProducer:
+    def instrument_producer(
+        producer: Producer, tracer_provider=None
+    ) -> ProxiedProducer:
         tracer = trace.get_tracer(
             __name__, __version__, tracer_provider=tracer_provider
         )
@@ -200,7 +222,9 @@ def instrument_producer(producer: Producer, tracer_provider=None) -> ProxiedProd
         return manual_producer
 
     @staticmethod
-    def instrument_consumer(consumer: Consumer, tracer_provider=None) -> ProxiedConsumer:
+    def instrument_consumer(
+        consumer: Consumer, tracer_provider=None
+    ) -> ProxiedConsumer:
         tracer = trace.get_tracer(
             __name__, __version__, tracer_provider=tracer_provider
         )
@@ -210,14 +234,16 @@ def instrument_consumer(consumer: Consumer, tracer_provider=None) -> ProxiedCons
         return manual_consumer
 
     @staticmethod
-    def uninstrument_producer(producer) -> Producer:
+    def uninstrument_producer(producer: Producer) -> Producer:
         if isinstance(producer, ProxiedProducer):
             return producer.original_producer()
+        return producer
 
     @staticmethod
-    def uninstrument_consumer(consumer) -> Consumer:
+    def uninstrument_consumer(consumer: Consumer) -> Consumer:
         if isinstance(consumer, ProxiedConsumer):
             return consumer.original_consumer()
+        return consumer
 
     def instrumentation_dependencies(self) -> Collection[str]:
         return _instruments
@@ -237,16 +263,26 @@ def _instrument(self, **kwargs):
         self._tracer = tracer
 
         def _inner_wrap_produce(func, instance, args, kwargs):
-            return ConfluentKafkaInstrumentor.wrap_produce(func, instance, self._tracer, args, kwargs)
+            return ConfluentKafkaInstrumentor.wrap_produce(
+                func, instance, self._tracer, args, kwargs
+            )
 
         def _inner_wrap_poll(func, instance, args, kwargs):
-            return ConfluentKafkaInstrumentor.wrap_poll(func, instance, self._tracer, args, kwargs)
+            return ConfluentKafkaInstrumentor.wrap_poll(
+                func, instance, self._tracer, args, kwargs
+            )
 
-        wrapt.wrap_function_wrapper("kafka_instrumentation",
-                                    "AutoInstrumentedProducer.produce", _inner_wrap_produce)
+        wrapt.wrap_function_wrapper(
+            AutoInstrumentedProducer,
+            "produce",
+            _inner_wrap_produce,
+        )
 
-        wrapt.wrap_function_wrapper("kafka_instrumentation",
-                                    "AutoInstrumentedConsumer.poll", _inner_wrap_poll)
+        wrapt.wrap_function_wrapper(
+            AutoInstrumentedConsumer,
+            "poll",
+            _inner_wrap_poll,
+        )
 
     def _uninstrument(self, **kwargs):
         confluent_kafka.Producer = self._original_kafka_producer
@@ -261,22 +297,29 @@ def wrap_produce(func, instance, tracer, args, kwargs):
         if not topic:
             topic = args[0]
 
-        span_name = _get_span_name("send", topic) 
-        with tracer.start_as_current_span(name=span_name, kind=trace.SpanKind.PRODUCER) as span:
-            headers = KafkaPropertiesExtractor.extract_produce_headers(args, kwargs)
+        span_name = _get_span_name("send", topic)
+        with tracer.start_as_current_span(
+            name=span_name, kind=trace.SpanKind.PRODUCER
+        ) as span:
+            headers = KafkaPropertiesExtractor.extract_produce_headers(
+                args, kwargs
+            )
             if headers is None:
                 headers = []
                 kwargs["headers"] = headers
 
             topic = KafkaPropertiesExtractor.extract_produce_topic(args)
-            bootstrap_servers = KafkaPropertiesExtractor.extract_bootstrap_servers(instance)
-            _enrich_span(span, topic, bootstrap_servers, operation=MessagingOperationValues.RECEIVE)  # Replace
+            _enrich_span(
+                span,
+                topic,
+                operation=MessagingOperationValues.RECEIVE,
+            )  # Replace
             propagate.inject(
                 headers,
                 setter=_kafka_setter,
             )
             return func(*args, **kwargs)
-            
+
     @staticmethod
     def wrap_poll(func, instance, tracer, args, kwargs):
         if instance._current_consume_span:
@@ -285,7 +328,9 @@ def wrap_poll(func, instance, tracer, args, kwargs):
             instance._current_consume_span.end()
             instance._current_consume_span = None
 
-        with tracer.start_as_current_span("recv", end_on_exit=True, kind=trace.SpanKind.CONSUMER) as span:
+        with tracer.start_as_current_span(
+            "recv", end_on_exit=True, kind=trace.SpanKind.CONSUMER
+        ):
             record = func(*args, **kwargs)
             if record:
                 links = []
@@ -296,20 +341,20 @@ def wrap_poll(func, instance, tracer, args, kwargs):
                             links.append(Link(context=item.get_span_context()))
 
                 instance._current_consume_span = tracer.start_span(
-                    name=f"{record.topic()} process", links=links, kind=SpanKind.CONSUMER
+                    name=f"{record.topic()} process",
+                    links=links,
+                    kind=SpanKind.CONSUMER,
                 )
 
-                bootstrap_servers = KafkaPropertiesExtractor.extract_bootstrap_servers(instance)
                 _enrich_span(
                     instance._current_consume_span,
                     record.topic(),
-                    bootstrap_servers,
                     record.partition(),
                     record.offset(),
                     operation=MessagingOperationValues.PROCESS,
-
                 )
         instance._current_context_token = context.attach(
-            trace.set_span_in_context(instance._current_consume_span))
+            trace.set_span_in_context(instance._current_consume_span)
+        )
 
-        return record
+        return record

instrumentation/opentelemetry-instrumentation-confluent-kafka/src/opentelemetry/instrumentation/confluent_kafka/package.py

@@ -13,4 +13,4 @@
 # limitations under the License.
 
 
-_instruments = ("confluent-kafka ~= 1.8.2",)
+_instruments = ("confluent-kafka ~= 1.8.2",)

instrumentation/opentelemetry-instrumentation-confluent-kafka/src/opentelemetry/instrumentation/confluent_kafka/utils.py

@@ -1,16 +1,12 @@
-import json
-import trace
 from logging import getLogger
 from typing import List, Optional
 
-from opentelemetry import propagate
 from opentelemetry.propagators import textmap
 from opentelemetry.semconv.trace import (
-    SpanAttributes,
-    MessagingOperationValues,
     MessagingDestinationKindValues,
+    MessagingOperationValues,
+    SpanAttributes,
 )
-from opentelemetry.trace import Link, SpanKind
 
 _LOG = getLogger(__name__)
 
@@ -68,26 +64,25 @@ def set(self, carrier: textmap.CarrierT, key: str, value: str) -> None:
 
 
 _kafka_getter = KafkaContextGetter()
+
+
 def _enrich_span(
     span,
     topic,
-    bootstrap_servers: List[str],
     partition: Optional[int] = None,
     offset: Optional[int] = None,
     operation: Optional[MessagingOperationValues] = None,
-
 ):
 
     if not span.is_recording():
         return
 
     span.set_attribute(SpanAttributes.MESSAGING_SYSTEM, "kafka")
     span.set_attribute(SpanAttributes.MESSAGING_DESTINATION, topic)
-    span.set_attribute(SpanAttributes.MESSAGING_KAFKA_PARTITION, partition)
 
-    span.set_attribute(
-        SpanAttributes.MESSAGING_URL, json.dumps(bootstrap_servers)
-    )
+    if partition:
+        span.set_attribute(SpanAttributes.MESSAGING_KAFKA_PARTITION, partition)
+
     span.set_attribute(
         SpanAttributes.MESSAGING_DESTINATION_KIND,
         MessagingDestinationKindValues.QUEUE.value,
@@ -101,7 +96,10 @@ def _enrich_span(
     # https://stackoverflow.com/questions/65935155/identify-and-find-specific-message-in-kafka-topic
     # A message within Kafka is uniquely defined by its topic name, topic partition and offset.
     if partition and offset and topic:
-        span.set_attribute(SpanAttributes.MESSAGING_MESSAGE_ID, f"{topic}.{partition}.{offset}")
+        span.set_attribute(
+            SpanAttributes.MESSAGING_MESSAGE_ID,
+            f"{topic}.{partition}.{offset}",
+        )
 
 
 _kafka_setter = KafkaContextSetter()

instrumentation/opentelemetry-instrumentation-confluent-kafka/src/opentelemetry/instrumentation/confluent_kafka/version.py

@@ -12,4 +12,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-__version__ = "0.27b0"
+__version__ = "0.31b0"

instrumentation/opentelemetry-instrumentation-confluent-kafka/tests/test_instrumentation.py

@@ -13,41 +13,44 @@
 # limitations under the License.
 from unittest import TestCase
 
-
-from confluent_kafka import Producer, Consumer
-
-from kafka_instrumentation import ConfluentKafkaInstrumentor, ProxiedProducer, ProxiedConsumer
+from confluent_kafka import Consumer, Producer
+from opentelemetry.instrumentation.confluent_kafka import (
+    ConfluentKafkaInstrumentor,
+    ProxiedConsumer,
+    ProxiedProducer,
+)
 
 
 class TestConfluentKafka(TestCase):
     def test_instrument_api(self) -> None:
         instrumentation = ConfluentKafkaInstrumentor()
 
-        p = Producer({'bootstrap.servers': 'localhost:29092'})
-        p = instrumentation.instrument_producer(p)
-
-        self.assertEqual(p.__class__, ProxiedProducer)
+        producer = Producer({"bootstrap.servers": "localhost:29092"})
+        producer = instrumentation.instrument_producer(producer)
 
-        p = instrumentation.uninstrument_producer(p)
-        self.assertEqual(p.__class__, Producer)
+        self.assertEqual(producer.__class__, ProxiedProducer)
 
-        p = Producer({'bootstrap.servers': 'localhost:29092'})
-        p = instrumentation.instrument_producer(p)
+        producer = instrumentation.uninstrument_producer(producer)
+        self.assertEqual(producer.__class__, Producer)
 
-        self.assertEqual(p.__class__, ProxiedProducer)
+        producer = Producer({"bootstrap.servers": "localhost:29092"})
+        producer = instrumentation.instrument_producer(producer)
 
-        p = instrumentation.uninstrument_producer(p)
-        self.assertEqual(p.__class__, Producer)
+        self.assertEqual(producer.__class__, ProxiedProducer)
 
-        c = Consumer({
-            'bootstrap.servers': 'localhost:29092',
-            'group.id': 'mygroup',
-            'auto.offset.reset': 'earliest'
-        })
+        producer = instrumentation.uninstrument_producer(producer)
+        self.assertEqual(producer.__class__, Producer)
 
-        c = instrumentation.instrument_consumer(c)
-        self.assertEqual(c.__class__, ProxiedConsumer)
+        consumer = Consumer(
+            {
+                "bootstrap.servers": "localhost:29092",
+                "group.id": "mygroup",
+                "auto.offset.reset": "earliest",
+            }
+        )
 
-        c = instrumentation.uninstrument_consumer(c)
-        self.assertEqual(c.__class__, Consumer)
+        consumer = instrumentation.instrument_consumer(consumer)
+        self.assertEqual(consumer.__class__, ProxiedConsumer)
 
+        consumer = instrumentation.uninstrument_consumer(consumer)
+        self.assertEqual(consumer.__class__, Consumer)

opentelemetry-contrib-instrumentations/setup.cfg

@@ -37,6 +37,7 @@ install_requires =
 	opentelemetry-instrumentation-boto3sqs==0.31b0
 	opentelemetry-instrumentation-botocore==0.31b0
 	opentelemetry-instrumentation-celery==0.31b0
+	opentelemetry-instrumentation-confluent-kafka==0.31b0
 	opentelemetry-instrumentation-dbapi==0.31b0
 	opentelemetry-instrumentation-django==0.31b0
 	opentelemetry-instrumentation-elasticsearch==0.31b0

opentelemetry-instrumentation/src/opentelemetry/instrumentation/bootstrap_gen.py

@@ -48,6 +48,10 @@
         "library": "celery >= 4.0, < 6.0",
         "instrumentation": "opentelemetry-instrumentation-celery==0.31b0",
     },
+    "confluent-kafka": {
+        "library": "confluent-kafka ~= 1.8.2",
+        "instrumentation": "opentelemetry-instrumentation-confluent-kafka==0.31b0",
+    },
     "django": {
         "library": "django >= 1.10",
         "instrumentation": "opentelemetry-instrumentation-django==0.31b0",