Skip to content

[Doc] remove modules#16796

Closed
kaisecheng wants to merge 2 commits intoelastic:mainfrom
kaisecheng:doc_remove_modules
Closed

[Doc] remove modules#16796
kaisecheng wants to merge 2 commits intoelastic:mainfrom
kaisecheng:doc_remove_modules

Conversation

@kaisecheng
Copy link
Copy Markdown
Contributor

@kaisecheng kaisecheng commented Dec 13, 2024 •
edited
Loading

Remove all module related doc

@kaisecheng kaisecheng changed the title [Docremove modules doc [Doc] remove modules Dec 13, 2024
@kaisecheng kaisecheng linked an issue Dec 13, 2024 that may be closed by this pull request
Remove the Arcsight module from x-pack #16740
Closed
kaisecheng
kaisecheng commented Dec 13, 2024
View reviewed changes
docs/static/deploying.asciidoc
* SNMP trap - Logstash has a native <<plugins-inputs-snmptrap,SNMP trap input>>.
* CEF - Logstash accepts and parses CEF data from systems like Arcsight
SmartConnectors with the <<plugins-codecs-cef,CEF codec>>. See this
https://www.elastic.co/blog/integrating-elastic-stack-with-arcsight-siem-part-1[blog series]
Copy link
Copy Markdown
Contributor Author

@kaisecheng kaisecheng Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the link to module blog post

@kaisecheng kaisecheng mentioned this pull request Dec 13, 2024
Merged
10 tasks
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 13, 2024

📃 DOCS PREVIEW ✨ https://logstash_bk_16796.docs-preview.app.elstc.co/diff

@robbavey
Copy link
Copy Markdown
Member

robbavey commented Dec 16, 2024

Let's work on this one with @karenzone before committing this removal - there are likely places where we will want to add placeholders to help users coming from previous documentation, and pointers to other elastic solutions to provide equivalent functionality

@karenzone
Copy link
Copy Markdown
Contributor

karenzone commented Dec 16, 2024 •
edited
Loading

Let's work on this one with @karenzone before committing this removal - there are likely places where we will want to add placeholders to help users coming from previous documentation, and pointers to other elastic solutions to provide equivalent functionality

Yes, please. This is not how we handle doc removals. I have two related issues to handle:

karenzone
karenzone suggested changes Dec 16, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs require a different approach. Please don't merge.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 17, 2024

📃 DOCS PREVIEW ✨ https://logstash_bk_16796.docs-preview.app.elstc.co/diff

@kaisecheng kaisecheng closed this Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@karenzone karenzone karenzone requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove the Arcsight module from x-pack

3 participants

@kaisecheng @robbavey @karenzone