fix: Update OpenTelemetry API for v0.30

- Use SdkTracerProvider instead of TracerProvider
- Use KeyValue for resource attributes
- Use HTTP exporter instead of tonic/gRPC
- Fix unused variable warning

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>

Author: Cortex Dev

cortex-otel/src/otel_provider.rs

@@ -1,13 +1,15 @@
 //! OpenTelemetry provider implementation.
 
+#[cfg(feature = "otel")]
+use opentelemetry::KeyValue;
 #[cfg(feature = "otel")]
 use opentelemetry::trace::TracerProvider as _;
 #[cfg(feature = "otel")]
 use opentelemetry_otlp::WithExportConfig;
 #[cfg(feature = "otel")]
 use opentelemetry_sdk::{
-    Resource, runtime,
-    trace::{self as sdktrace, Sampler},
+    Resource,
+    trace::{Sampler, SdkTracerProvider},
 };
 #[cfg(feature = "otel")]
 use opentelemetry_semantic_conventions::resource::SERVICE_NAME;
@@ -21,7 +23,7 @@ use crate::config::OtelSettings;
 /// OpenTelemetry provider for Cortex CLI.
 #[cfg(feature = "otel")]
 pub struct OtelProvider {
-    tracer_provider: sdktrace::TracerProvider,
+    tracer_provider: SdkTracerProvider,
 }
 
 #[cfg(feature = "otel")]
@@ -34,23 +36,21 @@ impl OtelProvider {
 
         let endpoint = settings.endpoint.as_ref()?;
 
-        // Build resource
-        let mut resource_builder =
-            Resource::builder().with_attribute(SERVICE_NAME, settings.service_name.clone());
+        // Build resource attributes
+        let mut attrs = vec![KeyValue::new(SERVICE_NAME, settings.service_name.clone())];
 
         if let Some(version) = &settings.service_version {
-            resource_builder = resource_builder.with_attribute(
+            attrs.push(KeyValue::new(
                 opentelemetry_semantic_conventions::resource::SERVICE_VERSION,
                 version.clone(),
-            );
+            ));
         }
 
         for (key, value) in &settings.resource_attributes {
-            resource_builder = resource_builder
-                .with_attribute(opentelemetry::KeyValue::new(key.clone(), value.clone()));
+            attrs.push(KeyValue::new(key.clone(), value.clone()));
         }
 
-        let resource = resource_builder.build();
+        let resource = Resource::builder_empty().with_attributes(attrs).build();
 
         // Configure sampler
         let sampler = if settings.sampling_ratio >= 1.0 {
@@ -61,26 +61,26 @@ impl OtelProvider {
             Sampler::TraceIdRatioBased(settings.sampling_ratio)
         };
 
-        // Build OTLP exporter
+        // Build OTLP exporter using HTTP (more portable than tonic/gRPC)
         let exporter = opentelemetry_otlp::SpanExporter::builder()
-            .with_tonic()
+            .with_http()
             .with_endpoint(endpoint)
             .with_timeout(std::time::Duration::from_secs(settings.export_timeout_secs))
             .build()
             .ok()?;
 
         // Build tracer provider
-        let tracer_provider = sdktrace::TracerProvider::builder()
+        let tracer_provider = SdkTracerProvider::builder()
             .with_resource(resource)
             .with_sampler(sampler)
-            .with_batch_exporter(exporter, runtime::Tokio)
+            .with_batch_exporter(exporter)
             .build();
 
         Some(Self { tracer_provider })
     }
 
     /// Get trace context headers from the current span.
-    pub fn headers(span: &Span) -> HeaderMap {
+    pub fn headers(_span: &Span) -> HeaderMap {
         use opentelemetry::propagation::TextMapPropagator;
         use opentelemetry_sdk::propagation::TraceContextPropagator;