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

Remote write compliance updates #9196

Merged
merged 3 commits into from Apr 27, 2021
Merged

Remote write compliance updates #9196

merged 3 commits into from Apr 27, 2021

Conversation

helenosheaa
Copy link
Member

relates to #9166

this change will mean we pass the prometheus remote write repeated labels compliance test which we are currently failing

  • added a check to the prometheus serializer to see if the tags value is "", if it is we drop the tag

this change will mean we mass the prometheus remote write empty labels compliance test, added a test for the above

@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Apr 26, 2021
@helenosheaa helenosheaa merged commit fe352db into master Apr 27, 2021
@helenosheaa helenosheaa deleted the remote-write-compliance branch April 27, 2021 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants