Skip to content

[Fleet] keep all agents selected in query selection mode - #135530

Merged
juliaElastic merged 1 commit into
elastic:mainfrom
juliaElastic:feat/query-force-select
Jun 30, 2022
Merged

juliaElastic merged 1 commit into
elastic:mainfrom
juliaElastic:feat/query-force-select

Conversation

@juliaElastic

Copy link
Copy Markdown
Contributor

Summary

Fix for #133523 (comment)

To verify:

  • enroll 20+ agents with horde
  • select all agents on agent list view by clicking Select everything on all pages
  • wait for agent refresh, eventually the agents on current page will change
  • expected: all agents should continue to be selected (all checkboxes checked). All agents selected label is still visible (query mode).

image

Checklist

@juliaElastic juliaElastic added release_note:fix auto-backport Deprecated - use backport:version if exact versions are needed v8.4.0 v8.3.1 labels Jun 30, 2022
@juliaElastic
juliaElastic requested a review from a team as a code owner June 30, 2022 11:05
@juliaElastic juliaElastic self-assigned this Jun 30, 2022
@botelastic botelastic Bot added the Team:Fleet - DEPRECATED Use Team:streams-ui label Jun 30, 2022
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@kpollich kpollich left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM - small change + added test assertion 🚀

Comment on lines +356 to +360
} else {
// force selecting all agents on current page if staying in query mode
if (tableRef?.current) {
tableRef.current.setSelection(agents);
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change makes sense to me. Thanks for the comment!

@kibana-ci

Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Fleet Cypress Tests / Add Integration - Real API should install integration without policy

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 837.7KB 837.8KB +48.0B

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

cc @juliaElastic

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
✅ 8.3

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 Jun 30, 2022
(cherry picked from commit 19caaef)

Co-authored-by: Julia Bardi <90178898+juliaElastic@users.noreply.github.com>
yakhinvadim pushed a commit to yakhinvadim/kibana that referenced this pull request Jul 5, 2022
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

Labels

auto-backport Deprecated - use backport:version if exact versions are needed release_note:fix Team:Fleet - DEPRECATED Use Team:streams-ui v8.3.2 v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants