Use ResourceAttributes.CLOUD_ACCOUNT_ID

tammy-baylis-swi · Mar 21, 2024 · 7f8194a · 7f8194a
1 parent f3421b6
commit 7f8194a
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/...metry-instrumentation-aws-lambda/src/opentelemetry/instrumentation/aws_lambda/__init__.py b/...metry-instrumentation-aws-lambda/src/opentelemetry/instrumentation/aws_lambda/__init__.py
@@ -87,6 +87,7 @@ def custom_event_context_extractor(lambda_event):
     TRACE_HEADER_KEY,
     AwsXRayPropagator,
 )
+from opentelemetry.semconv.resource import ResourceAttributes
 from opentelemetry.semconv.trace import SpanAttributes
 from opentelemetry.trace import (
     Span,
@@ -358,11 +359,8 @@ def _instrumented_lambda_handler_call(  # noqa pylint: disable=too-many-branches
                 # See more:
                 # https://github.com/open-telemetry/semantic-conventions/blob/main/docs/faas/aws-lambda.md#all-triggers
                 account_id = lambda_context.invoked_function_arn.split(":")[4]
-
-                # TODO: Update key with semconvgen 1.23.0
-                # https://github.com/open-telemetry/semantic-conventions/issues/551
                 span.set_attribute(
-                    "cloud.account.id",
+                    ResourceAttributes.CLOUD_ACCOUNT_ID,
                     account_id,
                 )