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: add separate ceph-helpers-based smoke test #16572

Merged
merged 3 commits into from Jul 27, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Copy link
Member

liewegas commented Jul 25, 2017

No description provided.

@liewegas liewegas added this to the luminous milestone Jul 25, 2017

@liewegas liewegas requested a review from dzafman Jul 25, 2017

@liewegas liewegas force-pushed the liewegas:wip-pidfile branch from 3d9393f to 4b71134 Jul 25, 2017

@tchaikov tchaikov self-requested a review Jul 26, 2017

@dzafman
Copy link
Member

dzafman left a comment

See my alternative fix for test_pidfile in #16587

Running test in a loop overnight.

@liewegas liewegas force-pushed the liewegas:wip-pidfile branch from 4b71134 to db3ceca Jul 26, 2017

@liewegas liewegas changed the title test: make test_pidfile more reliable; add seperate ceph-helpers-based smoke test test: add seperate ceph-helpers-based smoke test Jul 26, 2017


local funcs=${@:-$(set | sed -n -e 's/^\(TEST_[0-9a-z_]*\) .*/\1/p')}
for func in $funcs ; do
setup $dir || return 1

This comment has been minimized.

Copy link
@tchaikov

tchaikov Jul 26, 2017

Contributor

set -e to avoid repeating || return 1

@liewegas liewegas changed the title test: add seperate ceph-helpers-based smoke test test: add separate ceph-helpers-based smoke test Jul 26, 2017

@liewegas

This comment has been minimized.

Copy link
Member Author

liewegas commented Jul 26, 2017

retest this please

@liewegas liewegas force-pushed the liewegas:wip-pidfile branch from db3ceca to 6be179a Jul 26, 2017

@liewegas

This comment has been minimized.

Copy link
Member Author

liewegas commented Jul 27, 2017

retest this please

liewegas added some commits Jul 26, 2017

test/smoke.sh: add simple ceph-helpers-based smoke test
This is just enough during make check to know things aren't totally
broken.

Signed-off-by: Sage Weil <sage@redhat.com>
qa/standalone/README
Signed-off-by: Sage Weil <sage@redhat.com>
qa/standalone/crush/crush-classes: fix test
Signed-off-by: Sage Weil <sage@redhat.com>

@liewegas liewegas force-pushed the liewegas:wip-pidfile branch from 0d86ec3 to e469a80 Jul 27, 2017

@liewegas liewegas merged commit 541de39 into ceph:master Jul 27, 2017

3 of 4 checks passed

make check (arm64) running make check
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

@liewegas liewegas deleted the liewegas:wip-pidfile branch Jul 27, 2017

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.