-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Graphite tags parser #8564
Graphite tags parser #8564
Conversation
if len(tagValue) != 2 || len(tagValue[0]) == 0 || len(tagValue[1]) == 0 { | ||
continue | ||
} | ||
if strings.IndexAny(tagValue[0], "!^") != -1 { |
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.
fyi strings.ContainsAny
(cherry picked from commit f6e2d45)
@ssoroka hi, how to know in which version this will be released ? |
https://github.com/influxdata/telegraf/releases/tag/v1.17.0 |
we'll get it into 1.17.1 for you. Should be in a couple weeks. You can use the nightly build to get it until then |
actually it's already in right ? |
I just tested the |
Fixing #4372
Required for all PRs: