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

Filter out unneeded labels #606

Merged
merged 2 commits into from
Mar 29, 2022
Merged

Conversation

ekarlso
Copy link
Contributor

@ekarlso ekarlso commented Dec 5, 2021

Fixes #605

@ekarlso ekarlso requested review from a team and bogdandrutu December 5, 2021 13:02
@ekarlso ekarlso force-pushed the issue-605 branch 3 times, most recently from 4f0a177 to c9023e1 Compare December 5, 2021 13:18
@jpkrohling jpkrohling changed the title Fixes #605 - Filter out unneeded labels Filter out unneeded labels Dec 6, 2021
@jpkrohling jpkrohling requested review from jpkrohling and removed request for bogdandrutu December 6, 2021 10:09
pkg/collector/daemonset_test.go Outdated Show resolved Hide resolved
internal/config/main.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
pkg/collector/labels.go Show resolved Hide resolved
@ekarlso ekarlso force-pushed the issue-605 branch 3 times, most recently from 36b5dd9 to 61faf60 Compare December 6, 2021 21:31
main.go Outdated Show resolved Hide resolved
@ekarlso ekarlso requested a review from jpkrohling March 12, 2022 10:05
Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but shouldn't this be documented somewhere?

@ekarlso
Copy link
Contributor Author

ekarlso commented Mar 28, 2022

LGTM, but shouldn't this be documented somewhere?

I think it will totally depend on you setup but yeah if you want me to add docs please say where :)

@jpkrohling jpkrohling merged commit e3b764f into open-telemetry:main Mar 29, 2022
@jpkrohling
Copy link
Member

I'm merging so that this gets into the next release, even if the documentation is missing. Please, document it either in the readme, or with an e2e test, showing how to use this feature. If it's not documented, it doesn't exist.

ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* Fixes open-telemetry#605 - Filter out unneeded labels
      - 10.98.72.0/21

* Remove second parse
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.

Propagated labels causes issues for Deployment, StatefulSet and DaemonSet
2 participants