[Tracing Appender] Propagate event name to exporters

opentelemetry-appender-tracing/Cargo.toml

@@ -13,7 +13,7 @@ rust-version = "1.65"
 [dependencies]
 opentelemetry = { version = "0.21", path = "../opentelemetry", features = ["logs"] }
 opentelemetry_sdk = { version = "0.21", path = "../opentelemetry-sdk", features = ["logs"] }
-tracing = {version = "0.1.37", default-features = false, features = ["std"]}
+tracing = {version = "0.1", default-features = false, features = ["std"]}
 tracing-core = "0.1.31"
 tracing-subscriber = { version = "0.3.0", default-features = false, features = ["registry", "std"] }
 once_cell = "1.13.0"

opentelemetry-appender-tracing/examples/basic.rs

@@ -23,6 +23,6 @@ fn main() {
  let layer = layer::OpenTelemetryTracingBridge::new(&provider);
  tracing_subscriber::registry().with(layer).init();
 
- error!(target: "my-system", event_id = 20, event_name = "my-event_name", user_name = "otel", user_email = "otel@opentelemetry.io");
+ error!(name: "my-event-name", target: "my-system", event_id = 20, user_name = "otel", user_email = "otel@opentelemetry.io");
  drop(provider);
 }

opentelemetry-appender-tracing/src/layer.rs

@@ -103,6 +103,11 @@
  log_record.severity_number = Some(map_severity_to_otel_severity(meta.level().as_str()));
  log_record.severity_text = Some(meta.level().to_string().into());
 
+ // add the `name` metadata to attributes
+ // TBD - Propose this to be part of log_record metadata.
+ let vec = vec![("name", meta.name())];
+ log_record.attributes = Some(vec.into_iter().map(|(k, v)| (k.into(), v.into())).collect());
+
  // Not populating ObservedTimestamp, instead relying on OpenTelemetry
  // API to populate it with current time.
 

opentelemetry-user-events-logs/Cargo.toml

@@ -21,7 +21,7 @@ chrono = { version="0.4", default-features = false, features=["std"] }
 
 [dev-dependencies]
 opentelemetry-appender-tracing = { path = "../opentelemetry-appender-tracing" }
-tracing = { version = "0.1.37", default-features = false, features = ["std"] }
+tracing = { version = "0.1", default-features = false, features = ["std"] }
 tracing-core = "0.1.31"
 tracing-subscriber = { version = "0.3.0", default-features = false, features = ["registry", "std"] }
 microbench = "0.5"

opentelemetry-user-events-logs/examples/basic.rs

@@ -30,7 +30,7 @@ fn main() {
  // event_id is also passed as an attribute now, there is nothing in metadata where a
  // numeric id can be stored.
  error!(
- event_name = "my-event-name",
+ name: "my-event-name",
  event_id = 20,
  user_name = "otel user",
  user_email = "otel@opentelemetry.io"

opentelemetry-user-events-logs/src/logs/exporter.rs

@@ -52,7 +52,8 @@
 }
 
 const EVENT_ID: &str = "event_id";
-const EVENT_NAME: &str = "event_name";
+const EVENT_NAME_PRIMARY: &str = "event_name";
+const EVENT_NAME_SECONDARY: &str = "name";
 
 //TBD - How to configure provider name and provider group
 impl UserEventsExporter {
@@ -222,11 +223,17 @@
  event_id = *value;
  continue;
  }
- (EVENT_NAME, AnyValue::String(value)) => {
+ (EVENT_NAME_PRIMARY, AnyValue::String(value)) => {
  is_event_name = true;
  event_name = value.as_str();
  continue;
  }
+ (EVENT_NAME_SECONDARY, AnyValue::String(value)) => {
+ if !is_event_name {
+ event_name = value.as_str();
+ }
+ continue;
+ }
  _ => {
  if !is_part_c_present {
  eb.add_struct_with_bookmark("PartC", 1, 0, &mut cs_c_bookmark);
@@ -237,6 +244,7 @@
  }
  }
  }
+
  if is_part_c_present {
  eb.set_struct_field_count(cs_c_bookmark, cs_c_count);
  }
@@ -245,7 +253,7 @@
  let mut cs_b_bookmark: usize = 0;
  let mut cs_b_count = 0;
  eb.add_struct_with_bookmark("PartB", 1, 0, &mut cs_b_bookmark);
- eb.add_str("_typename", "Logs", FieldFormat::Default, 0);
+ eb.add_str("_typeName", "Logs", FieldFormat::Default, 0);
  cs_b_count += 1;
 
  if log_data.record.body.is_some() {
@@ -282,7 +290,7 @@
  eb.add_value("eventId", event_id, FieldFormat::SignedInt, 0);
  cs_b_count += 1;
  }
- if is_event_name {
+ if !event_name.is_empty() {
  eb.add_str("name", event_name, FieldFormat::Default, 0);
  cs_b_count += 1;
  }