feat: add missing component tag to aiohttp connection patch (#5195)

Adds missing component tag that should be present for aiohttp.

## Checklist

- [ ] Change(s) are motivated and described in the PR description.
- [ ] Testing strategy is described if automated tests are not included
in the PR.
- [ ] Risk is outlined (performance impact, potential for breakage,
maintainability, etc).
- [ ] Change is maintainable (easy to change, telemetry, documentation).
- [ ] [Library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/contributing.html#Release-Note-Guidelines)
are followed.
- [ ] Documentation is included (in-code, generated user docs, [public
corp docs](https://github.com/DataDog/documentation/)).
- [ ] Author is aware of the performance implications of this PR as
reported in the benchmarks PR comment.

## Reviewer Checklist

- [x] Title is accurate.
- [x] No unnecessary changes are introduced.
- [x] Description motivates each change.
- [x] Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes unless absolutely necessary.
- [x] Testing strategy adequately addresses listed risk(s).
- [x] Change is maintainable (easy to change, telemetry, documentation).
- [x] Release note makes sense to a user of the library.
- [x] Reviewer is aware of, and discussed the performance implications
of this PR as reported in the benchmarks PR comment.

---------

Co-authored-by: Munir Abdinur <[email protected]>

Author: wconti27

ddtrace/contrib/aiohttp/patch.py

@@ -48,13 +48,17 @@ def __init__(self, obj, pin):
 
     async def connect(self, req, *args, **kwargs):
         pin = Pin.get_from(self)
-        with pin.tracer.trace("%s.connect" % self.__class__.__name__):
+        with pin.tracer.trace("%s.connect" % self.__class__.__name__) as span:
+            # set component tag equal to name of integration
+            span.set_tag(COMPONENT, config.aiohttp.integration_name)
             result = await self.__wrapped__.connect(req, *args, **kwargs)
             return result
 
     async def _create_connection(self, req, *args, **kwargs):
         pin = Pin.get_from(self)
-        with pin.tracer.trace("%s._create_connection" % self.__class__.__name__):
+        with pin.tracer.trace("%s._create_connection" % self.__class__.__name__) as span:
+            # set component tag equal to name of integration
+            span.set_tag(COMPONENT, config.aiohttp.integration_name)
             result = await self.__wrapped__._create_connection(req, *args, **kwargs)
             return result
 

releasenotes/notes/add-missing-component-tag-on-aiohttp-tcp-spans-8c9fe55d4e8f9085.yaml

@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    aiohttp: Add missing component meta tag to aiohttp server spans.

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_200_request.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 7026000,
        "start": 1646414759575827000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_200_request_post.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 2740000,
        "start": 1646794689782860000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_500_request.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 2061000,
        "start": 1646414759680240000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_auth_200_request.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 1426000,
        "start": 1673646330399204000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_configure_global_service_name_env.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 4452000,
        "start": 1646721315459826000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_configure_service_name_pin.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 18051000,
        "start": 1646721987369091000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_trace_multiple.json

@@ -38,6 +38,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 26278000,
        "start": 1646794505598411000
      }],
@@ -81,6 +84,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 70000,
        "start": 1646794505627940000
      }],
@@ -124,6 +130,9 @@
        "parent_id": 1,
        "type": "",
        "error": 0,
+       "meta": {
+         "component": "aiohttp"
+       },
        "duration": 136000,
        "start": 1646794505631851000
      }]]

tests/snapshots/tests.contrib.aiohttp.test_aiohttp_client.test_trace_parenting.json

@@ -52,6 +52,9 @@
           "parent_id": 2,
           "type": "",
           "error": 0,
+          "meta": {
+            "component": "aiohttp"
+          },
           "duration": 2464000,
           "start": 1646794413226798000
         }]]