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: fix swift default auth error after auth strategy refactoring #15711

Merged
merged 1 commit into from Jun 16, 2017

Conversation

Projects
None yet
3 participants
@cbodley
Contributor

cbodley commented Jun 15, 2017

the changes in bd81c21 inadvertently changed swift's default auth error from EPERM to ACCESS. hacky fix to change it back

this fixes the Status returned: 403 Expected: 401 failures in swift test testInvalidAuthToken (test.functional.tests.TestAccount[UTF8])

@cbodley cbodley requested a review from rzarzynski Jun 15, 2017

@rzarzynski

LGTM as a quick fix. I plan to make a follow up clean-up next week. Thanks, @cbodley!

@rzarzynski

This comment has been minimized.

Contributor

rzarzynski commented Jun 15, 2017

jenkins retest this please (readable.sh)

@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 15, 2017

retest this please

@theanalyst theanalyst referenced this pull request Jun 16, 2017

Merged

rgw: implement get/put object tags for S3 #13753

13 of 13 tasks complete
rgw: fix error mapping after auth strategy refactoring
the changes in bd81c21 inadvertently
changed swift's default auth error from EPERM to ACCESS. hacky fix to
change it back

Signed-off-by: Casey Bodley <cbodley@redhat.com>
@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 16, 2017

rebased over #15730, should fix make check

(cc @yehudasa)

@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 16, 2017

jenkins test this please

@yehudasa yehudasa merged commit aa97368 into ceph:master Jun 16, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
make check make check succeeded
Details

@cbodley cbodley deleted the cbodley:wip-rgw-swift-default-auth-error branch Jun 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment