-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Rename google-pubsub Filebeat input to gcp-pubsub #22213
Conversation
Pinging @elastic/integrations-platforms (Team:Platforms) |
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.
When will the old input name be removed? If 8.0 then after this is backported into 7.x we could remove the old name from master.
In the release note we could say something like "the previous name will continue working until the next major release. Please update your configs".
Pinging @elastic/ingest-management (Team:Ingest Management) |
💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
@kaiyan-sheng Is there any integration using this input? if so they would need to be changed as well. |
@ph Thanks for reminding me. I don't think so. We only migrated AWS package with s3 input, not GCP. |
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.
LGTM
What does this PR do?
This PR is to rename
google-pubsub
input togcp-pubsub
input. This is a part of the renaming process to changegooglecloud
togcp
in both Metricbeat and Filebeat.Warning log message is also added when user is using the old input name:
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.How to test this PR locally
This should not be a breaking change so both
google-pubsub
andgcp-pubsub
input names should work.google-pubsub
name:gcp-pubsub
name:Related issues