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: destroy testprofile before creating one #6446

Merged
2 commits merged into from Nov 2, 2015

Conversation

Projects
None yet
3 participants
@ghost
Copy link

ghost commented Nov 2, 2015

No description provided.

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

ldachary added some commits Nov 2, 2015

tests: add destroy_ec_profile{,_pp} helpers
Signed-off-by: Loic Dachary <ldachary@redhat.com>
tests: destroy testprofile before creating one
The testprofile erasure code profile is destroyed before creating a new
one so that it does not fail when another testprofile erasure code
profile already exists with different parameters.

This must be done when creating erasure coded pools with the C++
interface, in the same way it's done with the C interface.

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

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

This comment has been minimized.

Copy link
Contributor

athanatos commented Nov 2, 2015

lgtm

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

Loic Dachary
Merge pull request #6446 from dachary/wip-13664-test-ec
tests: destroy testprofile before creating one

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

@ghost ghost merged commit 7f8bf7a into ceph:master Nov 2, 2015

@ghost

This comment has been minimized.

Copy link
Author

ghost commented Nov 2, 2015

tested locally with qa/workunits/rados/test-upgrade-v9.0.1.sh from the firefly branch

@ghost ghost assigned athanatos Nov 2, 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.