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.

Sign up for GitHub

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

Jump to bottom

[ML] Anomaly Explorer URL in alerting context doesn't contain a space id #160762

Closed
darnautov opened this issue Jun 28, 2023 · 1 comment · Fixed by #160899
Closed

[ML] Anomaly Explorer URL in alerting context doesn't contain a space id #160762

darnautov opened this issue Jun 28, 2023 · 1 comment · Fixed by #160899
Assignees
@darnautov
Labels
bug Fixes for quality problems that affect the customer experience Feature:Alerting/RuleTypes Issues related to specific Alerting Rules Types Feature:Anomaly Detection ML anomaly detection :ml Team:ML Team label for ML (also use :ml) v8.9.0 v8.10.0

Comments

@darnautov
Copy link
Contributor

The "Open in Anomaly Explorer" URL that is served as part of the alert context doesn't contain the current space identifier, hence it only works with the default space. In case an alert is triggered for an anomaly detection job that is assigned to another space, the URL is invalid and the user gets an error message about a not found job or a permission error without access to the default space.
@darnautov darnautov added bug Fixes for quality problems that affect the customer experience :ml Feature:Anomaly Detection ML anomaly detection Feature:Alerting/RuleTypes Issues related to specific Alerting Rules Types Team:ML Team label for ML (also use :ml) v8.10.0 labels Jun 28, 2023
@darnautov darnautov self-assigned this Jun 28, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@darnautov darnautov mentioned this issue Jun 29, 2023
Merged
1 task
darnautov added a commit that referenced this issue Jul 5, 2023
@darnautov
[ML] Fix Anomaly Explorer URL for alerting context with non-default s…
7728519 
…pace (#160899)

## Summary

Fixes #160762 


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Jul 5, 2023
@darnautov
[ML] Fix Anomaly Explorer URL for alerting context with non-default s…
67a97a3 
…pace (elastic#160899)

## Summary

Fixes elastic#160762

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 7728519)
kibanamachine added a commit that referenced this issue Jul 5, 2023
@kibanamachine @darnautov
[8.9] [ML] Fix Anomaly Explorer URL for alerting context with non-def…
b53af79 
…ault space (#160899) (#161235)

# Backport

This will backport the following commits from `main` to `8.9`:
- [[ML] Fix Anomaly Explorer URL for alerting context with non-default
space (#160899)](#160899)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Dima
Arnautov","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-07-05T08:53:02Z","message":"[ML]
Fix Anomaly Explorer URL for alerting context with non-default space
(#160899)\n\n## Summary\r\n\r\nFixes #160762 \r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"77285193045db36453f367d61552638218fde013","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","Feature:Anomaly
Detection","Team:ML","v8.9.0","v8.10.0"],"number":160899,"url":"https://github.com/elastic/kibana/pull/160899","mergeCommit":{"message":"[ML]
Fix Anomaly Explorer URL for alerting context with non-default space
(#160899)\n\n## Summary\r\n\r\nFixes #160762 \r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"77285193045db36453f367d61552638218fde013"}},"sourceBranch":"main","suggestedTargetBranches":["8.9"],"targetPullRequestStates":[{"branch":"8.9","label":"v8.9.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/160899","number":160899,"mergeCommit":{"message":"[ML]
Fix Anomaly Explorer URL for alerting context with non-default space
(#160899)\n\n## Summary\r\n\r\nFixes #160762 \r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"77285193045db36453f367d61552638218fde013"}}]}]
BACKPORT-->

Co-authored-by: Dima Arnautov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@darnautov darnautov

Labels
bug Fixes for quality problems that affect the customer experience Feature:Alerting/RuleTypes Issues related to specific Alerting Rules Types Feature:Anomaly Detection ML anomaly detection :ml Team:ML Team label for ML (also use :ml) v8.9.0 v8.10.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ML] Fixes Anomaly Explorer URL for alerting context with non-default space darnautov/kibana
2 participants
@darnautov @elasticmachine