Skip to content

[Monitoring] Add catch clause to handle exceptions on loading page - #82179

Merged
chrisronline merged 2 commits into
elastic:masterfrom
chrisronline:monitoring/handle_loading_page_errors
Nov 2, 2020
Merged

chrisronline merged 2 commits into
elastic:masterfrom
chrisronline:monitoring/handle_loading_page_errors

Conversation

@chrisronline

Copy link
Copy Markdown
Contributor

Resolves #82128

The easiest way to reproduce this is to configure your local Kibana to a remote monitoring cluster (through monitoring.ui.elasticsearch.* configs) using credentials that do not also exist on your local Elasticsearch cluster.

Once that is setup, try and load the Stack Monitoring UI through the left nav.

It should land on the #/no-access and show a proper error message (instead of an infinite Loading spinner)

@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/stack-monitoring (Team:Monitoring)

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

async chunks size

id before after diff
monitoring 964.4KB 964.5KB +161.0B

History

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

@igoristic igoristic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Seems to be woking 👍

@chrisronline
chrisronline merged commit 53a4fcd into elastic:master Nov 2, 2020
@chrisronline
chrisronline deleted the monitoring/handle_loading_page_errors branch November 2, 2020 19:21
chrisronline added a commit to chrisronline/kibana that referenced this pull request Nov 2, 2020
chrisronline added a commit to chrisronline/kibana that referenced this pull request Nov 2, 2020
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Monitoring] Access denied page is not working

4 participants