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

OTel Resource Detector for AWS #426

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

danschultzer
Copy link

@danschultzer danschultzer commented Dec 10, 2024

Adds OTel Resource Detector for AWS, with support for ECS.

It's essentially a mirror of the JS version.

Resolves #101

Copy link

linux-foundation-easycla bot commented Dec 10, 2024

CLA Signed


The committers listed above are authorized under a signed CLA.

@danschultzer danschultzer force-pushed the otel-resource-detector-aws branch 2 times, most recently from 5f30af1 to 83e0e34 Compare December 10, 2024 06:36
@@ -0,0 +1,139 @@
-module(otel_resource_aws_ecs).
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The naming convention here follows the naming convention in opentelemetry with otel_resource_app_env and otel_resource_env_var

@danschultzer danschultzer force-pushed the otel-resource-detector-aws branch from 83e0e34 to 96bac1d Compare December 10, 2024 07:44
"tagPrefix": "opentelemetry-resource-detector-aws-v",
"buildTool": "rebar3",
"language": "erlang",
"authorizedUsers": ["danschultzer"]
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume I have to manage the releases for this, but lmk if either of you @bryannaegele @tsloughter should be added here.

@danschultzer
Copy link
Author

@bryannaegele @tsloughter can I get a review of this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resource Detector: AWS
1 participant