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: testGetContentType and testHead failed #5718
Conversation
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com> Conflicts: src/rgw/rgw_rest.cc
Reviewed-by: Loic Dachary <ldachary@redhat.com>
It fails consistently with: filter="rgw/multifs/{overrides.yaml clusters/fixed-2.yaml frontend/civetweb.yaml fs/xfs.yaml rgw_pool_type/replicated.yaml tasks/rgw_swift.yaml}" ./virtualenv/bin/teuthology-suite --priority 101 --suite rgw --filter="$filter" --suite-branch hammer --distro ubuntu --email loic@dachary.org --ceph hammer-backports-loic --machine-type plana,burnupi,mira where hammer-backports-loic is a a branch with hammer and only this pull request on top of it. Here is an example of a failure : http://pulpito.ceph.com/loic-2015-09-01_00:12:35-rgw-hammer-backports-loic---basic-multi. The same job run with the hammer suite succeeds: this is not an environmental problem. |
@oritwas I'm afraid I'm going to need your expertise on this one. I don't know enough of rgw to figure out why it fails. |
This is strange. I will look at it |
@oritwas thanks ! Note that it's not blocking anything. I got confused at first because there were many pull request in the same branch. Now that it's isolated and easy to reproduce, we'll just set it aside. |
@dachary , it apears we missed two commits, see https://github.com/ceph/ceph/tree/wip-swift-backports-hammer |
@oritwas picked the commits, many thanks ! I also created http://tracker.ceph.com/issues/12918 to link 4b7dbf6 to its original issue http://tracker.ceph.com/issues/12158. I presume 4c5ac21 is a commit that belongs to http://tracker.ceph.com/issues/12157#note-7. |
Swift sends Content-Type HTTP header even if the response doesn't contain body. We have this behaviour implemented until applying some changes in end_header() function. Unfortunately, lack of Content-Type causes early exits in many Tempest's tests for Swift API verification. Fixes: #12157 Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com> (cherry picked from commit 106aeba)
Fixes: 12095 Signed-off-by: Orit Wasserman <owasserm@redhat.com> (cherry picked from commit da259e4)
filter="rgw/multifs/{overrides.yaml clusters/fixed-2.yaml frontend/civetweb.yaml fs/xfs.yaml rgw_pool_type/replicated.yaml tasks/rgw_swift.yaml}" ./virtualenv/bin/teuthology-suite --priority 101 --suite rgw --filter="$filter" --suite-branch hammer --distro ubuntu --email loic@dachary.org --ceph wip-12859-hammer-loic --machine-type plana,burnupi,mira |
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@yehudasa does this backport look good to merge ? It passed a run of the hammer rgw suite ( see http://tracker.ceph.com/issues/13356#note-5 for details ). |
Reviewed-by: Loic Dachary <ldachary@redhat.com>
testGetContentType and testHead failed
http://tracker.ceph.com/issues/12246
http://tracker.ceph.com/issues/12632
http://tracker.ceph.com/issues/12634
http://tracker.ceph.com/issues/12859
http://tracker.ceph.com/issues/12918