fix build

Author: lalitb

opentelemetry-otlp/tests/integration_test/Cargo.toml

@@ -21,9 +21,9 @@ opentelemetry-semantic-conventions = { path = "../../../opentelemetry-semantic-c
 
 
 [features]
-hyper-client = ["opentelemetry-otlp/hyper-client", "opentelemetry-otlp/trace", "opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
-reqwest-client = ["opentelemetry-otlp/reqwest-client", "opentelemetry-otlp/trace","opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
-reqwest-blocking-client = ["opentelemetry-otlp/reqwest-blocking-client", "opentelemetry-otlp/trace","opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
+hyper-client = ["opentelemetry-otlp/hyper-client", "opentelemetry-otlp/http-proto", "opentelemetry-otlp/trace", "opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
+reqwest-client = ["opentelemetry-otlp/reqwest-client", "opentelemetry-otlp/http-proto", "opentelemetry-otlp/trace","opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
+reqwest-blocking-client = ["opentelemetry-otlp/reqwest-blocking-client", "opentelemetry-otlp/http-proto", "opentelemetry-otlp/trace","opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
 tonic-client = ["opentelemetry-otlp/grpc-tonic", "opentelemetry-otlp/trace", "opentelemetry-otlp/logs", "opentelemetry-otlp/metrics"]
 
 # Keep tonic as the default client

opentelemetry-otlp/tests/integration_test/tests/logs.rs

@@ -5,7 +5,7 @@ use log::{info, Level};
 use opentelemetry::logs::LogError;
 use opentelemetry::KeyValue;
 use opentelemetry_appender_log::OpenTelemetryLogBridge;
-use opentelemetry_otlp::{LogExporter, WithExportConfig};
+use opentelemetry_otlp::LogExporter;
 use opentelemetry_sdk::logs::LoggerProvider;
 use opentelemetry_sdk::{logs as sdklogs, runtime, Resource};
 use std::error::Error;
@@ -16,12 +16,15 @@ fn init_logs() -> Result<sdklogs::LoggerProvider, LogError> {
     let exporter_builder = LogExporter::builder();
     #[cfg(feature = "tonic-client")]
     let exporter_builder = exporter_builder.with_tonic();
-    #[cfg(feature = "hyper-client")]
+    #[cfg(any(
+        feature = "hyper-client",
+        feature = "reqwest-client",
+        feature = "reqwest-blocking-client"
+    ))]
     let exporter_builder = exporter_builder.with_http();
 
-    let exporter = exporter_builder
-        .with_endpoint("0.0.0.0:4317")
-        .build()?;
+    let exporter = exporter_builder.build()?;
+    //let exporter = LogExporter::builder().with_tonic().build()?;
 
     Ok(LoggerProvider::builder()
         .with_batch_exporter(exporter, runtime::Tokio)

opentelemetry-otlp/tests/integration_test/tests/traces.rs

@@ -8,6 +8,8 @@ use opentelemetry::{
     trace::{TraceContextExt, Tracer},
     Key, KeyValue,
 };
+use opentelemetry_otlp::SpanExporter;
+
 use opentelemetry_proto::tonic::trace::v1::TracesData;
 use opentelemetry_sdk::{runtime, trace as sdktrace, Resource};
 use std::error::Error;
@@ -16,9 +18,18 @@ use std::io::Write;
 use std::os::unix::fs::MetadataExt;
 
 fn init_tracer_provider() -> Result<sdktrace::TracerProvider, TraceError> {
-    let exporter = opentelemetry_otlp::SpanExporter::builder()
-        .with_tonic()
-        .build()?;
+    let exporter_builder = SpanExporter::builder();
+    #[cfg(feature = "tonic-client")]
+    let exporter_builder = exporter_builder.with_tonic();
+    #[cfg(any(
+        feature = "hyper-client",
+        feature = "reqwest-client",
+        feature = "reqwest-blocking-client"
+    ))]
+    let exporter_builder = exporter_builder.with_http();
+
+    let exporter = exporter_builder.build()?;
+
     Ok(opentelemetry_sdk::trace::TracerProvider::builder()
         .with_batch_exporter(exporter, runtime::Tokio)
         .with_config(

scripts/integration_tests.sh

@@ -1,11 +1,20 @@
-# Run tests with the grpc-tonic feature
-cd ./opentelemetry-otlp/tests/integration_test/tests && cargo test --no-default-features --features "tonic-client" -- --ignored
+set -e
+TEST_DIR="./opentelemetry-otlp/tests/integration_test/tests"
 
-# Run tests with the reqwest-client feature
-#cd ./opentelemetry-otlp/tests/integration_test/tests && cargo test --no-default-features --features "reqwest-client" -- --ignored
+if [ -d "$TEST_DIR" ]; then
+    cd "$TEST_DIR"
+    # Run tests with the grpc-tonic feature
+    cargo test --no-default-features --features "tonic-client" -- --ignored
 
-# Run tests with the reqwest-blocking-client feature
-#cd ./opentelemetry-otlp/tests/integration_test/tests && cargo test --no-default-features --features "reqwest-blocking-client" -- --ignored
+    # Run tests with the reqwest-client feature
+    cargo test --no-default-features --features "reqwest-client" -- --ignored
 
-# Run tests with the hyper-client feature
-#cd ./opentelemetry-otlp/tests/integration_test/tests && cargo test --no-default-features --features "hyper-client" -- --ignored
+    # TODO - Uncomment the following lines once the reqwest-blocking-client feature is working.
+    # cargo test --no-default-features --features "reqwest-blocking-client" -- --ignored
+
+    # Run tests with the hyper-client feature
+    cargo test --no-default-features --features "hyper-client" -- --ignored
+else
+    echo "Directory $TEST_DIR does not exist. Skipping tests."
+    exit 1
+fi