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

Eks resource detector bugfix #575

Merged
merged 19 commits into from
Feb 16, 2021
Merged

Conversation

bhautikpip
Copy link
Contributor

  1. Fixes EKS resource detector initialization bug. with this fix users should be able to follow below initialization.
eksResourceDetector := eks.NewResourceDetector()

@bhautikpip bhautikpip changed the title Eks bugfix Eks resource detector bugfix Feb 10, 2021
@codecov
Copy link

codecov bot commented Feb 10, 2021

Codecov Report

Merging #575 (ccd4616) into main (e532370) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #575     +/-   ##
=======================================
- Coverage   77.9%   77.9%   -0.1%     
=======================================
  Files         55      55             
  Lines       2594    2595      +1     
=======================================
  Hits        2022    2022             
- Misses       442     443      +1     
  Partials     130     130             
Impacted Files Coverage Δ
detectors/aws/eks/detector.go 29.1% <0.0%> (-0.4%) ⬇️

detectors/aws/eks/detector.go Outdated Show resolved Hide resolved
alolita pushed a commit to aws-otel/aws-otel.github.io that referenced this pull request Feb 11, 2021
* Update go-sdk.mdx

1. Updated creation of OTEL exporter as per upstream `0.16.0`
2. Updated ECS and EKS resource detector initialization due to recent bug fix - open-telemetry/opentelemetry-go-contrib#569
open-telemetry/opentelemetry-go-contrib#575

However, we should wait for upstream to release `0.17.0` which will have these changes released.

* Added `otelgrpc` dependency
@bhautikpip
Copy link
Contributor Author

bhautikpip commented Feb 12, 2021

Is this good to merge? @MrAlias @XSAM

@MrAlias MrAlias merged commit bd90f3e into open-telemetry:main Feb 16, 2021
@MrAlias MrAlias mentioned this pull request Feb 17, 2021
plantfansam referenced this pull request in plantfansam/opentelemetry-go-contrib Mar 18, 2022
* Remove GetDescriptor

* Add Must var hotfix
@pellared pellared added this to the untracked milestone Nov 8, 2024
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.

5 participants