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

Don't show "You have not been authorized to create any Rule types" message while request to get rule types is in flight #145407

Conversation

CoenWarmer
Copy link
Contributor

@CoenWarmer CoenWarmer commented Nov 16, 2022

Closes #145396.

Summary

This PR makes sure /api/alerting/rule_types is done loading before it shows the available rules. Therefore, the UI will not default anymore to the "You have not been authorized to create any Rule types" message while the request is in flight.

Screen.Recording.2022-11-16.at.17.13.08.mov

@kibana-ci
Copy link
Collaborator

💚 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
triggersActionsUi 681.3KB 681.3KB +27.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 67 73 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@XavierM XavierM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks good! but it will have been nice to have some kind of test but it look like that we did not have any at the first place :(

@XavierM XavierM added v8.6.0 release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" and removed backport labels Nov 16, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

@CoenWarmer CoenWarmer merged commit e01649e into elastic:main Nov 17, 2022
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 17, 2022
…ssage while request to get rule types is in flight (elastic#145407)

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

💚 All backports created successfully

Status Branch Result
8.6

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 17, 2022
…es" message while request to get rule types is in flight (#145407) (#145506)

# Backport

This will backport the following commits from `main` to `8.6`:
- [Don't show "You have not been authorized to create any Rule types"
message while request to get rule types is in flight
(#145407)](#145407)

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

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

<!--BACKPORT [{"author":{"name":"Coen
Warmer","email":"[email protected]"},"sourceCommit":{"committedDate":"2022-11-17T07:19:07Z","message":"Don't
show \"You have not been authorized to create any Rule types\" message
while request to get rule types is in flight
(#145407)","sha":"e01649e7b23d9bb0b855b2ba2081ae72cc49f815","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","Team:
Actionable
Observability","v8.6.0","v8.7.0"],"number":145407,"url":"https://github.com/elastic/kibana/pull/145407","mergeCommit":{"message":"Don't
show \"You have not been authorized to create any Rule types\" message
while request to get rule types is in flight
(#145407)","sha":"e01649e7b23d9bb0b855b2ba2081ae72cc49f815"}},"sourceBranch":"main","suggestedTargetBranches":["8.6"],"targetPullRequestStates":[{"branch":"8.6","label":"v8.6.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/145407","number":145407,"mergeCommit":{"message":"Don't
show \"You have not been authorized to create any Rule types\" message
while request to get rule types is in flight
(#145407)","sha":"e01649e7b23d9bb0b855b2ba2081ae72cc49f815"}}]}]
BACKPORT-->

Co-authored-by: Coen Warmer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.6.0 v8.7.0
Projects
None yet
5 participants