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

[UII] Fix Fleet Kibana SOs not loading on _debug page #183948

Merged
merged 1 commit into from
May 21, 2024

Conversation

jen-huang
Copy link
Contributor

@jen-huang jen-huang commented May 21, 2024

Summary

As the title says :)

@jen-huang jen-huang added release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.14.0 v8.15.0 labels May 21, 2024
@jen-huang jen-huang self-assigned this May 21, 2024
@jen-huang jen-huang requested a review from a team as a code owner May 21, 2024 17:40
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@jen-huang jen-huang enabled auto-merge (squash) May 21, 2024 17:41
Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

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

LGTM 🚀
Not really related to that PR user do not have access to that index in serverless, should we hide that section here? maybe not needed as the _debug is already not accessible without the url

@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
fleet 1.3MB 1.3MB +8.0B

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

cc @jen-huang

@jen-huang jen-huang merged commit 9e109af into elastic:main May 21, 2024
28 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 21, 2024
## Summary

As the title says :)

(cherry picked from commit 9e109af)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.14

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 May 21, 2024
…age (#183948) (#183952)

# Backport

This will backport the following commits from `main` to `8.14`:
- [[UII] Fix Fleet Kibana SOs not loading on `_debug` page
(#183948)](#183948)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Jen
Huang","email":"its.jenetic@gmail.com"},"sourceCommit":{"committedDate":"2024-05-21T18:58:04Z","message":"[UII]
Fix Fleet Kibana SOs not loading on `_debug` page (#183948)\n\n##
Summary\r\n\r\nAs the title says
:)","sha":"9e109aff36e66b7eb3ee07115349180b04188774","branchLabelMapping":{"^v8.15.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v8.14.0","v8.15.0"],"title":"[UII]
Fix Fleet Kibana SOs not loading on `_debug`
page","number":183948,"url":"#183948
Fix Fleet Kibana SOs not loading on `_debug` page (#183948)\n\n##
Summary\r\n\r\nAs the title says
:)","sha":"9e109aff36e66b7eb3ee07115349180b04188774"}},"sourceBranch":"main","suggestedTargetBranches":["8.14"],"targetPullRequestStates":[{"branch":"8.14","label":"v8.14.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.15.0","branchLabelMappingKey":"^v8.15.0$","isSourceBranch":true,"state":"MERGED","url":"#183948
Fix Fleet Kibana SOs not loading on `_debug` page (#183948)\n\n##
Summary\r\n\r\nAs the title says
:)","sha":"9e109aff36e66b7eb3ee07115349180b04188774"}}]}] BACKPORT-->

Co-authored-by: Jen Huang <its.jenetic@gmail.com>
@jen-huang jen-huang deleted the fix/debugger-so branch May 21, 2024 21:51
@jen-huang
Copy link
Contributor Author

Not really related to that PR user do not have access to that index in serverless, should we hide that section here? maybe not needed as the _debug is already not accessible without the url

I don't really see this as a product page where we should enforce a lot of conditions. We should hide that section if the page errors out completely on serverless though (haven't tested).

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:Fleet Team label for Observability Data Collection Fleet team v8.14.0 v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants