feat: clean up trace SDK APIs and remove integration tests for jaeger

Author: TommyCpp

opentelemetry-appender-tracing/src/layer.rs

@@ -212,7 +212,8 @@ mod tests {
     use opentelemetry::{logs::AnyValue, Key};
     use opentelemetry_sdk::logs::LoggerProvider;
     use opentelemetry_sdk::testing::logs::InMemoryLogsExporter;
-    use opentelemetry_sdk::trace::{config, Sampler, TracerProvider};
+    use opentelemetry_sdk::trace;
+    use opentelemetry_sdk::trace::{Sampler, TracerProvider};
     use tracing::error;
     use tracing_subscriber::layer::SubscriberExt;
 
@@ -300,7 +301,7 @@ mod tests {
 
         // setup tracing as well.
         let tracer_provider = TracerProvider::builder()
-            .with_config(config().with_sampler(Sampler::AlwaysOn))
+            .with_config(trace::Config::default().with_sampler(Sampler::AlwaysOn))
             .build();
         let tracer = tracer_provider.tracer("test-tracer");
 
@@ -464,7 +465,7 @@ mod tests {
 
         // setup tracing as well.
         let tracer_provider = TracerProvider::builder()
-            .with_config(config().with_sampler(Sampler::AlwaysOn))
+            .with_config(trace::Config::default().with_sampler(Sampler::AlwaysOn))
             .build();
         let tracer = tracer_provider.tracer("test-tracer");
 

opentelemetry-proto/src/proto/opentelemetry-proto

@@ -25,6 +25,10 @@
  reference or owned`LogData`. If the exporter needs to process the log data
  asynchronously, it should clone the log data to ensure it can be safely processed without
  lifetime issues.
+- Clean up public methods in SDK.
+    - [`TracerProvider::span_processors`] and [`TracerProvider::config`] was removed as it's not part of the spec.
+    - Added `non_exhaustive` annotation to [`trace::Config`]. Marked [`config`] as deprecated since it's only a wrapper for `Config::default`
+    - Removed [`Tracer::tracer_provder`] and [`Tracer::instrument_libraries`] as it's not part of the spec.
 
 ## v0.23.0
 
@@ -57,13 +61,7 @@
   - After `shutdown`, `LogProcessor` will not process any new logs
 - Moving LogRecord implementation to the SDK. [1702](https://github.com/open-telemetry/opentelemetry-rust/pull/1702).
     - Relocated `LogRecord` struct to SDK, as an implementation for the trait in the API.
-- **Breaking** [#1729](https://github.com/open-telemetry/opentelemetry-rust/pull/1729)
-  - Update the return type of `TracerProvider.span_processors()` from `&Vec<Box<dyn SpanProcessor>>` to `&[Box<dyn SpanProcessor>]`.
-  - Update the return type of `LoggerProvider.log_processors()` from `&Vec<Box<dyn LogProcessor>>` to `&[Box<dyn LogProcessor>]`.
-- Update `opentelemetry` dependency version to 0.23
-- Update `opentelemetry-http` dependency version to 0.12
-- **Breaking** [#1750](https://github.com/open-telemetry/opentelemetry-rust/pull/1729)
-  - Update the return type of `LoggerProvider.shutdown()` from `Vec<LogResult<()>>` to `LogResult<()>`.
+
 
 ## v0.22.1
 

opentelemetry-sdk/CHANGELOG.md

@@ -10,7 +10,8 @@ use opentelemetry::{
 };
 use opentelemetry_sdk::{
     export::trace::{ExportResult, SpanData, SpanExporter},
-    trace::{config, Sampler, TracerProvider},
+    trace,
+    trace::{Sampler, TracerProvider},
 };
 #[cfg(not(target_os = "windows"))]
 use pprof::criterion::{Output, PProfProfiler};
@@ -126,7 +127,9 @@ impl Display for Environment {
 
 fn parent_sampled_tracer(inner_sampler: Sampler) -> (TracerProvider, BoxedTracer) {
     let provider = TracerProvider::builder()
-        .with_config(config().with_sampler(Sampler::ParentBased(Box::new(inner_sampler))))
+        .with_config(
+            trace::Config::default().with_sampler(Sampler::ParentBased(Box::new(inner_sampler))),
+        )
         .with_simple_exporter(NoopExporter)
         .build();
     let tracer = provider.tracer(module_path!());

opentelemetry-sdk/benches/context.rs

@@ -17,7 +17,8 @@ use opentelemetry::trace::TracerProvider as _;
 use opentelemetry::Key;
 use opentelemetry_sdk::export::logs::{LogData, LogExporter};
 use opentelemetry_sdk::logs::{Logger, LoggerProvider};
-use opentelemetry_sdk::trace::{config, Sampler, TracerProvider};
+use opentelemetry_sdk::trace;
+use opentelemetry_sdk::trace::{Sampler, TracerProvider};
 
 #[derive(Debug)]
 struct VoidExporter;
@@ -51,7 +52,7 @@ fn log_benchmark_group<F: Fn(&Logger)>(c: &mut Criterion, name: &str, f: F) {
 
         // setup tracing as well.
         let tracer_provider = TracerProvider::builder()
-            .with_config(config().with_sampler(Sampler::AlwaysOn))
+            .with_config(trace::Config::default().with_sampler(Sampler::AlwaysOn))
             .build();
         let tracer = tracer_provider.tracer("bench-tracer");
 

opentelemetry-sdk/benches/log.rs

@@ -54,7 +54,7 @@ fn span_builder_benchmark_group(c: &mut Criterion) {
 
 fn not_sampled_provider() -> (sdktrace::TracerProvider, sdktrace::Tracer) {
     let provider = sdktrace::TracerProvider::builder()
-        .with_config(sdktrace::config().with_sampler(sdktrace::Sampler::AlwaysOff))
+        .with_config(sdktrace::Config::default().with_sampler(sdktrace::Sampler::AlwaysOff))
         .with_simple_exporter(NoopExporter)
         .build();
     let tracer = provider.tracer("not-sampled");

opentelemetry-sdk/benches/span_builder.rs

@@ -70,7 +70,7 @@ fn trace_benchmark_group<F: Fn(&sdktrace::Tracer)>(c: &mut Criterion, name: &str
 
     group.bench_function("always-sample", |b| {
         let provider = sdktrace::TracerProvider::builder()
-            .with_config(sdktrace::config().with_sampler(sdktrace::Sampler::AlwaysOn))
+            .with_config(sdktrace::Config::default().with_sampler(sdktrace::Sampler::AlwaysOn))
             .with_simple_exporter(VoidExporter)
             .build();
         let always_sample = provider.tracer("always-sample");
@@ -80,7 +80,7 @@ fn trace_benchmark_group<F: Fn(&sdktrace::Tracer)>(c: &mut Criterion, name: &str
 
     group.bench_function("never-sample", |b| {
         let provider = sdktrace::TracerProvider::builder()
-            .with_config(sdktrace::config().with_sampler(sdktrace::Sampler::AlwaysOff))
+            .with_config(sdktrace::Config::default().with_sampler(sdktrace::Sampler::AlwaysOff))
             .with_simple_exporter(VoidExporter)
             .build();
         let never_sample = provider.tracer("never-sample");

opentelemetry-sdk/benches/trace.rs

@@ -10,12 +10,14 @@ use std::env;
 use std::str::FromStr;
 
 /// Default trace configuration
+#[deprecated(since = "0.23.0", note = "Use Config::default() instead")]
 pub fn config() -> Config {
     Config::default()
 }
 
 /// Tracer configuration
 #[derive(Debug)]
+#[non_exhaustive]
 pub struct Config {
     /// The sampler that the sdk should use
     pub sampler: Box<dyn ShouldSample>,

opentelemetry-sdk/src/trace/config.rs

@@ -67,7 +67,7 @@ impl TracerProvider {
     }
 
     /// Config associated with this tracer
-    pub fn config(&self) -> &crate::trace::Config {
+    pub(crate) fn config(&self) -> &crate::trace::Config {
         &self.inner.config
     }
 

opentelemetry-sdk/src/trace/provider.rs

@@ -55,12 +55,12 @@ impl Tracer {
     }
 
     /// TracerProvider associated with this tracer.
-    pub fn provider(&self) -> Option<TracerProvider> {
+    pub(crate) fn provider(&self) -> Option<TracerProvider> {
         self.provider.upgrade().map(TracerProvider::new)
     }
 
     /// Instrumentation library information of this tracer.
-    pub fn instrumentation_library(&self) -> &InstrumentationLibrary {
+    pub(crate) fn instrumentation_library(&self) -> &InstrumentationLibrary {
         &self.instrumentation_lib
     }
 

opentelemetry-sdk/src/trace/tracer.rs

@@ -106,10 +106,7 @@ impl ZipkinPipelineBuilder {
                 ));
                 cfg
             } else {
-                Config {
-                    resource: Cow::Owned(Resource::empty()),
-                    ..Default::default()
-                }
+                Config::default().with_resource(Resource::empty())
             };
             (config, Endpoint::new(service_name, self.service_addr))
         } else {
@@ -119,11 +116,7 @@ impl ZipkinPipelineBuilder {
                 .unwrap()
                 .to_string();
             (
-                Config {
-                    // use a empty resource to prevent TracerProvider to assign a service name.
-                    resource: Cow::Owned(Resource::empty()),
-                    ..Default::default()
-                },
+                Config::default().with_resource(Resource::empty()),
                 Endpoint::new(service_name, self.service_addr),
             )
         }

opentelemetry-zipkin/src/exporter/mod.rs

@@ -1,5 +1 @@
-#COMPOSE_FILE=./opentelemetry-jaeger/tests/docker-compose.yaml
-#docker-compose -f $COMPOSE_FILE down -v &&
-#docker-compose -f $COMPOSE_FILE up --build --abort-on-container-exit --exit-code-from opentelemetry-jaeger
-
 cargo test ./opentelemetry-otlp/tests/integration_test/tests -- --ignored