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

tests: test/librados/test.cc must create profile #6452

Merged
1 commit merged into from Nov 3, 2015

Conversation

Projects
None yet
3 participants
@ghost
Copy link

ghost commented Nov 2, 2015

Now that the create_one_ec_pool function removes the testprofile each
time it is called, it must create the testprofile erasure code profile
again for the test to use.

http://tracker.ceph.com/issues/13664 Refs: #13664

Signed-off-by: Loic Dachary ldachary@redhat.com

tests: test/librados/test.cc must create profile
Now that the create_one_ec_pool function removes the testprofile each
time it is called, it must create the testprofile erasure code profile
again for the test to use.

http://tracker.ceph.com/issues/13664 Refs: #13664

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

@ghost ghost added bug fix tests labels Nov 2, 2015

@ghost ghost assigned athanatos Nov 2, 2015

@ghost

This comment has been minimized.

Copy link
Author

ghost commented Nov 2, 2015

@ghost

This comment has been minimized.

Copy link
Author

ghost commented Nov 3, 2015

bash -ex ~/software/ceph/ceph/qa/workunits/rados/test-upgrade-v9.0.1.sh (from firefly) ends with

******* SUCCESS **********
+ exit 0

ghost pushed a commit that referenced this pull request Nov 3, 2015

Loic Dachary
Merge pull request #6452 from dachary/wip-13664-test-ec
tests: test/librados/test.cc must create profile

Reviewed-by: Samuel Just <sjust@redhat.com>

@ghost ghost merged commit 76de0c3 into ceph:master Nov 3, 2015

This issue was closed.

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.