fix: Remove unnecessary serde_json dependency from logs feature and ensure opentelemetry-proto compatibility (#3114)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: cijothomas <[email protected]>

Author: Copilot

opentelemetry-proto/Cargo.toml

@@ -49,7 +49,7 @@ testing = ["opentelemetry/testing"]
 # add ons
 internal-logs = ["opentelemetry/internal-logs"]
 with-schemars = ["schemars"]
-with-serde = ["serde", "const-hex", "base64"]
+with-serde = ["serde", "const-hex", "base64", "serde_json"]
 
 [dependencies]
 tonic = { workspace = true, optional = true, features = ["codegen", "prost"] }
@@ -58,6 +58,7 @@ opentelemetry = { workspace = true }
 opentelemetry_sdk = { workspace = true }
 schemars = { workspace = true, optional = true }
 serde = { workspace = true, optional = true, features = ["serde_derive"] }
+serde_json = { workspace = true, optional = true }
 const-hex = { workspace = true, optional = true }
 base64 = { workspace = true, optional = true }
 

opentelemetry-sdk/Cargo.toml

@@ -42,7 +42,7 @@ pprof = { version = "0.14", features = ["flamegraph", "criterion"] }
 default = ["trace", "metrics", "logs", "internal-logs"]
 trace = ["opentelemetry/trace", "rand", "percent-encoding"]
 jaeger_remote_sampler = ["trace", "opentelemetry-http", "http", "serde", "serde_json", "url", "experimental_async_runtime"]
-logs = ["opentelemetry/logs", "serde_json"]
+logs = ["opentelemetry/logs"]
 spec_unstable_logs_enabled = ["logs", "opentelemetry/spec_unstable_logs_enabled"]
 metrics = ["opentelemetry/metrics"]
 testing = ["opentelemetry/testing", "trace", "metrics", "logs", "rt-tokio", "rt-tokio-current-thread", "tokio/macros", "tokio/rt-multi-thread"]