Please pylint

Author: xrmx

instrumentation/opentelemetry-instrumentation-botocore/src/opentelemetry/instrumentation/botocore/extensions/bedrock.py

@@ -149,7 +149,7 @@ def _extract_metrics_attributes(self) -> _AttributeMapT:
 
         model_id = self._call_context.params.get(_MODEL_ID_KEY)
         if not model_id:
-            return
+            return attributes
 
         attributes[GEN_AI_REQUEST_MODEL] = model_id
 
@@ -335,9 +335,10 @@ def before_service_call(
                 span.update_name(f"{operation_name} {request_model}")
 
         # this is used to calculate the operation duration metric, duration may be skewed by request_hook
+        # pylint: disable=attribute-defined-outside-init
         self._operation_start = default_timer()
 
-    # pylint: disable=no-self-use
+    # pylint: disable=no-self-use,too-many-locals
     def _converse_on_success(
         self,
         span: Span,
@@ -546,7 +547,7 @@ def invoke_model_stream_error_callback(exception):
                 )
                 return
 
-    # pylint: disable=no-self-use
+    # pylint: disable=no-self-use,too-many-locals
     def _handle_amazon_titan_response(
         self,
         span: Span,
@@ -607,7 +608,7 @@ def _handle_amazon_titan_response(
                             output_tokens, output_attributes
                         )
 
-    # pylint: disable=no-self-use
+    # pylint: disable=no-self-use,too-many-locals
     def _handle_amazon_nova_response(
         self,
         span: Span,

instrumentation/opentelemetry-instrumentation-botocore/tests/test_botocore_bedrock.py

@@ -56,6 +56,7 @@ def test_converse_with_content(
     bedrock_runtime_client,
     instrument_with_content,
 ):
+    # pylint:disable=too-many-locals
     messages = [{"role": "user", "content": [{"text": "Say this is a test"}]}]
 
     llm_model_value = "amazon.titan-text-lite-v1"