Project

General

Profile

Actions
Copy link

Bug #10334

closed

Bug #10268: s3tests.functional.test_s3.test_bucket_create_exists fails with 'S3CreateError not raised'

rgw: s3 need to return InvalidAccessKeyId instead of AccessDenied

Added by Yehuda Sadeh over 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Yehuda Sadeh
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

commit:56af795b1046a4c1bfba59e1fefde272bb0e5c1e

======================================================================
FAIL: s3tests.functional.test_s3.test_list_buckets_invalid_auth
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/yehudasa/s3-tests/virtualenv/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/yehudasa/s3-tests/s3tests/functional/test_s3.py", line 4078, in test_list_buckets_invalid_auth
    eq(e.error_code, 'InvalidAccessKeyId')
AssertionError: u'AccessDenied' != 'InvalidAccessKeyId'

Actions
Copy link
 #1

Updated by Yehuda Sadeh over 9 years ago

  • Subject changed from rgw: s3 need to return SignatureDoesNotMatch instead of AccessDenied to rgw: s3 need to return InvalidAccessKeyId instead of AccessDenied
Actions
Copy link
 #2

Updated by Josh Durgin over 9 years ago

  • Status changed from New to Pending Backport
  • Backport set to giant, firefly

commit:56af795b1046a4c1bfba59e1fefde272bb0e5c1e

Actions
Copy link
 #3

Updated by Loïc Dachary over 9 years ago

  • Description updated (diff)
Actions
Copy link
 #4

Updated by Loïc Dachary about 9 years ago

There seemed to be a harmless spurious backport in master

backports@workbench:~/ceph-backports$ git --no-pager log d3755324364d50706c840cab0868b9537f002ca1^!
commit d3755324364d50706c840cab0868b9537f002ca1
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Tue Dec 16 12:27:54 2014 -0800

    rgw: return InvalidAccessKeyId instead of AccessDenied

    Fixes: #10334

    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
    (cherry picked from commit 56af795b1046a4c1bfba59e1fefde272bb0e5c1e)
backports@workbench:~/ceph-backports$ 
backports@workbench:~/ceph-backports$ git --no-pager log '56af795b1046a4c1bfba59e1fefde272bb0e5c1e^!'
commit 56af795b1046a4c1bfba59e1fefde272bb0e5c1e
Author: Yehuda Sadeh <yehuda@redhat.com>
Date:   Tue Dec 16 12:27:54 2014 -0800

    rgw: return InvalidAccessKeyId instead of AccessDenied

    Fixes: #10334

    Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
backports@workbench:~/ceph-backports$ git describe 56af795b1046a4c1bfba59e1fefde272bb0e5c1e
v0.89-542-g56af795
backports@workbench:~/ceph-backports$ git describe d3755324364d50706c840cab0868b9537f002ca1
v0.90-605-gd375532

Actions
Copy link
 #5

Updated by Loïc Dachary about 9 years ago

  • Status changed from Pending Backport to Resolved
  • Backport deleted (giant, firefly)

similar to http://tracker.ceph.com/issues/10329 and I remember (although I don't have traces of that) that Yehuda said this one does not need backporting

Actions
Copy link

Also available in: Atom PDF