Ceph » rbd

NOTE: this will require a second commit to address since the first commit did not fully resolve the issue.

http://qa-proxy.ceph.com/teuthology/jdillaman-2020-04-14_08:24:43-rbd-wip-jd-testing-2-distro-basic-smithi/4953440/teuthology.log

Additional PR: https://github.com/ceph/ceph/pull/34753

It should fix failures like this one: http://qa-proxy.ceph.com/teuthology/trociny-2020-04-24_15:23:36-rbd-wip-mgolub-testing-distro-basic-smithi/4981088/teuthology.log

@Jason Borden The octopus backport PR for this issue was already merged. Is it missing something?

Yes, see comment #7. It has an additional fix that we should backport. I'm just re-using these tracker tickets instead of opening new ones.

Jason Dillaman wrote:

Yes, see comment #7. It has an additional fix that we should backport. I'm just re-using these tracker tickets instead of opening new ones.

I believe both commits are now in octopus:

commit f93516d78734740c30c5532e5032be1893004bb1
Author: Mykola Golub <mgolub@suse.com>
Date:   Sat Apr 25 08:36:25 2020 +0100

    qa/workunits/rbd: retry the addition of a mirror pool peer

    fb4311f5 has fixed this for setup, but "remove mirroring pool" 
    test needs fixing too.

    Fixes: https://tracker.ceph.com/issues/44938
    Signed-off-by: Mykola Golub <mgolub@suse.com>
    (cherry picked from commit 7eced158a9a3c47cc408b35219b4428e97e018fb)

commit 4644cd663de27bd19b07eb8dca0153032060694b
Author: Jason Dillaman <dillaman@redhat.com>
Date:   Wed Apr 15 16:27:07 2020 -0400

    qa/workunits/rbd: retry the addition of a mirror pool peer

    We might race with the remote rbd-mirror daemon creating a
    tx-only peer when adding a new peer. Therefore, delete the
    tx-only peer and attempt to re-create it.

    Fixes: https://tracker.ceph.com/issues/44938
    Signed-off-by: Jason Dillaman <dillaman@redhat.com>
    (cherry picked from commit fb4311f597a98b6870d7895e6403fb32356bfbe9)