-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[SLOs] Slo form little things #193990
[SLOs] Slo form little things #193990
Conversation
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
478b489
to
0e5e25d
Compare
0e5e25d
to
8131e8e
Compare
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
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, done just a code review on owned files
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!
Starting backport for target branches: 8.x https://github.com/elastic/kibana/actions/runs/11069476859 |
## Summary Slo form little things !! Will auto add values for APM defaults to all instead of forcing user !! ### Changes Details | Before | After -- | -- | -- Data View component | <img width="1262" alt="image" src="https://github.com/user-attachments/assets/79a5cbcd-04e1-49e6-aa86-036b36c8a6e6"> | <img width="1391" alt="image" src="https://github.com/user-attachments/assets/973db8f5-179d-4f89-b57b-39c7b57eb360"> APM SLI | <img width="1183" alt="image" src="https://github.com/user-attachments/assets/a3e083fa-df13-408b-b02d-e26695398225"> | <img width="1158" alt="image" src="https://github.com/user-attachments/assets/a94cfe07-32b0-4367-95e8-ab10d232e98c"> (cherry picked from commit c2a2042)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.x`: - [[SLOs] Slo form little things (#193990)](#193990) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-27T11:04:20Z","message":"[SLOs] Slo form little things (#193990)\n\n## Summary\r\n\r\nSlo form little things !!\r\n\r\nWill auto add values for APM defaults to all instead of forcing user !!\r\n\r\n### Changes \r\n\r\nDetails | Before | After\r\n-- | -- | --\r\nData View component | <img width=\"1262\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/79a5cbcd-04e1-49e6-aa86-036b36c8a6e6\">\r\n| <img width=\"1391\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/973db8f5-179d-4f89-b57b-39c7b57eb360\">\r\nAPM SLI | <img width=\"1183\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a3e083fa-df13-408b-b02d-e26695398225\">\r\n| <img width=\"1158\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a94cfe07-32b0-4367-95e8-ab10d232e98c\">","sha":"c2a204269af459fc05145d2977493ad975c8df62","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-management"],"title":"[SLOs] Slo form little things","number":193990,"url":"https://github.com/elastic/kibana/pull/193990","mergeCommit":{"message":"[SLOs] Slo form little things (#193990)\n\n## Summary\r\n\r\nSlo form little things !!\r\n\r\nWill auto add values for APM defaults to all instead of forcing user !!\r\n\r\n### Changes \r\n\r\nDetails | Before | After\r\n-- | -- | --\r\nData View component | <img width=\"1262\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/79a5cbcd-04e1-49e6-aa86-036b36c8a6e6\">\r\n| <img width=\"1391\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/973db8f5-179d-4f89-b57b-39c7b57eb360\">\r\nAPM SLI | <img width=\"1183\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a3e083fa-df13-408b-b02d-e26695398225\">\r\n| <img width=\"1158\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a94cfe07-32b0-4367-95e8-ab10d232e98c\">","sha":"c2a204269af459fc05145d2977493ad975c8df62"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/193990","number":193990,"mergeCommit":{"message":"[SLOs] Slo form little things (#193990)\n\n## Summary\r\n\r\nSlo form little things !!\r\n\r\nWill auto add values for APM defaults to all instead of forcing user !!\r\n\r\n### Changes \r\n\r\nDetails | Before | After\r\n-- | -- | --\r\nData View component | <img width=\"1262\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/79a5cbcd-04e1-49e6-aa86-036b36c8a6e6\">\r\n| <img width=\"1391\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/973db8f5-179d-4f89-b57b-39c7b57eb360\">\r\nAPM SLI | <img width=\"1183\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a3e083fa-df13-408b-b02d-e26695398225\">\r\n| <img width=\"1158\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a94cfe07-32b0-4367-95e8-ab10d232e98c\">","sha":"c2a204269af459fc05145d2977493ad975c8df62"}}]}] BACKPORT--> Co-authored-by: Shahzad <[email protected]>
Summary
Slo form little things !!
Will auto add values for APM defaults to all instead of forcing user !!
Changes