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_rest_s3: apply missed base64 try-catch #11672

Merged
merged 1 commit into from Dec 5, 2016

Conversation

mattbenjamin
Copy link
Contributor

@mattbenjamin mattbenjamin commented Oct 27, 2016

Fixes a case missed in 0a4c91e, in Jewel backport only.

Fixes: http://tracker.ceph.com/issues/17663
(cherry picked from commit 87ed19e)

Resolves: rhbz#1381694

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
@ghost ghost changed the base branch from jewel to jewel-next November 9, 2016 09:57
@ghost
Copy link

ghost commented Nov 9, 2016

jenkins test this please (no logs)

@ghost
Copy link

ghost commented Nov 9, 2016

jenkins test this please (general jenkins failure)

ghost pushed a commit that referenced this pull request Nov 9, 2016
…y-catch

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

ghost commented Nov 11, 2016

@mattbenjamin could you please confirm that #11757 is unneeded and that this pull request fixes http://tracker.ceph.com/issues/17663 ?

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 ghost assigned mattbenjamin Nov 11, 2016
@ghost
Copy link

ghost commented Nov 23, 2016

@mattbenjamin ping

ghost pushed a commit that referenced this pull request Nov 23, 2016
…y-catch

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

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

ghost commented Dec 5, 2016

@mattbenjamin ping

@yehudasa yehudasa merged commit 13d68c9 into ceph:jewel-next Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants