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

build/ops: add sanity checks to run-make-check.sh #12683

Merged
merged 1 commit into from Jan 12, 2017

Conversation

smithfarm
Copy link
Contributor

Signed-off-by: Nathan Cutler ncutler@suse.com

Signed-off-by: Nathan Cutler <ncutler@suse.com>
Copy link
Contributor

@badone badone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK to me

@smithfarm
Copy link
Contributor Author

smithfarm commented Jan 12, 2017

Testing manually in Xenial docker container without sudo package:

$ bash run-make-check.sh 
Checking hostname sanity... OK
Please install sudo and re-run. This script assumes it is running
as a normal user with the ability to run commands as root via sudo.

(Running as root, same thing happens.)

After installing sudo, but not configuring it:

$ bash run-make-check.sh 
Checking hostname sanity... OK
[sudo] password for smithfarm:

As root:

# bash run-make-check.sh
For best results, run this script as a normal user configured
with the ability to run commands as root via sudo.
Checking hostname sanity... OK
[execution continues]

After configuring sudo, execution proceeds as usual.

Next test - CentOS 7.3

@smithfarm
Copy link
Contributor Author

Tested manually in CentOS 7.2.1511 docker container:

As normal user in minimal environment:

$ bash run-make-check.sh
Checking hostname sanity... OK
Please install bc and re-run.

As root, same behavior.

After installing bc, behavior is the same as for Xenial, above. It complains about sudo, etc.

@smithfarm smithfarm merged commit 8c724a5 into ceph:master Jan 12, 2017
@smithfarm smithfarm deleted the wip-run-make-check-deps branch January 12, 2017 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants