-
Notifications
You must be signed in to change notification settings - Fork 25k
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 Monitoring ingest pipelines #77459
Remove Monitoring ingest pipelines #77459
Conversation
Pinging @elastic/es-data-management (Team:Data Management) |
@elasticmachine update branch |
@elasticmachine run elasticsearch-ci/part-2 |
Digging into these test failures. From the looks of things, the order of the pipelines in the expected error message is different now. I do not understand what might have caused that yet. |
Added @lcawl as reviewer since you were suggested for the deprecation PR, and because this one also has docs changes |
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
@elasticmachine update branch |
Changing to the deprecation label since we have re-added the removed settings in 8.0 to allow for a longer deprecation cycle. The settings will not break 8.0 nodes on start up, but it will not have any functionality tied to them. |
Monitoring installs a number of ingest pipelines which have been historically used to upgrade documents when mappings and document structures change between versions. Since there aren't any changes to the document format, nor will there be by the time the format is completely retired, we can comfortably remove these pipelines.
Relates #50770