logstash icon indicating copy to clipboard operation
logstash copied to clipboard

Update plugins-metadata.json with integration-snmp: set skiplist to false

Open karenzone opened this issue 9 months ago • 3 comments

Adding integration-snmp to plugins-metadata.json so that it will be picked up for the Logstash Reference plugin docgen

Why?

After the VPR job ran and picked up integration-snmp, I kicked off a docgen for Logstash Reference on main. 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 and input-snmp-trap settings in https://github.com/elastic/logstash/blame/main/rakelib/plugins-metadata.json? Screen Shot 2024-05-03 at 11 12 35 AM

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?

karenzone avatar May 03 '24 15:05 karenzone

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

elastic-sonarqube[bot] avatar May 03 '24 15:05 elastic-sonarqube[bot]

:green_heart: Build Succeeded

cc @karenzone

elasticmachine avatar May 03 '24 15:05 elasticmachine

@jsvd, what changes should I make to input-snmp and input-snmp-trap settings in https://github.com/elastic/logstash/blame/main/rakelib/plugins-metadata.json?

Once we flip the integration to become the default, we should make the individual plugins non default and add them to the skip list

jsvd avatar May 10 '24 13:05 jsvd