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

[Ingest Manager] Enrollment Flyout text corrections to cite correct rpm/deb binary path and to cite use Administrator privileges #73932

Closed
EricDavisX opened this issue Jul 31, 2020 · 4 comments
Labels
Team:Fleet Team label for Observability Data Collection Fleet team v7.9.0

Comments

@EricDavisX
Copy link
Contributor

[Ingest Manager] Enrollment Flyout text corrections to cite correct binary path and to use Administrator rights

Kibana version:
7.9 BC5 deployed to elastic cloud prod

Describe the bug:
the enrollment flyout doesn't cite to use Administrator access, due to this incorrect usage happened a lot during an internal dog-fooding test and we expect customer / user usage to follow suit. we should call this out in the flyout if possible, instead of leaving it to docs. secondly, the .rpm/.deb instructions are mis-leading indicating a binary path that doesn't exist

Steps to reproduce:

  1. install stack of 7.9, BC latest
  2. to to ingest manager and enable Fleet user
  3. click on the 'enroll new agent' flyout and see text.
  • it was recently updated and is massively improved, but can be tweaked a bit more.

Expected behavior:
the instructions are sufficient to enroll Agent and it set up with elastic solutions pieces (Endpoint Security)

The 2 suggestions I suggest we make are:

  1. to remove the ./ from the .rpm/.deb section for Linux. This is because the service executable can be used, and after install, there is no local binary to call in that location so it doesn't work.
    for example, instead of citing this:
    ./elastic-agent enroll https://8a4af220da674c2287033be796421b1b.europe-west1.gcp.cloud.es.io:443 V1VpTm4zTUI5WjU5ZUl5X2tlWkY6NjBrSUk4TkpUQ3VvVU5odGdQMUpudw==

it would cite:
elastic-agent enroll https://8a4af220da674c2287033be796421b1b.europe-west1.gcp.cloud.es.io:443 V1VpTm4zTUI5WjU5ZUl5X2tlWkY6NjBrSUk4TkpUQ3VvVU5odGdQMUpudw==

  1. add this text to the step 3 after the current text to cite Administator privilege is required.
    for example, from this:
    From the agent’s directory, run the appropriate commands to enroll and start an Elastic Agent. You can reuse these commands to setup agents on more than one machine.

to:
From the agent’s directory, run the appropriate commands to enroll and start an Elastic Agent. You can reuse these commands to setup agents on more than one machine. Be sure to run the enrollment steps as a user with Administrator privilege on the system.

Screenshots (if relevant):
Screen Shot 2020-07-31 at 9 16 25 AM

@EricDavisX EricDavisX added Ingest Management:beta1 Team:Fleet Team label for Observability Data Collection Fleet team labels Jul 31, 2020
@elasticmachine
Copy link
Contributor

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

EricDavisX added a commit that referenced this issue Aug 3, 2020
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
EricDavisX added a commit to EricDavisX/kibana that referenced this issue Aug 3, 2020
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
EricDavisX added a commit that referenced this issue Aug 3, 2020
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
EricDavisX added a commit that referenced this issue Aug 3, 2020
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@EricDavisX
Copy link
Contributor Author

backports are open for 7.x and 7.9 after merge of master for final change

  • closing this out

@EricDavisX EricDavisX reopened this Aug 5, 2020
@EricDavisX
Copy link
Contributor Author

repoening: I followed poor process and didn't fully test the / to \ change - I'll stage a revert immediately and also look at what likely encoding problem this is...

@EricDavisX
Copy link
Contributor Author

master, 7.x and 7.9 backports are done, closing again.

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 v7.9.0
Projects
None yet
Development

No branches or pull requests

3 participants