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

luminous: cephfs: ceph_volume_client: allow atomic update of RADOS objects #24084

Merged
merged 2 commits into from Sep 27, 2018

Conversation

Projects
None yet
5 participants
@batrick
Member

batrick commented Sep 13, 2018

rishabh-d-dave added some commits Jun 7, 2018

ceph-volume-client: allow atomic updates for RADOS objects
put_object_versioned() takes the version of the object and verifies if
the version of the object is the expected one before updating the data
in the object. This verification of version before actually writing
makes put_objcet_version() atomic.

Rest of the changes include adding get_object_and_version() so that
current version of the object can be obtained and modification of
get_object() and put_object() to use get_object_and_version() and
put_object_versioned() respectively.

Fixes: http://tracker.ceph.com/issues/24173
Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit ca7253c)
qa/ceph-volume: add a test for put_object_versioned()
Test if the version passed to put_object_versioned() is used to
crosscheck.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 8ab6f84)

@batrick batrick added the cephfs label Sep 13, 2018

@batrick batrick added this to the luminous milestone Sep 13, 2018

@yuriw

This comment has been minimized.

Contributor

yuriw commented Sep 26, 2018

@batrick

This comment has been minimized.

Member

batrick commented Sep 27, 2018

Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>

@yuriw yuriw merged commit f3e0ffc into ceph:luminous Sep 27, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@batrick batrick deleted the batrick:i24862 branch Sep 27, 2018

@smithfarm smithfarm changed the title from luminous: ceph_volume_client: allow atomic update of RADOS objects to luminous: cephfs: ceph_volume_client: allow atomic update of RADOS objects Oct 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment