Skip to content
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

jewel: rgw: json encode/decode of RGWBucketInfo missing index_type field #11759

Merged
merged 1 commit into from Jan 31, 2017

Conversation

Abhishekvrshny
Copy link

Fixes: http://tracker.ceph.com/issues/17755

We didn't dump the index_type field of RGWBucketInfo, also didn't
allow modifying it via metadata api.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit e8f26c0)
@Abhishekvrshny Abhishekvrshny added this to the jewel milestone Nov 3, 2016
@Abhishekvrshny Abhishekvrshny self-assigned this Nov 3, 2016
@ghost ghost changed the base branch from jewel to jewel-next November 9, 2016 09:55
ghost pushed a commit that referenced this pull request Nov 9, 2016
…tInfo missing index_type field

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned mattbenjamin Nov 11, 2016
@ghost
Copy link

ghost commented Nov 11, 2016

@mattbenjamin does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/17851#note-3 . Note that it will not be in 10.2.4, reason why it targets jewel-next.

@ghost
Copy link

ghost commented Nov 14, 2016

jenkins test this please (hung on unittest_throttle)

@ghost
Copy link

ghost commented Nov 23, 2016

@mattbenjamin ping

ghost pushed a commit that referenced this pull request Nov 23, 2016
…tInfo missing index_type field

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Dec 5, 2016
…tInfo missing index_type field

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Dec 5, 2016

@mattbenjamin ping

@ghost ghost changed the base branch from jewel-next to jewel December 21, 2016 23:31
@mattbenjamin mattbenjamin merged commit c4f8684 into ceph:jewel Jan 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants