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 /pods endpoint support in kubeletstats receiver to add extra labels #569
Add /pods endpoint support in kubeletstats receiver to add extra labels #569
Conversation
7bd18b0
to
63c82b6
Compare
Codecov Report
@@ Coverage Diff @@
## master #569 +/- ##
===========================================
+ Coverage 71.09% 86.13% +15.03%
===========================================
Files 14 189 +175
Lines 602 10213 +9611
===========================================
+ Hits 428 8797 +8369
- Misses 150 1093 +943
- Partials 24 323 +299
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
d768350
to
42f934c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM Dmitrii, thanks for taking care of this.
42f934c
to
9239d77
Compare
This commits adds ability to optionally fetch extra metadata from /pods endpoint in kubeletstats receiver and use that data to set additional labels on metric resource. For now only container.id label is supported
9239d77
to
0e49957
Compare
Fix build links
This PR changes the circleci build config to build the project in both go-1.13 and go-1.14 versions. Co-authored-by: Joshua MacDonald <jmacd@users.noreply.github.com> Co-authored-by: Liz Fong-Jones <lizf@honeycomb.io>
Description:
This commits adds ability to optionally fetch extra metadata from /pods endpoint in kubeletstats receiver and use that data to set additional labels on metric resource. For now only container.id label is supported
Testing: Unit tests and manual testing
Documentation: Additional section in README.md