]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw/rgw_dencoder: set identity_type when creating samples
authorKefu Chai <kchai@redhat.com>
Mon, 16 Aug 2021 08:01:42 +0000 (16:01 +0800)
committerKefu Chai <tchaikov@gmail.com>
Thu, 1 Sep 2022 17:08:25 +0000 (01:08 +0800)
otherwise identity_type is a random number on stack. and there is good
chance that it does not match with the one decoded from the encoded
blob.

this change should address the failures like:

**** rgw_log_entry test 2 binary reencode check failed ****
   ceph-dencoder type rgw_log_entry select_test 2 encode export /tmp/typ-O8hVYK2Gb
   ceph-dencoder type rgw_log_entry select_test 2 encode decode encode export /tmp/typ-G09itHHIC
   cmp /tmp/typ-O8hVYK2Gb /tmp/typ-G09itHHIC

The following tests FAILED:
132 - check-generated.sh (Failed)
..

Fixes: https://tracker.ceph.com/issues/52278
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 22095f4bd831f0d736ba364ea9e46febe06576f9)

src/rgw/rgw_dencoder.cc

index 3835b7776a8885f49a368333be4228f159e6482d..10fc0eb303a26ead2d4f15c6b8c0f679f5170ec7 100644 (file)
@@ -80,6 +80,7 @@ void rgw_log_entry::generate_test_instances(list<rgw_log_entry*>& o)
   e->referrer = "referrer";
   e->bucket_id = "10";
   e->trans_id = "trans_id";
+  e->identity_type = TYPE_RGW;
   o.push_back(e);
   o.push_back(new rgw_log_entry);
 }