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

[receiver/kubeletstats] collect network metrics for all interfaces #30626

Closed
wants to merge 4 commits into from

Conversation

jinja2
Copy link
Contributor

@jinja2 jinja2 commented Jan 16, 2024

Description:

k8s.pod.network.[io|errors] and k8s.node.network.[io|errors] are currently only collected for the default network interface. This PR updates the kubeletstats receiver to collect metrics for all interfaces.

Link to tracking Issue: 30196

Testing: Updated unit tests for the scraper

Documentation:

crobert-1 and others added 2 commits January 16, 2024 13:00
…telemetry#28651)

**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
This fixes inconsistency introduced with the creation of this package.
In open-telemetry#25096 @cparkins was added as a code owner in the
[metadata.yaml](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/pkg/translator/azure/metadata.yaml)
but not the top level `CODEOWNERS` file.

Co-authored-by: Alex Boten <aboten@lightstep.com>
…telemetry#28651)

**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
This fixes inconsistency introduced with the creation of this package.
In open-telemetry#25096 @cparkins was added as a code owner in the
[metadata.yaml](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/pkg/translator/azure/metadata.yaml)
but not the top level `CODEOWNERS` file.

Co-authored-by: Alex Boten <aboten@lightstep.com>
@jinja2 jinja2 requested a review from a team January 16, 2024 21:18
@jinja2 jinja2 marked this pull request as draft January 16, 2024 21:19
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Jan 31, 2024
Copy link
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@ChrsMark
Copy link
Member

ChrsMark commented Jul 15, 2024

Hey @jinja2! Any chance we can revive this one? We have an extra issue reported at #33993.

Edit: Happy to pick your commit, do any necessary updates and file a PR adding you as a co-author.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants