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

[aws detector] Additional Attributes #410

Conversation

dackroyd
Copy link
Contributor

Adding extra host/cloud attributes from the semantic conventions that can be provided by AWS instance metadata:

  • host.image.id
  • host.name
  • host.type
  • cloud.zone

Metadata (currently only hostname) failures will still allow the resource to be returned, along with a wrapped ErrPartialResource (per the Detector interface design)

Fixes #403

detectors/aws/aws_test.go Outdated Show resolved Hide resolved
@MrAlias MrAlias added area: detector Related to a detector package priority:p3 labels Oct 27, 2020
@MrAlias MrAlias added this to the RC1 milestone Oct 27, 2020
Adding extra host/cloud attributes from the semantic conventions that can
 be provided by AWS instance metadata:

 * `host.image.id`
 * `host.name`
 * `host.type`
 * `cloud.zone`

 Metadata (currently only `hostname`) failures will still allow the
  resource to be returned, along with a wrapped ErrPartialResource
  (per the Detector interface design)
@dackroyd dackroyd force-pushed the add-missing-resource-attributes-to-aws-detection branch from 776adb4 to d524be6 Compare October 28, 2020 02:08
@MrAlias MrAlias merged commit 13ae395 into open-telemetry:master Oct 30, 2020
vchitai pushed a commit to vchitai/opentelemetry-go-contrib that referenced this pull request Nov 13, 2020
Adding extra host/cloud attributes from the semantic conventions that can
 be provided by AWS instance metadata:

 * `host.image.id`
 * `host.name`
 * `host.type`
 * `cloud.zone`

 Metadata (currently only `hostname`) failures will still allow the
  resource to be returned, along with a wrapped ErrPartialResource
  (per the Detector interface design)

Co-authored-by: Tyler Yahn <[email protected]>
@MrAlias MrAlias mentioned this pull request Nov 20, 2020
plantfansam referenced this pull request in plantfansam/opentelemetry-go-contrib Mar 18, 2022
Co-authored-by: Joshua MacDonald <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: detector Related to a detector package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing resource attributes from AWS detector
3 participants