[stormshield] Fix pipeline descriptions and correct documentation claims#20030
Open
arenard wants to merge 1 commit into
Open
[stormshield] Fix pipeline descriptions and correct documentation claims#20030arenard wants to merge 1 commit into
arenard wants to merge 1 commit into
Conversation
|
💚 CLA has been signed |
Contributor
✅ Elastic Docs Style Checker (Vale)No issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
9 tasks
Contributor
🚀 Benchmarks reportTo see the full report comment with |
a7be5bf to
296568d
Compare
e6370d1 to
64ce75f
Compare
64ce75f to
130df13
Compare
Fix the copy-pasted descriptions of the default and count ingest pipelines, remove field definitions that never appear in events (the pipeline consumes tz into event.timezone, converts duration into event.duration, renames msg to message, and drops origdst/origdstport after converting them to destination fields), and correct the documentation: the agent syslog processor is configured for RFC5424, so the README no longer presents the Legacy format as supported, and the compatibility section now covers SNS 4.x and 5.x, which share the same WELF key-value audit log format. Also document how to collect SNS IPFIX flow records with the NetFlow integration, update vendor links to the SNS v5 documentation, and use arrows for menu paths per the documentation style guide.
130df13 to
18090cd
Compare
Contributor
|
✅ All changelog entries have the correct PR link. |
💚 Build Succeeded
History
|
|
Pinging @elastic/integration-experience (Team:Integration-Experience) |
|
No issues across the latest commits 18090cd.
🤖 AI-Generated Review | Vera Review Bot | 📚 Knowledge base: integration-skills
|
This was referenced Jul 10, 2026
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed commit message
Fix the copy-pasted descriptions of the default and count ingest pipelines, remove field definitions that never appear in events (
stormshield.tz,stormshield.duration,stormshield.msg,stormshield.origdst,stormshield.origdstport— the pipeline consumestzintoevent.timezone, convertsdurationintoevent.duration, renamesmsgtomessage, and dropsorigdst/origdstportafter converting them todestination.*), and correct the documentation:Also documents how to collect SNS IPFIX flow records with the NetFlow integration, updates vendor documentation links to SNS v5, and uses arrows for menu paths per the documentation style guide.
Checklist
changelog.ymlfile.Author's Checklist
sample_event.json(the definitions were dead documentation)descriptionfields were touched in the pipelinesHow to test this PR locally
All pass against stack 9.4.3.
Related issues