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

pacific: librbd: localize snap_remove op for mirror snapshots #51431

Merged
merged 1 commit into from Jun 16, 2023

Conversation

chrisphoffman
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/59710


backport of #51166
parent tracker: https://tracker.ceph.com/issues/59393

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

@chrisphoffman chrisphoffman requested a review from a team as a code owner May 10, 2023 23:56
@chrisphoffman chrisphoffman added this to the pacific milestone May 10, 2023
@github-actions github-actions bot added the tests label May 10, 2023
@chrisphoffman
Copy link
Contributor Author

chrisphoffman commented May 11, 2023

To be rebased: includes #51429 to avoid further conflicts.

@idryomov
Copy link
Contributor

To be rebased: includes #51429 to avoid further conflicts.

I would make this really obvious by basing directly on wip-59506-pacific branch -- so that SHA1s are same.

@yuriw
Copy link
Contributor

yuriw commented Jun 15, 2023

@chrisphoffman This was tested and ready for merge

skipping merge because of DNM

ref: https://trello.com/c/MTsDz3Xr

A client may attempt a lock request not quickly enough to
obtain exclusive lock for operations when another competing
client responds quicker. This can happen when a peer site has
different performance characteristics or latency. Instead of
relying on this unpredictable behavior, localize operation to
primary cluster.

Fixes: https://tracker.ceph.com/issues/59393
Signed-off-by: Christopher Hoffman <choffman@redhat.com>
(cherry picked from commit ac552c9)

Conflicts:
	src/cls/rbd/cls_rbd.cc [ commit 3a93b40 ("librbd:
	  s/boost::variant/std::variant/") not in pacific ]
	src/librbd/mirror/snapshot/UnlinkPeerRequest.cc [ ditto ]
@idryomov
Copy link
Contributor

To be rebased: includes #51429 to avoid further conflicts.

Now done.

@idryomov idryomov removed the DNM label Jun 16, 2023
@idryomov idryomov merged commit bda07a6 into ceph:pacific Jun 16, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants