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

quincy: rgw/sts: AssumeRole no longer writes to user metadata #52049

Merged
merged 3 commits into from Jan 12, 2024

Conversation

trociny
Copy link
Contributor

@trociny trociny commented Jun 14, 2023

backport tracker: https://tracker.ceph.com/issues/59612


backport of #51161
parent tracker: https://tracker.ceph.com/issues/59495

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@trociny trociny requested a review from a team as a code owner June 14, 2023 10:05
@trociny trociny added this to the quincy milestone Jun 14, 2023
@trociny trociny added the rgw label Jun 14, 2023
`storeARN()` was storing the role's ARN in `RGWUserInfo::assumed_role_arn`,
but that field was unused

Fixes: https://tracker.ceph.com/issues/59495

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 2f60200)

Conflicts:
	src/rgw/rgw_sts.cc (trivial)
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 9da3120)

Conflicts:
	src/rgw/store/dbstore/common/dbstore.h (no 'static constexpr' keywords)
	src/rgw/store/dbstore/sqlite/sqliteDB.cc (trivial)
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit af5f9ca)
@cbodley
Copy link
Contributor

cbodley commented Oct 24, 2023

jenkins test api

@yuriw yuriw merged commit 84f7d57 into ceph:quincy Jan 12, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants