Skip to content

[Fleet] Fix agent logs filtering from URL state#109982

Merged
jen-huang merged 1 commit intoelastic:masterfrom
jen-huang:fix/109402
Aug 25, 2021
Merged

[Fleet] Fix agent logs filtering from URL state#109982
jen-huang merged 1 commit intoelastic:masterfrom
jen-huang:fix/109402

Conversation

@jen-huang
Copy link
Copy Markdown
Contributor

Summary

Resolves #109402

In #106267 we converted Fleet routing to not use hashes. For agent logs, we have links to pre-populate filters from the URL state. This is done by using Kibana's URL state management helper, getStateFromKbnUrl, which by default assumes the state is stored in the hash part of the URL. This PR passes in the right argument to retrieve it from the main query instead.

@jen-huang jen-huang added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team auto-backport Deprecated - use backport:version if exact versions are needed v7.15.0 labels Aug 25, 2021
@jen-huang jen-huang self-assigned this Aug 25, 2021
@jen-huang jen-huang requested a review from a team as a code owner August 25, 2021 00:15
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@jen-huang jen-huang enabled auto-merge (squash) August 25, 2021 00:15
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 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 812.5KB 812.5KB +22.0B

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

cc @jen-huang

Copy link
Copy Markdown
Contributor

@hop-dev hop-dev left a comment

Choose a reason for hiding this comment

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

Nice! 👍

@jen-huang jen-huang merged commit 5d19559 into elastic:master Aug 25, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

The following labels were identified as gaps in your version labels and will be added automatically:

  • v7.16.0

If any of these should not be on your pull request, please manually remove them.

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Backport successful

Status Branch Result
7.15
7.x

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Aug 25, 2021
Co-authored-by: Jen Huang <its.jenetic@gmail.com>
kibanamachine added a commit that referenced this pull request Aug 25, 2021
Co-authored-by: Jen Huang <its.jenetic@gmail.com>
@jen-huang jen-huang deleted the fix/109402 branch August 25, 2021 15:26
@dikshachauhan-qasource
Copy link
Copy Markdown

Hi @EricDavisX

We are validating this PR in lieu of reported issue at #109402.

We have validated this on 7.15 BC3 cloud-Prod build.

  • User is now navigated to correct filtered logs when clicked on logs or metrics logs option under integration on agent details page.

Screenshot:
image

Build details:
Build: 43818
Commit: 6f7562b

Thanks
QAS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.15.0 v7.16.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fleet]: View logs and View metrics for System integration under Agent details page not giving correct results on Logs tab.

5 participants