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

[processors/resourcedetection] Add missing entries for cloud.infrastructure_service #2777

Merged
merged 3 commits into from
Mar 19, 2021

Conversation

jrcamp
Copy link
Contributor

@jrcamp jrcamp commented Mar 19, 2021

This adds missing cloud.infrastructure_service for ec2, gcp, and azure. The
docs were corrected for AWS services to reflect what the code was already
doing.

Detectors for gke and eks will be added in a following PR. Those detectors will
add their own service-specific cloud.infrastructure_service.

…ructure_service`

This adds missing `cloud.infrastructure_service` for ec2, gcp, and azure. The
docs were corrected for AWS services to reflect what the code was already
doing.

Detectors for gke and eks will be added in a following PR. Those detectors will
add their own service-specific `cloud.infrastructure_service`.
@jrcamp jrcamp requested a review from a team as a code owner March 19, 2021 18:43
@codecov
Copy link

codecov bot commented Mar 19, 2021

Codecov Report

Merging #2777 (e6a009c) into main (3bdf190) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2777      +/-   ##
==========================================
- Coverage   91.56%   91.56%   -0.01%     
==========================================
  Files         447      447              
  Lines       22056    22058       +2     
==========================================
+ Hits        20196    20197       +1     
- Misses       1391     1392       +1     
  Partials      469      469              
Flag Coverage Δ
integration 69.13% <ø> (ø)
unit 90.49% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...resourcedetectionprocessor/internal/azure/azure.go 100.00% <100.00%> (ø)
...resourcedetectionprocessor/internal/gcp/gce/gce.go 100.00% <100.00%> (ø)
processor/groupbytraceprocessor/event.go 95.96% <0.00%> (-0.81%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3bdf190...e6a009c. Read the comment docs.

Copy link
Member

@aabmass aabmass left a comment

Choose a reason for hiding this comment

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

GCE change LGTM

@tigrannajaryan tigrannajaryan merged commit 05d0dcb into open-telemetry:main Mar 19, 2021
@jrcamp jrcamp deleted the infra-svc branch March 19, 2021 22:55
pmatyjasek-sumo referenced this pull request in pmatyjasek-sumo/opentelemetry-collector-contrib Apr 28, 2021
…ructure_service` (#2777)

This adds missing `cloud.infrastructure_service` for ec2, gcp, and azure. The
docs were corrected for AWS services to reflect what the code was already
doing.

Detectors for gke and eks will be added in a following PR. Those detectors will
add their own service-specific `cloud.infrastructure_service`.
alexperez52 referenced this pull request in open-o11y/opentelemetry-collector-contrib Aug 18, 2021
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.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

7 participants