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

test: Replace bc command with printf command #21013

Merged
merged 1 commit into from Mar 23, 2018

Conversation

Projects
None yet
2 participants
@dzafman
Copy link
Member

commented Mar 23, 2018

The bc command is not part of install-deps.sh and printf command works even better.

test: Replace bc command with printf command
Signed-off-by: David Zafman <dzafman@redhat.com>
@dzafman

This comment has been minimized.

Copy link
Member Author

commented Mar 23, 2018

Testing passed by on luminous which has the same teuthology configuration:

http://pulpito.ceph.com/dzafman-2018-03-22_17:32:01-rados:standalone-wip-zafman-testing3-distro-basic-smithi/

@dzafman

This comment has been minimized.

Copy link
Member Author

commented Mar 23, 2018

@smithfarm This should merge because the backports refer to the sha1 of the commit here.

@smithfarm smithfarm merged commit f1936a2 into ceph:master Mar 23, 2018

4 of 5 checks passed

make check (arm64) make check failed
Details
Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@dzafman dzafman deleted the dzafman:wip-remove-bc branch Mar 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.