Copy style of contrib instrumentations

Author: NathanielRN

python/src/otel/otel_sdk/opentelemetry/instrumentation/aws_lambda/__init__.py

@@ -58,6 +58,7 @@ def lambda_handler(event, context):
 from opentelemetry.instrumentation.aws_lambda.version import __version__
 from opentelemetry.instrumentation.instrumentor import BaseInstrumentor
 from opentelemetry.instrumentation.utils import unwrap
+from opentelemetry.semconv.trace import SpanAttributes
 from opentelemetry.trace import SpanKind, get_tracer, get_tracer_provider
 
 logger = logging.getLogger(__name__)
@@ -68,19 +69,25 @@ def instrumentation_dependencies(self) -> Collection[str]:
         return _instruments
 
     def _instrument(self, **kwargs):
-        self._tracer = get_tracer(__name__, __version__, kwargs.get("tracer_provider"))
-
-        self._tracer_provider = get_tracer_provider()
+        """Instruments Lambda Handlers on AWS Lambda
+
+        Args:
+            **kwargs: Optional arguments
+                ``tracer_provider``: a TracerProvider, defaults to global
+        """
+        tracer = get_tracer(
+            __name__, __version__, kwargs.get("tracer_provider")
+        )
 
-        lambda_handler = os.environ.get("ORIG_HANDLER", os.environ.get("_HANDLER"))
+        lambda_handler = os.environ.get(
+            "ORIG_HANDLER", os.environ.get("_HANDLER")
+        )
         wrapped_names = lambda_handler.rsplit(".", 1)
         self._wrapped_module_name = wrapped_names[0]
         self._wrapped_function_name = wrapped_names[1]
 
-        wrap_function_wrapper(
-            self._wrapped_module_name,
-            self._wrapped_function_name,
-            self._functionPatch,
+        _instrument(
+            tracer, self._wrapped_module_name, self._wrapped_function_name
         )
 
     def _uninstrument(self, **kwargs):
@@ -89,35 +96,50 @@ def _uninstrument(self, **kwargs):
             self._wrapped_function_name,
         )
 
-    def _functionPatch(self, original_func, instance, args, kwargs):
-        lambda_context = args[1]
-        ctx_aws_request_id = lambda_context.aws_request_id
-        ctx_invoked_function_arn = lambda_context.invoked_function_arn
-        orig_handler = os.environ.get("ORIG_HANDLER", os.environ.get("_HANDLER"))
+
+def _instrument(tracer, wrapped_module_name, wrapped_function_name):
+    def _instrumented_lambda_call(call_wrapped, instance, args, kwargs):
+        orig_handler_name = ".".join(
+            [wrapped_module_name, wrapped_function_name]
+        )
 
         # TODO: enable propagate from AWS by env variable
         xray_trace_id = os.environ.get("_X_AMZN_TRACE_ID", "")
-
-        lambda_name = os.environ.get("AWS_LAMBDA_FUNCTION_NAME")
-        function_version = os.environ.get("AWS_LAMBDA_FUNCTION_VERSION")
-
         propagator = AwsXRayFormat()
         parent_context = propagator.extract({"X-Amzn-Trace-Id": xray_trace_id})
 
-        with self._tracer.start_as_current_span(
-            name=orig_handler, context=parent_context, kind=SpanKind.SERVER
+        with tracer.start_as_current_span(
+            name=orig_handler_name, context=parent_context, kind=SpanKind.SERVER
         ) as span:
-            # Refer: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/semantic_conventions/faas.md#example
-            span.set_attribute("faas.execution", ctx_aws_request_id)
-            span.set_attribute("faas.id", ctx_invoked_function_arn)
-
-            # TODO: fix in Collector because they belong resource attrubutes
-            span.set_attribute("faas.name", lambda_name)
-            span.set_attribute("faas.version", function_version)
-
-            result = original_func(*args, **kwargs)
+            if span.is_recording():
+                lambda_context = args[1]
+                # Refer: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/semantic_conventions/faas.md#example
+                span.set_attribute(
+                    SpanAttributes.FAAS_EXECUTION, lambda_context.aws_request_id
+                )
+                span.set_attribute(
+                    "faas.id", lambda_context.invoked_function_arn
+                )
+
+                # TODO: fix in Collector because they belong resource attrubutes
+                span.set_attribute(
+                    "faas.name", os.environ.get("AWS_LAMBDA_FUNCTION_NAME")
+                )
+                span.set_attribute(
+                    "faas.version",
+                    os.environ.get("AWS_LAMBDA_FUNCTION_VERSION"),
+                )
+
+            result = call_wrapped(*args, **kwargs)
 
         # force_flush before function quit in case of Lambda freeze.
-        self._tracer_provider.force_flush()
+        tracer_provider = get_tracer_provider()
+        tracer_provider.force_flush()
 
         return result
+
+    wrap_function_wrapper(
+        wrapped_module_name,
+        wrapped_function_name,
+        _instrumented_lambda_call,
+    )