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.

Sign up for GitHub

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

Jump to bottom

luminous: rgw: wrong error message is returned when putting container with a name that is too long #17811

Merged
merged 9 commits into from Oct 15, 2017
Merged

luminous: rgw: wrong error message is returned when putting container with a name that is too long #17811

merged 9 commits into from Oct 15, 2017

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm commented Sep 19, 2017 •
edited

rzarzynski and others added 9 commits September 19, 2017 18:46
@rzarzynski @smithfarm
rgw: add support for max_meta_name_length of Swift API's /info.
4b82d1e 
Fixes: http://tracker.ceph.com/issues/17938
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 593d656)
@rzarzynski @smithfarm
rgw: refactor rgw_get_request_metadata to reduce the number of dynall…
cd21915 
…ocs.

Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 64b7923)
@rzarzynski @smithfarm
rgw: add support for max_meta_value_length of Swift API's /info.
912132b 
Fixes: http://tracker.ceph.com/issues/17936
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 06b1f25)
@rzarzynski @smithfarm
rgw: honor custom rgw_err::message in Swift's error handling.
02c208a 
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit e851614)
@rzarzynski @smithfarm
rgw: Swift API returns 400 Bad Request on too long container names.
70e150c 
Fixes: http://tracker.ceph.com/issues/17935
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 1845e41)
@rzarzynski @smithfarm
rgw: add support for max_meta_count of Swift API's /info.
429f5e1 
Fixes: http://tracker.ceph.com/issues/17934
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 9e53c14)
@rzarzynski @smithfarm
rgw: seed::get_torrent_file returns errors in the usual way.
71e37b1 
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 21ad80a)
@rzarzynski @smithfarm
rgw: return proper message when deleting non-empty Swift's container.
6080248 
The bug that is fixed in this patch has been responsible for failing
the Tempest's test_delete_non_empty_container test case.

The investigation has been made by: Marcus Watts <mwatts@redhat.com>.

Fixes: http://tracker.ceph.com/issues/21169
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
(cherry picked from commit 308c8d3)
@rzarzynski @smithfarm
rgw: rename the configurables for metadata limits to start with rgw_.
119995a 
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
(cherry picked from commit 9b06985)
@smithfarm smithfarm self-assigned this Sep 19, 2017
@smithfarm smithfarm added this to the luminous milestone Sep 19, 2017
@rzarzynski rzarzynski mentioned this pull request Sep 20, 2017
Closed
@rzarzynski
Copy link
Contributor

jenkins retest this please (Error fetching remote repo 'origin')

@rzarzynski
Copy link
Contributor

Teuthology run: http://pulpito.ceph.com/rzarzynski-2017-09-20_14:57:33-rgw-wip-swift-testing-luminous---basic-smithi.

mdw-at-linuxbox
mdw-at-linuxbox approved these changes Oct 14, 2017
View reviewed changes
Copy link
Contributor

@mdw-at-linuxbox mdw-at-linuxbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only failed on centos - probably 7.3/7.4 or other unrelated problem. And this fix works on master and luminous isn't that far off. Let's merge this before the code drifts too much!

@mattbenjamin mattbenjamin merged commit 462b93e into ceph:luminous Oct 15, 2017
@smithfarm smithfarm deleted the wip-21456-luminous branch October 15, 2017 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdw-at-linuxbox mdw-at-linuxbox mdw-at-linuxbox approved these changes

@mattbenjamin mattbenjamin Awaiting requested review from mattbenjamin

Assignees

@smithfarm smithfarm

Labels
bug-fix rgw
Projects
None yet
Milestone
luminous
4 participants
@smithfarm @rzarzynski @mdw-at-linuxbox @mattbenjamin