-
Couldn't load subscription status.
- Fork 5k
Change filebeat azure eventhub input to use processor v2 as default #47292
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
base: main
Are you sure you want to change the base?
Conversation
|
This pull request doesn't have a |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
|
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
| # this field accommodate a description without length limits. | ||
| description: This PR is to change Filebeat azure eventhub input to use processor V2 as default instead of V1. Since we've decided to retire processor v1, we plan to set processor v2 as the default option in the next release, and completely remove v1 in the following release. | ||
|
|
||
| # REQUIRED for breaking-change, deprecation, known-issue |
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.
@zmoog What would be the impact and action here for this breaking change?
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.
Is this a breaking change? If we can continue to support the connection string with the same setting this should not be.
Are we forced to change the config format, making it incompatible with the existing configs?
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.
Not a breaking change! I think we should use deprecation instead. @zmoog wdyt?
Proposed commit message
This PR is to change Filebeat azure eventhub input and related modules to use processor v2 as the default.
Checklist
./changelog/fragmentsusing the changelog tool.Disruptive User Impact
For users that are currently still using
v1processor version, they need to add this into their config manually for the new version to work forv1.How to test this PR locally
Test cases I would like to cover for this change are:
v1for backwards compatibilityv2Do we need to test the migration path on Beats?
Related issues
This PR takes care of the Beats side of elastic/integrations#15703. We will need to make the same change in the Integrations.