chore: Prepare 0.29 release (open-telemetry#2844)

Author: cijothomas

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## vNext
 
+## 0.29.0
+
+Released 2025-Mar-21
+
 Fixes [1682](https://github.com/open-telemetry/opentelemetry-rust/issues/1682).
 "spec_unstable_logs_enabled" feature now do not suppress logs for other layers.
 
@@ -53,6 +57,7 @@ of string.
   instead of previously reporting the user provided attribute "error".
   `error!(....error = &OTelSdkError::AlreadyShutdown as &dyn std::error::Error...)`
 - perf - small perf improvement by avoiding string allocation of `target`
+- Update `opentelemetry` dependency version to 0.29.
 
 ## 0.28.1
 

Cargo.toml

@@ -12,12 +12,12 @@ rust-version = "1.75.0"
 
 [dependencies]
 log = { workspace = true, optional = true }
-opentelemetry = { version = "0.28", path = "../opentelemetry", features = ["logs"] }
+opentelemetry = { version = "0.29", path = "../opentelemetry", features = ["logs"] }
 tracing = { workspace = true, features = ["std"]}
 tracing-core = { workspace = true }
 tracing-log = { workspace = true, optional = true }
 tracing-subscriber = { workspace = true, features = ["registry", "std"] }
-tracing-opentelemetry = { workspace = true, optional = true }
+# tracing-opentelemetry = { workspace = true, optional = true }
 
 [dev-dependencies]
 log = { workspace = true }
@@ -36,7 +36,8 @@ pprof = { version = "0.14", features = ["flamegraph", "criterion"] }
 default = []
 experimental_metadata_attributes = ["dep:tracing-log"]
 spec_unstable_logs_enabled = ["opentelemetry/spec_unstable_logs_enabled"]
-experimental_use_tracing_span_context = ["tracing-opentelemetry"]
+# TODO - Enable this in 0.29.1 (once tracing-opentelemetry v0.30 is released)
+# experimental_use_tracing_span_context = ["tracing-opentelemetry"]
 
 
 [[bench]]

src/layer.rs

@@ -244,25 +244,25 @@ where
         // Visit fields.
         event.record(&mut visitor);
 
-        #[cfg(feature = "experimental_use_tracing_span_context")]
-        if let Some(span) = _ctx.event_span(event) {
-            use tracing_opentelemetry::OtelData;
-            let opt_span_id = span
-                .extensions()
-                .get::<OtelData>()
-                .and_then(|otd| otd.builder.span_id);
-
-            let opt_trace_id = span.scope().last().and_then(|root_span| {
-                root_span
-                    .extensions()
-                    .get::<OtelData>()
-                    .and_then(|otd| otd.builder.trace_id)
-            });
-
-            if let Some((trace_id, span_id)) = opt_trace_id.zip(opt_span_id) {
-                log_record.set_trace_context(trace_id, span_id, None);
-            }
-        }
+        // #[cfg(feature = "experimental_use_tracing_span_context")]
+        // if let Some(span) = _ctx.event_span(event) {
+        //     use tracing_opentelemetry::OtelData;
+        //     let opt_span_id = span
+        //         .extensions()
+        //         .get::<OtelData>()
+        //         .and_then(|otd| otd.builder.span_id);
+
+        //     let opt_trace_id = span.scope().last().and_then(|root_span| {
+        //         root_span
+        //             .extensions()
+        //             .get::<OtelData>()
+        //             .and_then(|otd| otd.builder.trace_id)
+        //     });
+
+        //     if let Some((trace_id, span_id)) = opt_trace_id.zip(opt_span_id) {
+        //         log_record.set_trace_context(trace_id, span_id, None);
+        //     }
+        // }
 
         //emit record
         self.logger.emit(log_record);
@@ -673,66 +673,66 @@ mod tests {
         }
     }
 
-    #[cfg(feature = "experimental_use_tracing_span_context")]
-    #[test]
-    fn tracing_appender_inside_tracing_crate_context() {
-        use opentelemetry_sdk::trace::InMemorySpanExporterBuilder;
+    // #[cfg(feature = "experimental_use_tracing_span_context")]
+    // #[test]
+    // fn tracing_appender_inside_tracing_crate_context() {
+    //     use opentelemetry_sdk::trace::InMemorySpanExporterBuilder;
 
-        // Arrange
-        let exporter: InMemoryLogExporter = InMemoryLogExporter::default();
-        let logger_provider = SdkLoggerProvider::builder()
-            .with_simple_exporter(exporter.clone())
-            .build();
+    //     // Arrange
+    //     let exporter: InMemoryLogExporter = InMemoryLogExporter::default();
+    //     let logger_provider = SdkLoggerProvider::builder()
+    //         .with_simple_exporter(exporter.clone())
+    //         .build();
 
-        // setup tracing layer to compare trace/span IDs against
-        let span_exporter = InMemorySpanExporterBuilder::new().build();
-        let tracer_provider = SdkTracerProvider::builder()
-            .with_simple_exporter(span_exporter.clone())
-            .build();
-        let tracer = tracer_provider.tracer("test-tracer");
+    //     // setup tracing layer to compare trace/span IDs against
+    //     let span_exporter = InMemorySpanExporterBuilder::new().build();
+    //     let tracer_provider = SdkTracerProvider::builder()
+    //         .with_simple_exporter(span_exporter.clone())
+    //         .build();
+    //     let tracer = tracer_provider.tracer("test-tracer");
 
-        let level_filter = tracing_subscriber::filter::LevelFilter::ERROR;
-        let log_layer =
-            layer::OpenTelemetryTracingBridge::new(&logger_provider).with_filter(level_filter);
+    //     let level_filter = tracing_subscriber::filter::LevelFilter::ERROR;
+    //     let log_layer =
+    //         layer::OpenTelemetryTracingBridge::new(&logger_provider).with_filter(level_filter);
 
-        let subscriber = tracing_subscriber::registry()
-            .with(log_layer)
-            .with(tracing_opentelemetry::layer().with_tracer(tracer));
+    //     let subscriber = tracing_subscriber::registry()
+    //         .with(log_layer)
+    //         .with(tracing_opentelemetry::layer().with_tracer(tracer));
 
-        // Avoiding global subscriber.init() as that does not play well with unit tests.
-        let _guard = tracing::subscriber::set_default(subscriber);
+    //     // Avoiding global subscriber.init() as that does not play well with unit tests.
+    //     let _guard = tracing::subscriber::set_default(subscriber);
 
-        // Act
-        tracing::error_span!("outer-span").in_scope(|| {
-            error!("first-event");
+    //     // Act
+    //     tracing::error_span!("outer-span").in_scope(|| {
+    //         error!("first-event");
 
-            tracing::error_span!("inner-span").in_scope(|| {
-                error!("second-event");
-            });
-        });
+    //         tracing::error_span!("inner-span").in_scope(|| {
+    //             error!("second-event");
+    //         });
+    //     });
 
-        assert!(logger_provider.force_flush().is_ok());
+    //     assert!(logger_provider.force_flush().is_ok());
 
-        let logs = exporter.get_emitted_logs().expect("No emitted logs");
-        assert_eq!(logs.len(), 2, "Expected 2 logs, got: {logs:?}");
+    //     let logs = exporter.get_emitted_logs().expect("No emitted logs");
+    //     assert_eq!(logs.len(), 2, "Expected 2 logs, got: {logs:?}");
 
-        let spans = span_exporter.get_finished_spans().unwrap();
-        assert_eq!(spans.len(), 2);
+    //     let spans = span_exporter.get_finished_spans().unwrap();
+    //     assert_eq!(spans.len(), 2);
 
-        let trace_id = spans[0].span_context.trace_id();
-        assert_eq!(trace_id, spans[1].span_context.trace_id());
-        let inner_span_id = spans[0].span_context.span_id();
-        let outer_span_id = spans[1].span_context.span_id();
-        assert_eq!(outer_span_id, spans[0].parent_span_id);
+    //     let trace_id = spans[0].span_context.trace_id();
+    //     assert_eq!(trace_id, spans[1].span_context.trace_id());
+    //     let inner_span_id = spans[0].span_context.span_id();
+    //     let outer_span_id = spans[1].span_context.span_id();
+    //     assert_eq!(outer_span_id, spans[0].parent_span_id);
 
-        let trace_ctx0 = logs[0].record.trace_context().unwrap();
-        let trace_ctx1 = logs[1].record.trace_context().unwrap();
+    //     let trace_ctx0 = logs[0].record.trace_context().unwrap();
+    //     let trace_ctx1 = logs[1].record.trace_context().unwrap();
 
-        assert_eq!(trace_ctx0.trace_id, trace_id);
-        assert_eq!(trace_ctx1.trace_id, trace_id);
-        assert_eq!(trace_ctx0.span_id, outer_span_id);
-        assert_eq!(trace_ctx1.span_id, inner_span_id);
-    }
+    //     assert_eq!(trace_ctx0.trace_id, trace_id);
+    //     assert_eq!(trace_ctx1.trace_id, trace_id);
+    //     assert_eq!(trace_ctx0.span_id, outer_span_id);
+    //     assert_eq!(trace_ctx1.span_id, inner_span_id);
+    // }
 
     #[test]
     fn tracing_appender_standalone_with_tracing_log() {