]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
test/librbd: make ListSnapsWholeObject actually test stuff
authorIlya Dryomov <idryomov@gmail.com>
Sun, 19 Nov 2023 21:44:28 +0000 (22:44 +0100)
committerIlya Dryomov <idryomov@gmail.com>
Sat, 2 Dec 2023 12:50:04 +0000 (13:50 +0100)
commitbd52297a71100dd35874fa3aeda81fca1d828b38
tree80238f827bbfa2297a92cc7cb7bb7f23b2304a35
parentc0747922040841c371dbe2706354c08c73d8e59f
test/librbd: make ListSnapsWholeObject actually test stuff

Despite being added in commit 66dd53d9c4d9 ("librbd: optionally return
full object extent for any snapshot deltas") ostensibly to test the new
LIST_SNAPS_FLAG_WHOLE_OBJECT code, it surely doesn't do that because
the flag isn't even passed to MockObjectListSnapsRequest::create().

I can only guess, but it looks like snap ID 3 was intended to be
a starting point.  Otherwise, with 0 and CEPH_NOSNAP passed as snap
IDs, the overlap that is set up for the clone wouldn't affect the
computation in any way.

Use snap ID 3 as a starting point and run both with and without
LIST_SNAPS_FLAG_WHOLE_OBJECT on the same snapset to pinpoint the
difference.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
src/test/librbd/io/test_mock_ObjectRequest.cc