-
Notifications
You must be signed in to change notification settings - Fork 6k
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
rgw: Ensure buckets too old to decode a layout have layout logs #43778
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
even better than last time! thanks
168e544
to
3db5dc2
Compare
Jenkins, retest this please. |
@cbodley I think this looks reasonable, the failures appear to be pubsub related. https://pulpito.ceph.com/aemerson-2021-11-02_23:30:11-rgw-wip-53132-distro-basic-smithi/ |
jenkins test make check |
looks great @adamemerson, this can merge once jenkins is happy |
Jenkins test make check |
dangit jenkins! |
jenkins test make check |
When decoding `RGWBucketInfo` data from before Pacific, we won't call `rgw::BucketLayout::decode`, but will instead synthesize the layout information. This leaves the `rgw::BucketLayout::logs` empty, as the fallback to populate it only applies to old versions of `rgw::BucketLayout`. Add a check at the end of `RGWBUcketInfo::decode` to populate it if empty. Fixes: https://tracker.ceph.com/issues/53132 Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
3db5dc2
to
3279509
Compare
Jenkins run api test |
jenkins test api |
1 similar comment
jenkins test api |
When decoding
RGWBucketInfo
data from before Pacific, we won't callrgw::BucketLayout::decode
, but will instead synthesize the layoutinformation. This leaves the
rgw::BucketLayout::logs
empty, as thefallback to populate it only applies to old versions of
rgw::BucketLayout
.Add a check at the end of
RGWBUcketInfo::decode
to populate it ifempty.
Checklist