Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Amazon ECS Resource Detector #466

Merged
merged 7 commits into from
Nov 30, 2020

Conversation

KKelvinLo
Copy link
Member

Which problem is this PR solving?

  • This PR adds an Amazon ECS Resource Detector, the specs for the Amazon ECS plugin can be found here.

Key Functionality

  • returns resource object with containerID and containerName if running on Amazon ECS
  • returns empty resource if application is not running on Amazon ECS

Short description of the changes

  • Add functionality for Amazon ECS resource detector
  • Add corresponding unit tests for Amazon ECS resource detector

@Aneurysm9
Copy link
Member

@KKelvinLo can you resolve the conflict and ensure this branch is up-to-date with the trunk so we can get it merged? Thanks!

@KKelvinLo
Copy link
Member Author

@Aneurysm9 merge conflict should be fixed, cheers!

@Aneurysm9 Aneurysm9 merged commit 645d23a into open-telemetry:master Nov 30, 2020
plantfansam referenced this pull request in plantfansam/opentelemetry-go-contrib Mar 18, 2022
* Move dependencies of tools package to a tools directory

* remove TOOLS_MOD_DIR from ALL_GO_MOD_DIRS in a Makefile.
then run 'go mod tidy' on ALL_GO_MOD_DIRS and TOOLS_MOD_DIR.

Co-authored-by: Rahul Patel <rghetia@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants