Skip to content

[Synthetics] Pass readonly to monitor spaces component !!#230714

Merged
shahzad31 merged 2 commits intoelastic:mainfrom
shahzad31:fix-space-editing-for-project-monitor
Aug 6, 2025
Merged

[Synthetics] Pass readonly to monitor spaces component !!#230714
shahzad31 merged 2 commits intoelastic:mainfrom
shahzad31:fix-space-editing-for-project-monitor

Conversation

@shahzad31
Copy link
Contributor

@shahzad31 shahzad31 commented Aug 6, 2025

Summary

Pass readonly to monitor spaces component, caught this while checking project monitor config, where it should have been disabled !!

Before

image

After

image

@shahzad31 shahzad31 requested a review from a team as a code owner August 6, 2025 08:07
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label Aug 6, 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 Aug 6, 2025
@shahzad31 shahzad31 added release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels v9.1.1 v8.19.1 and removed author:obs-ux-management PRs authored by the obs ux management team labels Aug 6, 2025
@shahzad31 shahzad31 enabled auto-merge (squash) August 6, 2025 08:16
@shahzad31 shahzad31 merged commit c805fab into elastic:main Aug 6, 2025
12 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19, 9.1

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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 +22.0B

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 6, 2025
…0714)

## Summary

Pass readonly to monitor spaces component, caught this while checking
project monitor config, where it should have been disabled !!

### Before
<img width="1445" height="402" alt="image"
src="https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca"
/>

### After
<img width="1388" height="562" alt="image"
src="https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de"
/>

(cherry picked from commit c805fab)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 6, 2025
…0714)

## Summary

Pass readonly to monitor spaces component, caught this while checking
project monitor config, where it should have been disabled !!

### Before
<img width="1445" height="402" alt="image"
src="https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca"
/>

### After
<img width="1388" height="562" alt="image"
src="https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de"
/>

(cherry picked from commit c805fab)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.19
9.1

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

Questions ?

Please refer to the Backport tool documentation

@shahzad31 shahzad31 deleted the fix-space-editing-for-project-monitor branch August 6, 2025 14:20
kibanamachine added a commit that referenced this pull request Aug 6, 2025
) (#230794)

# Backport

This will backport the following commits from `main` to `9.1`:
- [[Synthetics] Pass readonly to monitor spaces component !!
(#230714)](#230714)

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

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

<!--BACKPORT
[{"author":{"name":"Shahzad","email":"shahzad31comp@gmail.com"},"sourceCommit":{"committedDate":"2025-08-06T13:32:21Z","message":"[Synthetics]
Pass readonly to monitor spaces component !! (#230714)\n\n##
Summary\n\nPass readonly to monitor spaces component, caught this while
checking\nproject monitor config, where it should have been disabled
!!\n\n### Before\n<img width=\"1445\" height=\"402\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca\"\n/>\n\n\n\n###
After\n<img width=\"1388\" height=\"562\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de\"\n/>","sha":"c805fab7f918db9521d8403267867ca0e4d1c3c7","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-management","backport:version","v9.2.0","v9.1.1","v8.19.1"],"title":"[Synthetics]
Pass readonly to monitor spaces component
!!","number":230714,"url":"https://github.com/elastic/kibana/pull/230714","mergeCommit":{"message":"[Synthetics]
Pass readonly to monitor spaces component !! (#230714)\n\n##
Summary\n\nPass readonly to monitor spaces component, caught this while
checking\nproject monitor config, where it should have been disabled
!!\n\n### Before\n<img width=\"1445\" height=\"402\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca\"\n/>\n\n\n\n###
After\n<img width=\"1388\" height=\"562\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de\"\n/>","sha":"c805fab7f918db9521d8403267867ca0e4d1c3c7"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230714","number":230714,"mergeCommit":{"message":"[Synthetics]
Pass readonly to monitor spaces component !! (#230714)\n\n##
Summary\n\nPass readonly to monitor spaces component, caught this while
checking\nproject monitor config, where it should have been disabled
!!\n\n### Before\n<img width=\"1445\" height=\"402\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca\"\n/>\n\n\n\n###
After\n<img width=\"1388\" height=\"562\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de\"\n/>","sha":"c805fab7f918db9521d8403267867ca0e4d1c3c7"}},{"branch":"9.1","label":"v9.1.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Shahzad <shahzad31comp@gmail.com>
kibanamachine added a commit that referenced this pull request Aug 6, 2025
…0714) (#230793)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Synthetics] Pass readonly to monitor spaces component !!
(#230714)](#230714)

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

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

<!--BACKPORT
[{"author":{"name":"Shahzad","email":"shahzad31comp@gmail.com"},"sourceCommit":{"committedDate":"2025-08-06T13:32:21Z","message":"[Synthetics]
Pass readonly to monitor spaces component !! (#230714)\n\n##
Summary\n\nPass readonly to monitor spaces component, caught this while
checking\nproject monitor config, where it should have been disabled
!!\n\n### Before\n<img width=\"1445\" height=\"402\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca\"\n/>\n\n\n\n###
After\n<img width=\"1388\" height=\"562\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de\"\n/>","sha":"c805fab7f918db9521d8403267867ca0e4d1c3c7","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-management","backport:version","v9.2.0","v9.1.1","v8.19.1"],"title":"[Synthetics]
Pass readonly to monitor spaces component
!!","number":230714,"url":"https://github.com/elastic/kibana/pull/230714","mergeCommit":{"message":"[Synthetics]
Pass readonly to monitor spaces component !! (#230714)\n\n##
Summary\n\nPass readonly to monitor spaces component, caught this while
checking\nproject monitor config, where it should have been disabled
!!\n\n### Before\n<img width=\"1445\" height=\"402\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca\"\n/>\n\n\n\n###
After\n<img width=\"1388\" height=\"562\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de\"\n/>","sha":"c805fab7f918db9521d8403267867ca0e4d1c3c7"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230714","number":230714,"mergeCommit":{"message":"[Synthetics]
Pass readonly to monitor spaces component !! (#230714)\n\n##
Summary\n\nPass readonly to monitor spaces component, caught this while
checking\nproject monitor config, where it should have been disabled
!!\n\n### Before\n<img width=\"1445\" height=\"402\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca\"\n/>\n\n\n\n###
After\n<img width=\"1388\" height=\"562\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de\"\n/>","sha":"c805fab7f918db9521d8403267867ca0e4d1c3c7"}},{"branch":"9.1","label":"v9.1.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Shahzad <shahzad31comp@gmail.com>
gergoabraham pushed a commit to gergoabraham/kibana that referenced this pull request Aug 7, 2025
…0714)

## Summary

Pass readonly to monitor spaces component, caught this while checking
project monitor config, where it should have been disabled !!

### Before
<img width="1445" height="402" alt="image"
src="https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca"
/>



### After
<img width="1388" height="562" alt="image"
src="https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de"
/>
@wildemat wildemat mentioned this pull request Aug 7, 2025
10 tasks
@mistic mistic added the v8.19.2 label Aug 7, 2025
denar50 pushed a commit to denar50/kibana that referenced this pull request Aug 8, 2025
…0714)

## Summary

Pass readonly to monitor spaces component, caught this while checking
project monitor config, where it should have been disabled !!

### Before
<img width="1445" height="402" alt="image"
src="https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca"
/>



### After
<img width="1388" height="562" alt="image"
src="https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de"
/>
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Aug 18, 2025
…0714)

## Summary

Pass readonly to monitor spaces component, caught this while checking
project monitor config, where it should have been disabled !!

### Before
<img width="1445" height="402" alt="image"
src="https://github.com/user-attachments/assets/aed1f3b3-e802-4468-9685-b0960d4e54ca"
/>



### After
<img width="1388" height="562" alt="image"
src="https://github.com/user-attachments/assets/a876b33e-61d4-48e0-a0dd-fbb8352be4de"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v8.19.2 v9.1.2 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants