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] Update scope name to otelcol/kubeletstatsreceiver #9400

Merged
merged 5 commits into from
Apr 22, 2022

Conversation

TylerHelmuth
Copy link
Member

Description:
Update Scope Name to otelcol/kubeletstatsreceiver in all scenarios. Update to v2 metrics builder will happen in a different PR.

Link to tracking Issue:
Fixes #9399

Testing:
Updated unit tests to check for scope name.

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
@dmitryax
Copy link
Member

Looks like a related test is failing

@dmitryax dmitryax self-requested a review April 21, 2022 22:04
@TylerHelmuth
Copy link
Member Author

Looks like a related test is failing

Ya that k8sclient test is flaky. Being tracked in #9091.

I think the check-collector-module-version test is failing on every branch right now.

@codeboten
Copy link
Contributor

this should fix the check-collector-module-version: https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/9402/files

@codeboten codeboten merged commit 19ba19f into open-telemetry:main Apr 22, 2022
@TylerHelmuth TylerHelmuth deleted the issue-9399 branch April 22, 2022 17:33
djaglowski pushed a commit to djaglowski/opentelemetry-collector-contrib that referenced this pull request May 10, 2022
…iver (open-telemetry#9400)

* Update scope name to otelcol/kubeletstatsreceiver

* remove unused type parameter

* Update changelog

Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
Co-authored-by: Alex Boten <aboten@lightstep.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.

update receiver/kubeletstats to set consistent scope name
4 participants