update flag

Author: lalitb

CONTRIBUTING.md

@@ -169,7 +169,7 @@ It's important to regularly review and remove the `otel_unstable` flag from the
 The potential features include:
 
 - Stable and non-experimental features that compliant to specification, and have a feature flag to minimize compilation size. Example: feature flags for signals (like `logs`, `traces`, `metrics`) and runtimes (`rt-tokio`, `rt-tokio-current-thread`, `rt-async-std`).
-- Stable and non-experimental features, although not part of the specification, are crucial for enhancing the tracing/log crate's functionality or boosting performance. These features are also subject to discussion and approval by the OpenTelemetry Rust Maintainers. An example of such a feature is `logs_level_enabled`.
+- Stable and non-experimental features, although not part of the specification, are crucial for enhancing the tracing/log crate's functionality or boosting performance. These features are also subject to discussion and approval by the OpenTelemetry Rust Maintainers.
 
 All such features should adhere to naming convention  `<signal>_<feature_name>`
 

opentelemetry-appender-log/Cargo.toml

@@ -21,14 +21,14 @@ opentelemetry-semantic-conventions = { path = "../opentelemetry-semantic-convent
 ] }
 
 [features]
-logs_level_enabled = ["opentelemetry/logs_level_enabled"]
+experimental_logs_level_enabled = ["opentelemetry/experimental_logs_level_enabled"]
 with-serde = ["log/kv_serde", "serde"]
 experimental_metadata_attributes = ["dep:opentelemetry-semantic-conventions"]
 
 [dev-dependencies]
 opentelemetry_sdk = { path = "../opentelemetry-sdk", features = [
   "testing",
-  "logs_level_enabled",
+  "experimental_logs_level_enabled",
 ] }
 opentelemetry-stdout = { path = "../opentelemetry-stdout", features = ["logs"] }
 log = { workspace = true, features = ["kv_serde"] }

opentelemetry-appender-log/src/lib.rs

@@ -89,7 +89,7 @@
 //!
 //! This library provides the following Cargo features:
 //!
-//! - `logs_level_enabled`: Allow users to control the log level.
+//! - `experimental_logs_level_enabled`: Allow users to control the log level.
 //! - `with-serde`: Support complex values as attributes without stringifying them.
 //!
 //! [Logs Bridge API]: https://opentelemetry.io/docs/specs/otel/logs/bridge-api/
@@ -117,11 +117,11 @@ where
     L: Logger + Send + Sync,
 {
     fn enabled(&self, _metadata: &Metadata) -> bool {
-        #[cfg(feature = "logs_level_enabled")]
+        #[cfg(feature = "experimental_logs_level_enabled")]
         return self
             .logger
             .event_enabled(severity_of_level(_metadata.level()), _metadata.target());
-        #[cfg(not(feature = "logs_level_enabled"))]
+        #[cfg(not(feature = "experimental_logs_level_enabled"))]
         true
     }
 
@@ -770,9 +770,9 @@ mod tests {
         // As a result of using `with_simple_exporter` while building the logger provider,
         // the processor used is a `SimpleLogProcessor` which has an implementation of `event_enabled`
         // that always returns true.
-        #[cfg(feature = "logs_level_enabled")]
+        #[cfg(feature = "experimental_logs_level_enabled")]
         assert!(otel_log_appender.enabled(&log::Metadata::builder().build()));
-        #[cfg(not(feature = "logs_level_enabled"))]
+        #[cfg(not(feature = "experimental_logs_level_enabled"))]
         assert!(otel_log_appender.enabled(&log::Metadata::builder().build()));
     }
 

opentelemetry-appender-tracing/Cargo.toml

@@ -33,10 +33,10 @@ pprof = { version = "0.13", features = ["flamegraph", "criterion"] }
 
 [features]
 experimental_metadata_attributes = ["dep:tracing-log"]
-logs_level_enabled = ["opentelemetry/logs_level_enabled"]
+logs_level_enabled = ["opentelemetry/experimental_logs_level_enabled"]
 
 
 [[bench]]
 name = "logs"
 harness = false
-required-features = ["logs_level_enabled"]
+required-features = ["experimental_logs_level_enabled"]

opentelemetry-appender-tracing/src/layer.rs

@@ -184,7 +184,7 @@ where
         self.logger.emit(log_record);
     }
 
-    #[cfg(feature = "logs_level_enabled")]
+    #[cfg(feature = "experimental_logs_level_enabled")]
     fn event_enabled(
         &self,
         _event: &tracing_core::Event<'_>,

opentelemetry-sdk/Cargo.toml

@@ -46,7 +46,7 @@ default = ["trace", "metrics", "logs", "internal-logs"]
 trace = ["opentelemetry/trace", "rand", "async-trait", "percent-encoding"]
 jaeger_remote_sampler = ["trace", "opentelemetry-http", "http", "serde", "serde_json", "url"]
 logs = ["opentelemetry/logs", "async-trait", "serde_json"]
-logs_level_enabled = ["logs", "opentelemetry/logs_level_enabled"]
+experimental_logs_level_enabled = ["logs", "opentelemetry/experimental_logs_level_enabled"]
 metrics = ["opentelemetry/metrics", "glob", "async-trait"]
 testing = ["opentelemetry/testing", "trace", "metrics", "logs", "rt-async-std", "rt-tokio", "rt-tokio-current-thread", "tokio/macros", "tokio/rt-multi-thread"]
 rt-tokio = ["tokio", "tokio-stream"]

opentelemetry-sdk/src/export/logs/mod.rs

@@ -3,7 +3,7 @@ use crate::logs::LogRecord;
 use crate::logs::{LogError, LogResult};
 use crate::Resource;
 use async_trait::async_trait;
-#[cfg(feature = "logs_level_enabled")]
+#[cfg(feature = "experimental_logs_level_enabled")]
 use opentelemetry::logs::Severity;
 use opentelemetry::InstrumentationScope;
 use std::fmt::Debug;
@@ -85,7 +85,7 @@ pub trait LogExporter: Send + Sync + Debug {
     async fn export(&mut self, batch: LogBatch<'_>) -> LogResult<()>;
     /// Shuts down the exporter.
     fn shutdown(&mut self) {}
-    #[cfg(feature = "logs_level_enabled")]
+    #[cfg(feature = "experimental_logs_level_enabled")]
     /// Chek if logs are enabled.
     fn event_enabled(&self, _level: Severity, _target: &str, _name: &str) -> bool {
         // By default, all logs are enabled

opentelemetry-sdk/src/lib.rs

@@ -89,7 +89,7 @@
 //!
 //! For `logs` the following feature flags are available:
 //!
-//! * `logs_level_enabled`: control the log level
+//! * `experimental_logs_level_enabled`: control the log level
 //!
 //! Support for recording and exporting telemetry asynchronously and perform
 //! metrics aggregation can be added via the following flags:

opentelemetry-sdk/src/logs/log_emitter.rs

@@ -3,7 +3,7 @@ use crate::{export::logs::LogExporter, runtime::RuntimeChannel, Resource};
 use crate::{logs::LogError, logs::LogResult};
 use opentelemetry::{otel_debug, trace::TraceContextExt, Context, InstrumentationScope};
 
-#[cfg(feature = "logs_level_enabled")]
+#[cfg(feature = "experimental_logs_level_enabled")]
 use opentelemetry::logs::Severity;
 
 use std::time::SystemTime;
@@ -282,7 +282,7 @@ impl opentelemetry::logs::Logger for Logger {
         }
     }
 
-    #[cfg(feature = "logs_level_enabled")]
+    #[cfg(feature = "experimental_logs_level_enabled")]
     fn event_enabled(&self, level: Severity, target: &str) -> bool {
         let provider = self.provider();
 

opentelemetry-sdk/src/logs/log_processor.rs

@@ -9,7 +9,7 @@ use futures_util::{
     future::{self, Either},
     {pin_mut, stream, StreamExt as _},
 };
-#[cfg(feature = "logs_level_enabled")]
+#[cfg(feature = "experimental_logs_level_enabled")]
 use opentelemetry::logs::Severity;
 use opentelemetry::{otel_debug, otel_error, otel_warn, InstrumentationScope};
 
@@ -61,7 +61,7 @@ pub trait LogProcessor: Send + Sync + Debug {
     /// After shutdown returns the log processor should stop processing any logs.
     /// It's up to the implementation on when to drop the LogProcessor.
     fn shutdown(&self) -> LogResult<()>;
-    #[cfg(feature = "logs_level_enabled")]
+    #[cfg(feature = "experimental_logs_level_enabled")]
     /// Check if logging is enabled
     fn event_enabled(&self, _level: Severity, _target: &str, _name: &str) -> bool {
         // By default, all logs are enabled