fix(confluent-kafka): remove deprecated messaging.url, fix CHANGELOG link

Signed-off-by: alliasgher <alliasgher123@gmail.com>

Author: alliasgher

CHANGELOG.md

@@ -13,8 +13,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
-- `opentelemetry-instrumentation-confluent-kafka`: Populate `messaging.url`, `server.address` and `server.port` span attributes from the producer/consumer `bootstrap.servers` config; previously `KafkaPropertiesExtractor.extract_bootstrap_servers` was defined but never called
-  ([#4104](https://github.com/open-telemetry/opentelemetry-python-contrib/issues/4104))
+- `opentelemetry-instrumentation-confluent-kafka`: Populate `server.address` and `server.port` span attributes from the producer/consumer `bootstrap.servers` config; previously `KafkaPropertiesExtractor.extract_bootstrap_servers` was defined but never called
+  ([#4423](https://github.com/open-telemetry/opentelemetry-python-contrib/pull/4423))
 
 ### Breaking changes
 

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

@@ -130,13 +130,11 @@ def _get_links_from_records(records):
 
 
 def _set_bootstrap_servers_attributes(span, bootstrap_servers):
-    """Populate messaging.url, server.address, server.port from a
-    bootstrap.servers string (e.g. ``host1:9092,host2:9092``)."""
+    """Populate server.address and server.port from a bootstrap.servers
+    string (e.g. ``host1:9092,host2:9092``)."""
     if not bootstrap_servers:
         return
 
-    span.set_attribute(SpanAttributes.MESSAGING_URL, bootstrap_servers)
-
     first_broker = bootstrap_servers.split(",")[0].strip()
     if not first_broker:
         return

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

@@ -465,10 +465,6 @@ def test_producer_sets_bootstrap_servers_attributes(self) -> None:
         producer.produce(topic="topic-1", key="k", value="v")
 
         span = self.memory_exporter.get_finished_spans()[0]
-        self.assertEqual(
-            span.attributes[SpanAttributes.MESSAGING_URL],
-            "broker-a:9092,broker-b:9093",
-        )
         self.assertEqual(span.attributes[SERVER_ADDRESS], "broker-a")
         self.assertEqual(span.attributes[SERVER_PORT], 9092)
 
@@ -495,9 +491,5 @@ def test_consumer_sets_bootstrap_servers_attributes(self) -> None:
             for s in self.memory_exporter.get_finished_spans()
             if s.name == "topic-1 process"
         )
-        self.assertEqual(
-            process_span.attributes[SpanAttributes.MESSAGING_URL],
-            "broker-1:9092",
-        )
         self.assertEqual(process_span.attributes[SERVER_ADDRESS], "broker-1")
         self.assertEqual(process_span.attributes[SERVER_PORT], 9092)