kibana icon indicating copy to clipboard operation
kibana copied to clipboard

[UII] Consider removing or refactoring agent count in confirmation modals

Open jen-huang opened this issue 1 year ago • 3 comments

When we update settings objects, we display a count of affected policies and agents:

image image

However this is not done all the time, for example updating proxy only shows a generic confirmation message without counts:

image

As relationships between policies and settings objects have gotten more complex (multiple agent policies, per integration outputs, space awareness), we should consider removing these granular counts in favor of a generic message to reduce the need for these deep lookups.

Alternatively, look into simplifying the logic and/or moving them server-side.

Original discussion: https://github.com/elastic/kibana/pull/189125#discussion_r1713870533

jen-huang avatar Aug 13 '24 15:08 jen-huang

Pinging @elastic/fleet (Team:Fleet)

elasticmachine avatar Aug 13 '24 15:08 elasticmachine

This could potentially be a welcome simplification for confirmation modals triggered by bulk agent actions, as there are props passed down for that purpose only: https://github.com/elastic/kibana/blob/439c7fa84c45b3c632193ce0ffd16b437ea21e08/x-pack/plugins/fleet/public/applications/fleet/sections/agents/agent_list_page/components/bulk_actions.tsx#L93-L96

jillguyonnet avatar Aug 19 '24 15:08 jillguyonnet

If this can help to make this a priority this will allow to support a higher number of agent policies (fetching more than 1000 agent policies) make the UI extremely slow more than 2 minute to get the confirm message when updating a default output https://github.com/elastic/ingest-dev/issues/3343

nchaulet avatar Aug 30 '24 17:08 nchaulet