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
qa/workunits/rbd: fix #12511
qa/workunits/rbd: fix #12511
Conversation
Signed-off-by: Sage Weil <sage@redhat.com>
Root of git checkout Signed-off-by: Sage Weil <sage@redhat.com>
e39935d
to
fc3f25e
Compare
fc3f25e
to
670a517
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
else | ||
. $(dirname $0)/rbd_mirror_helpers.sh | ||
fi | ||
. ./rbd_mirror_helpers.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@liewegas I suppose it wont work when running this locally from cmake build dir, i.e.:
../qa/workunits/rbd/rbd_mirror.sh
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh right. i'll do dirname unconditionally
670a517
to
93620d0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
93620d0
to
9409e85
Compare
Signed-off-by: Sage Weil <sage@redhat.com>
9409e85
to
cf29477
Compare
@liewegas Are we going to try to backport these changes to jewel? |
@smithfarm I think this will need to get backported to all active "testing" branches (including possibly some EoLed releases due to upgrade tests). |
Yep! I was just going to make sure they passed first. But either way
we'll backport these (+ fixes) so go ahead!
|
jewel and hammer
|
OK, created the master and backport tracker issues. |
http://tracker.ceph.com/issues/18271