dencoder: update ceph-object-corpus submodule for 18.2.0 coverage

[cbodley]

pull in submodule changes from ceph/ceph-object-corpus#17

Show available Jenkins commands

• jenkins retest this please
• jenkins test classic perf
• jenkins test crimson perf
• jenkins test signed
• jenkins test make check
• jenkins test make check arm64
• jenkins test submodules
• jenkins test dashboard
• jenkins test dashboard cephadm
• jenkins test api
• jenkins test docs
• jenkins render docs
• jenkins test ceph-volume all
• jenkins test ceph-volume tox
• jenkins test windows
• jenkins test rook e2e

[cbodley]

The following tests FAILED:
147 - readable.sh (Failed)

**** reencode of /home/jenkins-build/build/workspace/ceph-pull-requests/ceph-object-corpus/archive/18.2.0/objects/chunk_refs_t/ccb69d9ecd572c1f6ed9598899773cf1 resulted in a different dump ****

[NitzanMordhai]

@cbodley let me check it, chunk_refs_t changed to be non deterministic, it could be that we will need first merge the common changes for dencoder first

[github-actions]

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

[github-actions]

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

[cbodley]

hey @NitzanMordhai, what more do we need to do to validate this? does ceph/ceph-object-corpus#17 need a rebase in order to pass 'make check' here?

[NitzanMordhai]

hey @NitzanMordhai, what more do we need to do to validate this? does ceph/ceph-object-corpus#17 need a rebase in order to pass 'make check' here?

#52210 was merge, we can rebase that one and the make check should resolve

[cbodley]

now points the submodule at the rebased branch ceph/ceph-object-corpus#17

[cbodley]

289/290 Test #148: readable.sh ...............................***Failed 2621.41 sec

/tmp/test_object_1-3bwS7gIZl /tmp/test_object_2-cFY7UCrpP differ: byte 133, line 8
**** reencode of /home/jenkins-build/build/workspace/ceph-pull-requests/ceph-object-corpus/archive/18.2.0/objects/chunk_refs_t/ccb69d9ecd572c1f6ed9598899773cf1 resulted in a different dump ****
8c8
<             "hash": 684235776,
---
>             "hash": 1703915520,
13c13
<             "hash": 684235777,
---
>             "hash": 1703915521,

that looks like the same chunk_refs_t failure identified above

[github-actions]

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

[NitzanMordhai]

jenkins test make check

[NitzanMordhai]

jenkins test make check

[NitzanMordhai]

@cbodley we have an issue with chunk_refs on clang, the value for hash wasn't initilized, and thats the reason it happened. I was able to recreate the error.

[NitzanMordhai]

wait to #56136

[rzarzynski]

Asked Laura & Yuri to prioritize the dependency.

[cbodley]

wait to #56136

that merged, so i've pushed a fresh rebase

[cbodley]

291/293 Test #148: readable.sh ............................... Passed 1215.70 sec

👍

[cbodley]

@neha-ojha @rzarzynski this is ready for review

[cbodley]

thanks @NitzanMordhai, thanks @rzarzynski!