-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Ingest] Allow to reassign agent to a new config #63847
[Ingest] Allow to reassign agent to a new config #63847
Conversation
Pinging @elastic/ingest-management (Team:Ingest Management) |
Lets focus this for now on just reassigning a single agent to a new config. This will make also error handling easier so we don't have to think about what if 2 out of 4 reassignements worked? |
…-support-config-reassignment
Updated the API to not support bulk operations. |
...applications/ingest_manager/sections/fleet/components/agent_reassign_config_flyout/index.tsx
Outdated
Show resolved
Hide resolved
@neptunian no the error is not expected, just tested locally it's working fine for me, how do you get the error? |
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When the reassignment flyout is open on the agent list page with many agents, the context of which agent is being reassigned gets lost since we don't have a visual indicator of which agent is selected (when it was a bulk action, the indicator is selected rows, but we are not doing bulk at the moment). Maybe we can include the selected agent host as part of the flyout info. @hbharding could you suggest a design for this? (Not a blocker for this PR; enhancement can be done in follow up PR.)
I have some other small copy suggestions below, otherwise tested locally and LGTM 👍
...s/ingest_manager/public/applications/ingest_manager/sections/fleet/agent_list_page/index.tsx
Outdated
Show resolved
Hide resolved
...applications/ingest_manager/sections/fleet/components/agent_reassign_config_flyout/index.tsx
Outdated
Show resolved
Hide resolved
...applications/ingest_manager/sections/fleet/components/agent_reassign_config_flyout/index.tsx
Outdated
Show resolved
Hide resolved
...applications/ingest_manager/sections/fleet/components/agent_reassign_config_flyout/index.tsx
Outdated
Show resolved
Hide resolved
...applications/ingest_manager/sections/fleet/components/agent_reassign_config_flyout/index.tsx
Outdated
Show resolved
Hide resolved
Btw I also see the same errors in agent sdout that @neptunian saw:
I think I have seen the above ^ outside of this PR, though. I also have a bunch of errors like this:
The only thing I did was rapidly reassign an agent between two configs. |
@jen-huang did you re-enroll multiple time an agent from the same folder? if yes it's probably related to this error elastic/beats#17389 |
Co-Authored-By: Jen Huang <its.jenetic@gmail.com>
Interesting, that may be the case. Thanks for the link. |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
Summary
Resolves #63021
This PR allow a fleet user to assign an already enrolled agent to a new configuration.
Done in this PR
UI Changes
API Change
Create a new route similar to the one used for unenrolling agent