Skip to content

Comments

[Synthetics] Fix for metric_item component#227969

Merged
cesco-f merged 2 commits intoelastic:mainfrom
cesco-f:update-synthetics-test-plan
Jul 15, 2025
Merged

[Synthetics] Fix for metric_item component#227969
cesco-f merged 2 commits intoelastic:mainfrom
cesco-f:update-synthetics-test-plan

Conversation

@cesco-f
Copy link
Contributor

@cesco-f cesco-f commented Jul 15, 2025

Fixed a bug when adding a single monitor to a dashboard.
In this same PR I've also updated the Synthetics test plan.

Before:

Screen.Recording.2025-07-15.at.12.11.42.mov

After:

Screen.Recording.2025-07-15.at.12.12.16.mov

@cesco-f cesco-f requested a review from a team as a code owner July 15, 2025 10:19
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label Jul 15, 2025
@elasticmachine
Copy link
Contributor

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

@github-actions github-actions bot added the author:obs-ux-management PRs authored by the obs ux management team label Jul 15, 2025
@cesco-f cesco-f enabled auto-merge (squash) July 15, 2025 10:34
@cesco-f cesco-f merged commit 3b9cfce into elastic:main Jul 15, 2025
11 of 12 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19, 9.1

https://github.com/elastic/kibana/actions/runs/16292763014

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #2 / Rule execution logic API - Basic License/Essentials Tier @ess @serverless @serverlessQA timestamp tests alerts generated from events with timestamp override field EQL should generate 2 alerts with @timestamp

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
synthetics 1.0MB 1.0MB -5.0B

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 15, 2025
@kibanamachine
Copy link
Contributor

cesco-f added a commit to cesco-f/kibana that referenced this pull request Jul 15, 2025
Fixed a bug when adding a single monitor to a dashboard.
In this same PR I've also updated the Synthetics test plan.

Before:

https://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a

After:

https://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4
(cherry picked from commit 3b9cfce)

# Conflicts:
#	x-pack/solutions/observability/plugins/synthetics/dev_docs/test_plan.md
@cesco-f
Copy link
Contributor Author

cesco-f commented Jul 15, 2025

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@cesco-f cesco-f deleted the update-synthetics-test-plan branch July 15, 2025 12:38
kibanamachine added a commit that referenced this pull request Jul 15, 2025
# Backport

This will backport the following commits from `main` to `9.1`:
- [[Synthetics] Fix for metric_item component
(#227969)](#227969)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Francesco
Fagnani","email":"fagnani.francesco@gmail.com"},"sourceCommit":{"committedDate":"2025-07-15T12:05:39Z","message":"[Synthetics]
Fix for metric_item component (#227969)\n\nFixed a bug when adding a
single monitor to a dashboard.\nIn this same PR I've also updated the
Synthetics test
plan.\n\nBefore:\n\n\nhttps://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a\n\nAfter:\n\n\nhttps://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4","sha":"3b9cfce984370d338169821dadb32bb6cf8ea88e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:obs-ux-management","backport:version","v9.1.0","v8.19.0","author:obs-ux-management","v9.2.0"],"title":"[Synthetics]
Fix for metric_item
component","number":227969,"url":"https://github.com/elastic/kibana/pull/227969","mergeCommit":{"message":"[Synthetics]
Fix for metric_item component (#227969)\n\nFixed a bug when adding a
single monitor to a dashboard.\nIn this same PR I've also updated the
Synthetics test
plan.\n\nBefore:\n\n\nhttps://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a\n\nAfter:\n\n\nhttps://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4","sha":"3b9cfce984370d338169821dadb32bb6cf8ea88e"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227969","number":227969,"mergeCommit":{"message":"[Synthetics]
Fix for metric_item component (#227969)\n\nFixed a bug when adding a
single monitor to a dashboard.\nIn this same PR I've also updated the
Synthetics test
plan.\n\nBefore:\n\n\nhttps://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a\n\nAfter:\n\n\nhttps://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4","sha":"3b9cfce984370d338169821dadb32bb6cf8ea88e"}}]}]
BACKPORT-->

Co-authored-by: Francesco Fagnani <fagnani.francesco@gmail.com>
cesco-f added a commit that referenced this pull request Jul 16, 2025
# Backport

This will backport the following commits from `main` to `8.19`:
- [[Synthetics] Fix for metric_item component
(#227969)](#227969)

<!--- Backport version: 10.0.1 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Francesco
Fagnani","email":"fagnani.francesco@gmail.com"},"sourceCommit":{"committedDate":"2025-07-15T12:05:39Z","message":"[Synthetics]
Fix for metric_item component (#227969)\n\nFixed a bug when adding a
single monitor to a dashboard.\nIn this same PR I've also updated the
Synthetics test
plan.\n\nBefore:\n\n\nhttps://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a\n\nAfter:\n\n\nhttps://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4","sha":"3b9cfce984370d338169821dadb32bb6cf8ea88e","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:obs-ux-management","backport:version","v9.1.0","v8.19.0","author:obs-ux-management","v9.2.0"],"title":"[Synthetics]
Fix for metric_item
component","number":227969,"url":"https://github.com/elastic/kibana/pull/227969","mergeCommit":{"message":"[Synthetics]
Fix for metric_item component (#227969)\n\nFixed a bug when adding a
single monitor to a dashboard.\nIn this same PR I've also updated the
Synthetics test
plan.\n\nBefore:\n\n\nhttps://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a\n\nAfter:\n\n\nhttps://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4","sha":"3b9cfce984370d338169821dadb32bb6cf8ea88e"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/227996","number":227996,"state":"OPEN"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/227969","number":227969,"mergeCommit":{"message":"[Synthetics]
Fix for metric_item component (#227969)\n\nFixed a bug when adding a
single monitor to a dashboard.\nIn this same PR I've also updated the
Synthetics test
plan.\n\nBefore:\n\n\nhttps://github.com/user-attachments/assets/6b76c75b-126b-486b-8dfa-5065a6d8dd7a\n\nAfter:\n\n\nhttps://github.com/user-attachments/assets/3aaf3d7e-7e86-4aca-b2ce-9dec613731f4","sha":"3b9cfce984370d338169821dadb32bb6cf8ea88e"}}]}]
BACKPORT-->
Bluefinger pushed a commit to Bluefinger/kibana that referenced this pull request Jul 22, 2025
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:obs-ux-management PRs authored by the obs ux management team backport:version Backport to applied version labels release_note:fix Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v8.19.0 v9.1.0 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants