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

[Fleet] Bulk action refactor and small fixes #142299

Merged
merged 11 commits into from
Oct 3, 2022

Conversation

juliaElastic
Copy link
Contributor

@juliaElastic juliaElastic commented Sep 30, 2022

Summary

Related to #141567

Refactor to report errors in agent update for all actions.
Refactor to reduce duplication around writing out errors as action results.

Fix for action status ack calculation: sometimes the cardinality agg is not precise (after 40k) and returns a bigger number than the doc count, taking the minimum in this case.
Before:
image
After:
image

Fix for an edge case in update tags: when adding and removing a tag quickly on a large agent selection, the remove tag request was not correct.

tags_fix.mov

Checklist

@juliaElastic juliaElastic self-assigned this Sep 30, 2022
@juliaElastic juliaElastic changed the title [Fleet] refactor to report errors in agent update for all actions [Fleet] Bulk action refactor and small fixes Sep 30, 2022
@juliaElastic juliaElastic added the release_note:skip Skip the PR/issue when compiling release notes label Sep 30, 2022
@juliaElastic juliaElastic marked this pull request as ready for review September 30, 2022 08:25
@juliaElastic juliaElastic requested a review from a team as a code owner September 30, 2022 08:25
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Sep 30, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@juliaElastic
Copy link
Contributor Author

juliaElastic commented Sep 30, 2022

@elasticmachine merge upstream

@juliaElastic
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@criamico criamico left a comment

Choose a reason for hiding this comment

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

I left a small comment, code LGTM

x-pack/plugins/fleet/server/services/agents/actions.ts Outdated Show resolved Hide resolved
@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 914.9KB 915.0KB +44.0B

History

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

cc @juliaElastic

@juliaElastic juliaElastic merged commit f8a9027 into elastic:main Oct 3, 2022
@juliaElastic juliaElastic added the backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) label Oct 3, 2022
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.5

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 3, 2022
* refactor to report errors in agent update for all actions

* fixed tests

* fixed types

* refactor to reduce duplication

* fixed test

* fix for cypress test

* passing error reason

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit f8a9027)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.5

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 Oct 3, 2022
* refactor to report errors in agent update for all actions

* fixed tests

* fixed types

* refactor to reduce duplication

* fixed test

* fix for cypress test

* passing error reason

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit f8a9027)

Co-authored-by: Julia Bardi <[email protected]>
WafaaNasr pushed a commit to WafaaNasr/kibana that referenced this pull request Oct 11, 2022
* refactor to report errors in agent update for all actions

* fixed tests

* fixed types

* refactor to reduce duplication

* fixed test

* fix for cypress test

* passing error reason

Co-authored-by: Kibana Machine <[email protected]>
WafaaNasr pushed a commit to WafaaNasr/kibana that referenced this pull request Oct 14, 2022
* refactor to report errors in agent update for all actions

* fixed tests

* fixed types

* refactor to reduce duplication

* fixed test

* fix for cypress test

* passing error reason

Co-authored-by: Kibana Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.5.0 v8.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants