Fix logs SDK dependency on trace feature

- Remove default-features from opentelemetry dependency in SDK
- Conditionally compile trace context extraction in logs
- Move trace-related imports behind feature gates
- Ensure logs work without trace feature while maintaining compatibility

Co-authored-by: cijothomas <[email protected]>

Author: Copilot

opentelemetry-sdk/Cargo.toml

@@ -11,7 +11,7 @@ rust-version = "1.75.0"
 autobenches = false
 
 [dependencies]
-opentelemetry = { version = "0.30", path = "../opentelemetry/" }
+opentelemetry = { version = "0.30", path = "../opentelemetry/", default-features = false }
 opentelemetry-http = { version = "0.30", path = "../opentelemetry-http", optional = true }
 futures-channel = { workspace = true }
 futures-executor = { workspace = true }

opentelemetry-sdk/src/logs/logger.rs

@@ -1,5 +1,7 @@
-use super::{SdkLogRecord, SdkLoggerProvider, TraceContext};
-use opentelemetry::{trace::TraceContextExt, Context, InstrumentationScope};
+use super::{SdkLogRecord, SdkLoggerProvider};
+#[cfg(feature = "trace")]
+use super::TraceContext;
+use opentelemetry::{Context, InstrumentationScope};
 
 #[cfg(feature = "spec_unstable_logs_enabled")]
 use opentelemetry::logs::Severity;
@@ -37,11 +39,15 @@ impl opentelemetry::logs::Logger for SdkLogger {
 
         //let mut log_record = record;
         if record.trace_context.is_none() {
-            Context::map_current(|cx| {
-                cx.has_active_span().then(|| {
-                    record.trace_context = Some(TraceContext::from(cx.span().span_context()))
-                })
-            });
+            #[cfg(feature = "trace")]
+            {
+                use opentelemetry::trace::TraceContextExt;
+                Context::map_current(|cx| {
+                    cx.has_active_span().then(|| {
+                        record.trace_context = Some(TraceContext::from(cx.span().span_context()))
+                    })
+                });
+            }
         }
         if record.observed_timestamp.is_none() {
             record.observed_timestamp = Some(now());

opentelemetry-sdk/src/logs/record.rs

@@ -1,8 +1,7 @@
 use crate::growable_array::GrowableArray;
 use opentelemetry::{
     logs::{AnyValue, Severity},
-    trace::{SpanContext, SpanId, TraceFlags, TraceId},
-    Key,
+    Key, SpanId, TraceFlags, TraceId,
 };
 use std::{borrow::Cow, time::SystemTime};
 
@@ -216,8 +215,9 @@ pub struct TraceContext {
     pub trace_flags: Option<TraceFlags>,
 }
 
-impl From<&SpanContext> for TraceContext {
-    fn from(span_context: &SpanContext) -> Self {
+#[cfg(feature = "trace")]
+impl From<&opentelemetry::trace::SpanContext> for TraceContext {
+    fn from(span_context: &opentelemetry::trace::SpanContext) -> Self {
         TraceContext {
             trace_id: span_context.trace_id(),
             span_id: span_context.span_id(),