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

Ability to share Synthetic monitoring between Spaces #194760

Closed
FreerideTheWorld opened this issue Oct 2, 2024 · 2 comments · Fixed by #196109
Closed

Ability to share Synthetic monitoring between Spaces #194760

FreerideTheWorld opened this issue Oct 2, 2024 · 2 comments · Fixed by #196109
Assignees
Labels
Team:obs-ux-management Observability Management User Experience Team

Comments

@FreerideTheWorld
Copy link

FreerideTheWorld commented Oct 2, 2024

Describe the feature:
Please allow Elasticsearch Admins the ability to share Synthetic monitors between Spaces.

Describe a specific use case for the feature:
We recently started working on some basic URL monitoring and realized that you have to build the monitor in the specific teams space. This means that any Elastic Admins need to move between Spaces to build/edit and check on these monitors. In an ideal scenario Elastic Admins have visibility to everything when logged into a role utilizing the superuser and kibana_admin roles, regardless of Space.

@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 2, 2024
@shahzad31 shahzad31 added the Team:obs-ux-management Observability Management User Experience Team label Oct 3, 2024
@elasticmachine
Copy link
Contributor

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

@botelastic botelastic bot removed the needs-team Issues missing a team label label Oct 3, 2024
@roshan-elastic
Copy link

Hey @drewpost - FYI

@shahzad31 shahzad31 self-assigned this Oct 14, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Oct 16, 2024
## Summary

Fixes elastic#194760 !!
Fixes elastic#169753 !!

Added an options to list monitors from all spaces which user has
permission for , user can either select default option which is to get
monitors from current space or all permitted spaces !!

### Testing

Create monitors in 3 spaces, assign 2 spaces to a role, and create a
user. Make sure monitors only appears to which user have space
permission.

<img width="1727" alt="image"
src="https://github.com/user-attachments/assets/972d213a-ee00-4950-be9f-a209393cb69a">

---------

Co-authored-by: Justin Kambic <[email protected]>
(cherry picked from commit 2f67874)
kibanamachine added a commit that referenced this issue Oct 16, 2024
#196596)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Synthetic] Show monitors from all permitted spaces !!
(#196109)](#196109)

<!--- 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-10-16T19:08:34Z","message":"[Synthetic]
Show monitors from all permitted spaces !! (#196109)\n\n##
Summary\n\nFixes #194760
!!\nFixes #169753 !!\n\nAdded an
options to list monitors from all spaces which user has\npermission for
, user can either select default option which is to get\nmonitors from
current space or all permitted spaces !!\n\n### Testing\n\nCreate
monitors in 3 spaces, assign 2 spaces to a role, and create a\nuser.
Make sure monitors only appears to which user have
space\npermission.\n\n<img width=\"1727\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/972d213a-ee00-4950-be9f-a209393cb69a\">\n\n---------\n\nCo-authored-by:
Justin Kambic
<[email protected]>","sha":"2f678744ab3b512cf5e212671a35b81edd1aeec9","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["v9.0.0","release_note:feature","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-management","v8.16.0"],"title":"[Synthetic]
Show monitors from all permitted spaces
!!","number":196109,"url":"https://github.com/elastic/kibana/pull/196109","mergeCommit":{"message":"[Synthetic]
Show monitors from all permitted spaces !! (#196109)\n\n##
Summary\n\nFixes #194760
!!\nFixes #169753 !!\n\nAdded an
options to list monitors from all spaces which user has\npermission for
, user can either select default option which is to get\nmonitors from
current space or all permitted spaces !!\n\n### Testing\n\nCreate
monitors in 3 spaces, assign 2 spaces to a role, and create a\nuser.
Make sure monitors only appears to which user have
space\npermission.\n\n<img width=\"1727\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/972d213a-ee00-4950-be9f-a209393cb69a\">\n\n---------\n\nCo-authored-by:
Justin Kambic
<[email protected]>","sha":"2f678744ab3b512cf5e212671a35b81edd1aeec9"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196109","number":196109,"mergeCommit":{"message":"[Synthetic]
Show monitors from all permitted spaces !! (#196109)\n\n##
Summary\n\nFixes #194760
!!\nFixes #169753 !!\n\nAdded an
options to list monitors from all spaces which user has\npermission for
, user can either select default option which is to get\nmonitors from
current space or all permitted spaces !!\n\n### Testing\n\nCreate
monitors in 3 spaces, assign 2 spaces to a role, and create a\nuser.
Make sure monitors only appears to which user have
space\npermission.\n\n<img width=\"1727\"
alt=\"image\"\nsrc=\"https://github.com/user-attachments/assets/972d213a-ee00-4950-be9f-a209393cb69a\">\n\n---------\n\nCo-authored-by:
Justin Kambic
<[email protected]>","sha":"2f678744ab3b512cf5e212671a35b81edd1aeec9"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Shahzad <[email protected]>
hop-dev pushed a commit to hop-dev/kibana that referenced this issue Oct 17, 2024
## Summary

Fixes elastic#194760 !!
Fixes elastic#169753 !!

Added an options to list monitors from all spaces which user has
permission for , user can either select default option which is to get
monitors from current space or all permitted spaces !!

### Testing

Create monitors in 3 spaces, assign 2 spaces to a role, and create a
user. Make sure monitors only appears to which user have space
permission.

<img width="1727" alt="image"
src="https://github.com/user-attachments/assets/972d213a-ee00-4950-be9f-a209393cb69a">

---------

Co-authored-by: Justin Kambic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:obs-ux-management Observability Management User Experience Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants