-
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
fix(inputs.gnmi): Add option to explicitly trim field-names #13517
Conversation
@protonmarco please note that you set the new |
Download PR build artifacts for linux_amd64.tar.gz, darwin_amd64.tar.gz, and windows_amd64.zip. 📦 Click here to get additional PR build artifactsArtifact URLs |
@srebhan Thanks for the heads up |
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.
Will let you merge, but wanted to let you know I'm +1 before long weekend
Fix verified on my end; i am reporter of #13512 |
(cherry picked from commit cec60db)
resolves #13512
This PR adds an option to explicitly trim leading dots and slashes from field-names defaulting the new option to false (i.e. keep the leading dots and slashes). This is required to fix a regression where the field-names contain a leading slash for Telegraf versions before v1.27.0 in case a subpath matched the subscription. In this case the field-name still contained a leading slash.