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

Remove event_api.tags.illegal option #16460

Closed

Conversation

kaisecheng
Copy link
Contributor

@kaisecheng kaisecheng commented Sep 17, 2024

_Do Not Merge until we got approval from breaking changes committee _

Release notes

Removed the event_api.tags.illegal option, which allowed setting a map value to the tags field. Since v9, such assignments will fail, resulting in a _tagsparsefailure in the tags field, and the illegal value will be written to the _tags field.

What does this PR do?

The commit removes the event_api.tags.illegal option which is added in #14822

Why is it important/What is the impact to the user?

This is a breaking change. User no longer be able to set event_api.tags.illegal to warn as a fallback to accept the illegal assignment of tags field.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files (and/or docker env variables)
  • I have added tests that prove my fix is effective or that my feature works

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

Copy link

@kaisecheng kaisecheng closed this Sep 17, 2024
@kaisecheng kaisecheng deleted the remove_tags_illegal_option branch September 17, 2024 20:28
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.

Remove event_api.tags.illegal (default becomes locked to 'rename')
3 participants