fix: memory limit should be defined in Bytes, not MB

open-telemetry · Dec 9, 2024 · b654a9d · b654a9d
1 parent 0c3771a
commit b654a9d
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/opentelemetry-aws/src/detector/lambda.rs b/opentelemetry-aws/src/detector/lambda.rs
@@ -27,7 +27,11 @@ impl ResourceDetector for LambdaResourceDetector {
 
         let aws_region = env::var(AWS_REGION_ENV_VAR).unwrap_or_default();
         let function_version = env::var(AWS_LAMBDA_FUNCTION_VERSION_ENV_VAR).unwrap_or_default();
-        let function_memory_limit = env::var(AWS_LAMBDA_MEMORY_LIMIT_ENV_VAR).unwrap_or_default();
+        // Convert memory limit from MB to Bytes as required by semantic conventions.
+        let function_memory_limit = env::var(AWS_LAMBDA_MEMORY_LIMIT_ENV_VAR)
+            .map(|s| s.parse::<i64>().unwrap_or_default() * 1024 * 1024)
+            .unwrap_or_default()
+            .to_string();
         // Instance attributes corresponds to the log stream name for AWS Lambda;
         // See the FaaS resource specification for more details.
         let instance = env::var(AWS_LAMBDA_LOG_STREAM_NAME_ENV_VAR).unwrap_or_default();