]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
cls/rbd: use default values for non-decoded fields in test instances 63933/head
authorKefu Chai <tchaikov@gmail.com>
Sat, 14 Jun 2025 13:44:05 +0000 (21:44 +0800)
committerKefu Chai <tchaikov@gmail.com>
Tue, 17 Jun 2025 14:39:26 +0000 (22:39 +0800)
commita329b00579546ad2c069d792bde34cd44cfa0b43
tree5f8aea92dba36f2d09dd14ca02ce19bdfaaf132a
parent95ca3c553ee0353d6ddc7b4c3f11c14830ea1207
cls/rbd: use default values for non-decoded fields in test instances

Previously, test instances for cls_rbd_snap used non-default values
for the "parent" field, which is ignored during decoding. The
check-generated.sh test passed because they reused the same instance
for re-encoding, preserving undecoded fields.

An upcoming change will allocate new instances for each encode/decode
verification instead of reusing instances. This will expose
discrepancies between original test instances and re-encoded values
when fields contain non-default values but aren't decoded.

This change sets ignored fields to their default values in test
instances, ensuring consistency between encoding and decoding
operations regardless of the verification approach used.

Since the incompatibility of cls_rbd_snap's on-disk format was
introduced in 32b14ed1, which was introduced Ceph v14, we will
mark this version the first incompatible version in ceph-object-corpus
in the sense that the re-encoded cls_rbd_snap with v8 struct version
is different from the original copy if its parent field is set with
< v8 struct version.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
src/cls/rbd/cls_rbd.h