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.

Sign up for GitHub

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

Jump to bottom

Password required for sudo use in benchmark install_dependencies.sh #97

Closed
fbbradheintz opened this issue Mar 12, 2020 · 2 comments
Closed

Password required for sudo use in benchmark install_dependencies.sh #97

fbbradheintz opened this issue Mar 12, 2020 · 2 comments
Assignees
@harshbafna
Labels
close_after_merge documentation Improvements or additions to documentation launch blocker staged_release merged to stage_release

Comments

@fbbradheintz
Copy link
Contributor

fbbradheintz commented Mar 12, 2020 •
edited
Loading

This was on DL AMI 27 Ubuntu 18 c4.xlarge.

sudo must be passworded for the user running install_dependencies.sh to complete successfully.

Currently, many default Ubuntu configurations on EC2 have the default ubuntu user able to do passwordless sudo - meaning that unless they add a password to that account, install_dependencies.sh can't complete.

sudo passwd <username> with a non-empty password was the remedy for me.
@fbbradheintz fbbradheintz added documentation Improvements or additions to documentation launch blocker labels Mar 12, 2020
@harshbafna
Copy link
Contributor

@fbbradheintz

While installing JMeter through brew, the install_depdendencies.sh script asks for following command line input.

Installing JMeter through Brew
+ yes ''
+ brew update
==> Select the Linuxbrew installation directory
- Enter your password to install to /home/linuxbrew/.linuxbrew (recommended)
- Press Control-D to install to /home/ubuntu/.linuxbrew
- Press Control-C to cancel installation
[sudo] password for ubuntu:

Here Press Control-D to install to /home/ubuntu/.linuxbrew as the ubuntu user on EC2 node has password-less sudo access.

We have updated the documentation in following commit in the stage_release branch : 087f6ed

@harshbafna harshbafna added staged_release merged to stage_release triaged_wait Waiting for the Reporter's resp labels Mar 13, 2020
@harshbafna harshbafna mentioned this issue Mar 13, 2020
Closed
@fbbradheintz fbbradheintz added close_after_merge and removed triaged_wait Waiting for the Reporter's resp labels Mar 16, 2020
@fbbradheintz
Copy link
Contributor Author

Looks good. Please close after merge.

@harshbafna harshbafna self-assigned this Mar 17, 2020
@harshbafna harshbafna mentioned this issue Mar 18, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@harshbafna harshbafna

Labels
close_after_merge documentation Improvements or additions to documentation launch blocker staged_release merged to stage_release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@harshbafna @fbbradheintz