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

ceph-disk/test: fix test_prepare.py::TestPrepare tests #7549

Merged
merged 1 commit into from Feb 6, 2016

Conversation

tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Feb 6, 2016

Signed-off-by: Kefu Chai kchai@redhat.com

actually we are verifying if the variable is an instance of
specified class. for example, the `prepare.data` should be
a `PrepareFilestoreData` if `--bluestore` is not specified.

Signed-off-by: Kefu Chai <kchai@redhat.com>
@ghost
Copy link

ghost commented Feb 6, 2016

Reviewed-by: Loic Dachary <ldachary@redhat.com>

@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 6, 2016

the failure is

tests/ceph-disk.sh:136: command_fixture:  '[' /home/jenkins-build/build/workspace/ceph-pull-requests/src/ceph-conf = /home/jenkins-build/build/workspace/ceph-pull-requests/src/ceph-disk/ceph-conf ']'

it is spotted in my local env, but i thought it was due to the way i started the test. but seem it's not. will look into it later.

tchaikov added a commit that referenced this pull request Feb 6, 2016
ceph-disk/test: fix test_prepare.py::TestPrepare tests

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@tchaikov tchaikov merged commit 7f854cc into ceph:master Feb 6, 2016
@tchaikov tchaikov deleted the wip-fix-ceph-disk-tests branch February 6, 2016 15:43
@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 6, 2016

thanks for your quick review @dachary =)

@tchaikov
Copy link
Contributor Author

tchaikov commented Feb 7, 2016

@dachary fixed in #7554. and happy lunar new year!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant