[8.x] Remove dependency on infra in monitoring plugin (#203551)#203836
[8.x] Remove dependency on infra in monitoring plugin (#203551)#203836stephmilovic merged 4 commits intoelastic:8.xfrom
Conversation
Blocked by elastic#203492 The monitoring plugin is currently marked as observability plugin because it's relying on the `infra` plugin. However, in practice, no functionality is actually used. This PR removes the dependency -it makes monitoring and logstash a `platform/private` plugin as well (logstash needs to go along with monitoring, but that should be fine because it's only required by monitoring). Some considerations left as comments in the code. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 59d3ac6)
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
Please do not merge this pull request. We disabled auto-merge because we are trying to merge a this big PR as part of sustainable architecture migration which is impossible with ever increasing stream of backports. We will resume the automerge after our PR is merged. Reach out to #sustainable-kibana-architecture for more info. |
|
Auto-merge has been re-enabled. Thank you for your patience. :heart |
|
@elasticmachine merge upstream |
|
@gbamparop Looks like this backport conflicts because #203492 hasn't been backported. I guess we should backport both of these? |
Created a backport here #204912 |
@flash1293 this is merged now |
|
Thanks @gbamparop , let's see whether it goes through now |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
Page load bundle
History
cc @flash1293 |
6 participants
Backport
This will backport the following commits from
mainto8.x:Questions ?
Please refer to the Backport tool documentation