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

Separate enroll and run Elastic agent into two steps #65749

Closed
kaiyan-sheng opened this issue May 7, 2020 · 6 comments
Closed

Separate enroll and run Elastic agent into two steps #65749

kaiyan-sheng opened this issue May 7, 2020 · 6 comments
Labels
Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@kaiyan-sheng
Copy link
Contributor

When trying to enroll a new agent, the third step shown in the screenshot below actually includes two separate parts: enroll the agent and start the agent.
Screen Shot 2020-05-07 at 11 03 01 AM

With the current "Copy command" button, both commands are copied at the same time. But these two commands are not able to run together because enroll has the prompt about overwriting old setting: This will replace your current settings. Do you want to continue? [Y/n]:.

Several potential solutions I can see:

  1. use --force to bypass the [Y/n] when enrolling the agent as @jfsiii mentioned in slack.
  2. separate enroll and run agent into two steps
  3. keep them in one step but make two "Copy command" button.
@kaiyan-sheng kaiyan-sheng added the Team:Fleet Team label for Observability Data Collection Fleet team label May 7, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/ingest-management (Team:Ingest Management)

@nchaulet
Copy link
Member

nchaulet commented May 7, 2020

@ph @ruflin I know we consider having a command doing both enrollment and run what the status of this command, In my opinion it will be a good option here.

@ruflin
Copy link
Contributor

ruflin commented May 11, 2020

Agree. @ph I think we discussed to do it for 7.9. But I couldn't find the issue in the Beats repo :-(

@ph
Copy link
Contributor

ph commented May 11, 2020

@ruflin here the issue elastic/beats#17330 :)

@ruflin
Copy link
Contributor

ruflin commented May 11, 2020

@blakerouse The above also ties into our discussion around a third command.

@ph
Copy link
Contributor

ph commented Oct 19, 2020

Closing this, replaced by "install" subcommand. elastic/beats#21206

@ph ph closed this as completed Oct 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Development

No branches or pull requests

5 participants