Merge branch 'arm-ci' of github.com:lalitb/opentelemetry-rust into ar…

…m-ci

Cargo.toml

@@ -19,6 +19,7 @@ debug = 1
 async-std = "1.10"
 async-trait = "0.1"
 bytes = "1"
+criterion = "0.5"
 futures-core = "0.3"
 futures-executor = "0.3"
 futures-util = { version = "0.3", default-features = false }
@@ -45,4 +46,4 @@ tokio-stream = "0.1.1"
 tracing = { version = "0.1", default-features = false }
 tracing-core = { version = "0.1", default-features = false }
 tracing-subscriber = { version = "0.3", default-features = false }
-url = { version = "2.2", default-features = false }
+url = { version = "=2.5.0", default-features = false } #pinning the version supporting rustc 1.65

opentelemetry-appender-tracing/Cargo.toml

@@ -24,8 +24,8 @@ log = { workspace = true }
 opentelemetry-stdout = { path = "../opentelemetry-stdout", features = ["logs"] }
 opentelemetry_sdk = { path = "../opentelemetry-sdk", features = ["logs", "testing"] }
 tracing-log = "0.2"
-async-trait = "0.1"
-criterion = "0.5.1"
+async-trait = { workspace = true }
+criterion = { workspace = true }
 
 [features]
 experimental_metadata_attributes = ["dep:tracing-log"]

opentelemetry-jaeger-propagator/Cargo.toml

@@ -25,7 +25,7 @@ opentelemetry = { version = "0.23", default-features = false, features = [
 ], path = "../opentelemetry" }
 
 [dev-dependencies]
-opentelemetry_sdk = { features = ["testing"], path = "../opentelemetry-sdk" }
+opentelemetry = { features = ["testing"], path = "../opentelemetry" }
 
 [features]
 default = []

opentelemetry-otlp/examples/basic-otlp-http/otel-collector-config.yaml

@@ -11,17 +11,17 @@ receivers:
  http:
 
 exporters:
- logging:
- loglevel: debug
+ debug:
+ verbosity: detailed
 
 service:
  pipelines:
  traces:
  receivers: [otlp]
- exporters: [logging]
+ exporters: [debug]
  metrics:
  receivers: [otlp]
- exporters: [logging]
+ exporters: [debug]
  logs:
  receivers: [otlp]
- exporters: [logging]
+ exporters: [debug]

opentelemetry-otlp/examples/basic-otlp-http/src/main.rs

@@ -50,19 +50,15 @@ fn init_tracer_provider() -> Result<sdktrace::TracerProvider, TraceError> {
 }
 
 fn init_metrics() -> Result<opentelemetry_sdk::metrics::SdkMeterProvider, MetricsError> {
- let provider = opentelemetry_otlp::new_pipeline()
+ opentelemetry_otlp::new_pipeline()
  .metrics(opentelemetry_sdk::runtime::Tokio)
  .with_exporter(
  opentelemetry_otlp::new_exporter()
  .http()
  .with_endpoint("http://localhost:4318/v1/metrics"),
  )
  .with_resource(RESOURCE.clone())
- .build();
- match provider {
- Ok(provider) => Ok(provider),
- Err(err) => Err(err),
- }
+ .build()
 }
 
 #[tokio::main]
@@ -85,6 +81,7 @@ async fn main() -> Result<(), Box<dyn Error + Send + Sync + 'static>> {
  );
 
  let meter_provider = result.unwrap();
+ global::set_meter_provider(meter_provider.clone());
 
  // Opentelemetry will not provide a global API to manage the logger
  // provider. Application users must manage the lifecycle of the logger

opentelemetry-otlp/examples/basic-otlp/otel-collector-config.yaml

@@ -11,17 +11,17 @@ receivers:
  http:
 
 exporters:
- logging:
- loglevel: debug
+ debug:
+ verbosity: detailed
 
 service:
  pipelines:
  traces:
  receivers: [otlp]
- exporters: [logging]
+ exporters: [debug]
  metrics:
  receivers: [otlp]
- exporters: [logging]
+ exporters: [debug]
  logs:
  receivers: [otlp]
- exporters: [logging]
+ exporters: [debug]

opentelemetry-otlp/tests/integration_test/Cargo.toml

@@ -9,11 +9,13 @@ publish = false
 once_cell = { workspace = true }
 opentelemetry = { path = "../../../opentelemetry", features = ["metrics", "logs"] }
 opentelemetry_sdk = { path = "../../../opentelemetry-sdk", features = ["rt-tokio", "logs", "testing"] }
-opentelemetry-otlp = { path = "../../../opentelemetry-otlp", features = ["tonic", "metrics", "logs"] }
-opentelemetry-semantic-conventions = { path = "../../../opentelemetry-semantic-conventions" }
-opentelemetry-proto = { path = "../../../opentelemetry-proto", features = ["gen-tonic-messages", "trace", "with-serde"] }
-opentelemetry-appender-log = { path = "../../../opentelemetry-appender-log", default-features = false}
+opentelemetry-proto = { path = "../../../opentelemetry-proto", features = ["gen-tonic-messages", "trace", "logs", "with-serde"] }
 log = { workspace = true }
 tokio = { version = "1.0", features = ["full"] }
 serde_json = "1"
 testcontainers = "0.15.0"
+
+[target.'cfg(unix)'.dependencies]
+opentelemetry-appender-log = { path = "../../../opentelemetry-appender-log", default-features = false}
+opentelemetry-otlp = { path = "../../../opentelemetry-otlp", features = ["tonic", "metrics", "logs"] }
+opentelemetry-semantic-conventions = { path = "../../../opentelemetry-semantic-conventions" }

opentelemetry-otlp/tests/integration_test/otel-collector-config.yaml

@@ -5,8 +5,6 @@ receivers:
  http:
 
 exporters:
- logging:
- loglevel: debug
  file:
  path: /testresults/result.json
 

opentelemetry-proto/Cargo.toml

@@ -60,4 +60,4 @@ hex = { version = "0.4.3", optional = true }
 tonic-build = { workspace = true }
 prost-build = { workspace = true }
 tempfile = "3.3.0"
-serde_json = "1.0"
+serde_json = { workspace = true }

opentelemetry-sdk/Cargo.toml

@@ -34,8 +34,7 @@ all-features = true
 rustdoc-args = ["--cfg", "docsrs"]
 
 [dev-dependencies]
-indexmap = "2.0"
-criterion = { version = "0.5", features = ["html_reports"] }
+criterion = { workspace = true, features = ["html_reports"] }
 temp-env = { workspace = true }
 
 [target.'cfg(not(target_os = "windows"))'.dev-dependencies]

opentelemetry-sdk/src/logs/log_emitter.rs

@@ -77,7 +77,7 @@ impl opentelemetry::logs::LoggerProvider for LoggerProvider {
 
  fn library_logger(&self, library: Arc<InstrumentationLibrary>) -> Self::Logger {
  // If the provider is shutdown, new logger will refer a no-op logger provider.
- if self.is_shutdown.load(std::sync::atomic::Ordering::Relaxed) {
+ if self.is_shutdown.load(Ordering::Relaxed) {
  return Logger::new(library, NOOP_LOGGER_PROVIDER.clone());
  }
  Logger::new(library, self.clone())