-
Notifications
You must be signed in to change notification settings - Fork 440
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
Conversation
4f0a177
to
c9023e1
Compare
36b5dd9
to
61faf60
Compare
- 10.98.72.0/21
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, 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 :) |
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. |
* Fixes open-telemetry#605 - Filter out unneeded labels - 10.98.72.0/21 * Remove second parse
Fixes #605