-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Update test plugins list for supported plugins test pipeline #17519
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
Update to reflect current list of supported plugins for `9.x`
This pull request does not have a backport label. Could you fix it @robbavey? 🙏
|
@@ -151,6 +151,7 @@ def initialize(name, support, type, organization = "logstash-plugins") | |||
PLUGIN_DEFINITIONS = [ | |||
PluginDefinition.new('logstash-input-azure_event_hubs', :tier1, :input), | |||
PluginDefinition.new('logstash-input-beats', :tier1, :input), | |||
PluginDefinition.new('logstash-input-dead_letter_queue', :tier1, :input), |
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.
good catch!
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.
one question I have asked to clarify, otherwise LGTM!
ci/test_plugins.rb
Outdated
# Removed because of https://github.com/logstash-plugins/logstash-input-couchdb_changes/issues/51 | ||
#PluginDefinition.new('logstash-input-couchdb_changes', :tier2, :input), | ||
PluginDefinition.new('logstash-input-gelf', :tier2, :input), | ||
PluginDefinition.new('logstash-input-couchdb', :tier2, :input), |
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.
logstash-plugins/logstash-input-couchdb_changes#51 seems still active, don't we need to exclude?
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.
Good catch - plus I got the name of the plugin wrong 🤦
9b371cd
to
ce18f93
Compare
|
💚 Build Succeeded
History
|
The list of supported plugins has changed since this was merged - this commit updates the list of plugins to test to reflect the current list of supported plugins for
9.x
.