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
[cilium] Add support for v1.14 metrics #17447
[cilium] Add support for v1.14 metrics #17447
Conversation
Signed-off-by: Jared Ledvina <jared.ledvina@datadoghq.com>
f100e19
to
a412dc0
Compare
Test Results 8 files 8 suites 15s ⏱️ Results for commit 2c6e044. ♻️ This comment has been updated with latest results. |
Signed-off-by: Jared Ledvina <jared.ledvina@datadoghq.com>
Signed-off-by: Jared Ledvina <jared.ledvina@datadoghq.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more. |
fix some tests
Signed-off-by: Jared Ledvina <jared.ledvina@datadoghq.com>
Co-authored-by: Kyle Neale <kyle.a.neale@gmail.com>
What does this PR do?
This PR updates the
cilium
integration to include support for new and changes metrics in the v1.14 release.Motivation
Natively supporting the v1.14 metrics means folks get all of the metrics out of the box without needed to run the openmetrics integration.
Additional Notes
This PR still needs to have updated tests. It's also unclear how to handle prometheus 'HistogramVec' type metrics within the integration.
Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.backport/<branch-name>
label to the PR and it will automatically open a backport PR once this one is merged