-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Update plugins-metadata.json with integration-snmp: set skiplist to false #16137
Conversation
@@ -433,6 +433,10 @@ | |||
"default-plugins": true, | |||
"skip-list": false | |||
}, | |||
"logstash-integration-snmp": { | |||
"default-plugins": false, |
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.
Note that I'm setting default -> false
for now
Quality Gate passedIssues Measures |
💚 Build Succeeded
cc @karenzone |
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
Once we flip the integration to become the default, we should make the individual plugins non default and add them to the skip list |
Adding
integration-snmp
to plugins-metadata.json so that it will be picked up for the Logstash Reference plugin docgenWhy?
After the VPR job ran and picked up
integration-snmp
, I kicked off a docgen for Logstash Reference onmain
. It didn't pick anything up, and I believe that's because we need this piece added to plugins-metadata.json.Aligning Logstash Reference (LSR) docgen settings
@jsvd, what changes should I make to
input-snmp
andinput-snmp-trap
settings in https://github.com/elastic/logstash/blame/main/rakelib/plugins-metadata.json?At 8.15 I believe that we'll want to set the individual plugins to
"skip-list": true
. Is the current config acceptable in the interim?