Skip to content
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

[8.14] [ResponseOps] Bring back EuiThemeProvider to fix o11y and stack monitoring rules in stack management (#188724) #188904

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

…oring rules in stack management (elastic#188724)

Related to elastic#186969 and
elastic/response-ops-team#218

## Summary

This PR brings back EuiThemeProvider to fix o11y and stack monitoring
rules in stack management.

## To check/do

- [x] Add an APM test that fails without this fix
- [x] Check if this solves the related SDH for [CPU
Usage](elastic/sdh-kibana#4829)
- Yes, it will solve that issue
([comment](elastic/sdh-kibana#4829 (comment)))
- [x] Smoke test **ALL** rule types in stack management
    - Can we load the rule form?
    - Can we adjust all the input fields without error?
    - Does it work both with and without data in the preview chart?

(cherry picked from commit ed32c98)
@kibanamachine kibanamachine enabled auto-merge (squash) July 23, 2024 08:47
@botelastic botelastic bot added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Jul 23, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 3.2MB 3.2MB +872.0B
triggersActionsUi 1.6MB 1.6MB +131.0B
total +1003.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @maryam-saeidi

@kibanamachine kibanamachine merged commit 7d17731 into elastic:8.14 Jul 23, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants