Do not assume "aws" partition in ecs detector. Fixes #3166

[ahobson]

No description provided.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: ahobson / name: Andrew Hobson (f9af75f)

[ahobson]

I have updated the PR with fixes for the linting problems

[codecov]

Codecov Report

Merging #3167 (64054fc) into main (d296842) will increase coverage by 0.1%.
The diff coverage is 85.7%.

❗ Current head 64054fc differs from pull request most recent head 6f573b3. Consider uploading reports for the commit 6f573b3 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##            main   #3167     +/-   ##
=======================================
+ Coverage   69.6%   69.7%   +0.1%     
=======================================
  Files        147     147             
  Lines       6919    6905     -14     
=======================================
- Hits        4821    4819      -2     
+ Misses      1978    1961     -17     
- Partials     120     125      +5     

Impacted Files	Coverage Δ
detectors/aws/ecs/ecs.go	39.6% <85.7%> (+21.6%)	⬆️
samplers/jaegerremote/sampler_remote.go	87.4% <0.0%> (-2.3%)	⬇️
...ation/github.com/gin-gonic/gin/otelgin/gintrace.go	82.5% <0.0%> (-1.0%)	⬇️
...ntation/github.com/gin-gonic/gin/otelgin/option.go	100.0% <0.0%> (ø)
...om/aws/aws-sdk-go-v2/otelaws/dynamodbattributes.go	100.0% <0.0%> (ø)

[ahobson]

Is there anything else I can do to help get this merged? It passes tests and lints for me locally

[Aneurysm9]

Please add a CHANGELOG entry.

[ahobson]

@Aneurysm9 Thank you for the feedback! I've updated the PR

[ahobson]

😊 I apologize for updating the Makefile, changes reverted.