chore: PR: OTLP retry PR feedback

Author: scottgerring

opentelemetry-otlp/CHANGELOG.md

@@ -11,7 +11,7 @@ Released 2025-Sep-25
 - Update `opentelemetry-proto` and `opentelemetry-http` dependency version to 0.31.0
 - Add HTTP compression support with `gzip-http` and `zstd-http` feature flags
 - Add retry with exponential backoff and throttling support for HTTP and gRPC exporters
-  This behaviour is opt in via the `experimental-grpc-retry` and `experimental-http-retry flags` on this crate.
+  This behaviour is opt in via the `experimental-grpc-retry` and `experimental-http-retry` flags on this crate. You can customize the retry policy using the `with_retry_policy` on the exporter builders.
 
 ## 0.30.0
 

opentelemetry-otlp/src/retry.rs

@@ -128,15 +128,15 @@ where
 
                 match error_type {
                     RetryErrorType::NonRetryable => {
-                        otel_warn!(name: "OtlpRetryNonRetryable", operation = operation_name, error = format!("{:?}", err));
+                        otel_warn!(name: "Export.Failed.NonRetryable", operation = operation_name, error = format!("{:?}", err));
                         return Err(err);
                     }
                     RetryErrorType::Retryable if attempt < policy.max_retries => {
                         attempt += 1;
                         // Use exponential backoff with jitter
-                        otel_info!(name: "OtlpRetryRetrying", operation = operation_name, error = format!("{:?}", err));
                         let jitter = generate_jitter(policy.jitter_ms);
                         let delay_with_jitter = std::cmp::min(delay + jitter, policy.max_delay_ms);
+                        otel_info!(name: "Export.InProgress.Retrying", operation = operation_name, error = format!("{:?}", err), attempt = attempt, delay_ms = delay_with_jitter, jitter_ms = jitter);
                         runtime
                             .delay(Duration::from_millis(delay_with_jitter))
                             .await;
@@ -145,13 +145,13 @@ where
                     RetryErrorType::Throttled(server_delay) if attempt < policy.max_retries => {
                         attempt += 1;
                         // Use server-specified delay (overrides exponential backoff)
-                        otel_info!(name: "OtlpRetryThrottled", operation = operation_name, error = format!("{:?}", err), delay = format!("{:?}", server_delay));
+                        otel_info!(name: "Export.InProgress.Throttled", operation = operation_name, error = format!("{:?}", err), attempt = attempt, delay_ms = server_delay.as_millis());
                         runtime.delay(server_delay).await;
                         // Don't update exponential backoff delay for next attempt since server provided specific timing
                     }
                     _ => {
                         // Max retries reached
-                        otel_warn!(name: "OtlpRetryExhausted", operation = operation_name, error = format!("{:?}", err), attempts = attempt);
+                        otel_warn!(name: "Export.Failed.Exhausted", operation = operation_name, error = format!("{:?}", err), retries = attempt);
                         return Err(err);
                     }
                 }