feat(instrumentation-aiohttp-client): Add support for HTTP metrics

[hectorhdzg]

Description

Added support for metrics in aiohttp-client instrumentation

Fixes #3516

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Unit tests

Does This PR Require a Core Repo Change?

• No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

• Followed the style guidelines of this project
• Changelogs have been updated
• Unit tests have been added
• Documentation has been updated

[tammy-baylis-swi]

This lgtm; thanks for tests, docstring updates, and consistency with how other instrumentors handle the semconv opt-in.

Would be good to also get other Approver and Maintainer eyes on this.

[emdneto]

Thanks! I left some changes that I think we should do to not back into this later.
See #3220 for more context

[emdneto]

I'm missing the SERVER_ADDRESS and SERVER_PORT in labels for the duration_histogram_new. They are required. can we set them?

[hectorhdzg]

I'm missing the SERVER_ADDRESS and SERVER_PORT in labels for the duration_histogram_new. They are required. can we set them?

@emdneto the attributes were missing in Spans as well, added both

[emdneto]

I'm missing the SERVER_ADDRESS and SERVER_PORT in labels for the duration_histogram_new. They are required. can we set them?

@emdneto the attributes were missing in Spans as well, added both

oh. Awesome! Thank you