-
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
remove deprecated modules netflow, fb_apache and azure #16514
Conversation
doc has instructions on using modules with netflow as example. The removal of doc will be addressed in another pr. |
TODO: companion PR for Looking at the implementation, the best hook for this is likely Or possibly by hard-coding a list of them in |
x-pack/modules/README.md
Outdated
@@ -1,99 +0,0 @@ | |||
# Module settings and structure |
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.
given we're not yet removing all modules, maybe we can lead the READMEs intact for now
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.
Other than maybe restoring the modules/README.md and x-pack/modules/README.md, LGTM
Follow up PRs:
8.x
deprecation entry when these modules are usedmain
cleanup docs related to netflow, fb_apache and azure modules
Quality Gate passedIssues Measures |
💛 Build succeeded, but was flaky
Failed CI StepsHistory
cc @jsvd |
Release notes
[rn:skip]
What does this PR do?
removed deprecated modules netflow, fb_apache and azure
Why is it important/What is the impact to the user?
Checklist
Author's Checklist
How to test this PR locally
Related issues
netflow
,fb_apache
,azure
modules #16357Use cases
Screenshots
Logs