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

(rook-ceph/konkong) add s3 policy for s3-butler and README.md #627

Merged
merged 1 commit into from
Oct 9, 2024
Merged

(rook-ceph/konkong) add s3 policy for s3-butler and README.md #627

merged 1 commit into from
Oct 9, 2024

Conversation

cbarria
Copy link
Contributor

@cbarria cbarria commented Oct 9, 2024 •
edited
Loading

this are the policies and the instructions on how to execute ticket IT-5301
this most probably is going to be asked to be replicated, so don't forget about this commits.

@cbarria cbarria added the enhancement New feature or request label Oct 9, 2024
@cbarria cbarria self-assigned this Oct 9, 2024
gseriche
gseriche approved these changes Oct 9, 2024
View reviewed changes
Copy link

@gseriche gseriche left a comment

Choose a reason for hiding this comment

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

Is the flag --access-key --secret-key necessary?
I understand the endpoint is Ceph.

@cbarria
Copy link
Contributor Author

cbarria commented Oct 9, 2024 •
edited
Loading

Is the flag --access-key --secret-key necessary? I understand the endpoint is Ceph.

i just removed the KEY and SECRET to show this on public
BTW: if you remove those, ceph will generate random KEY and SECRET in the type of IAM like

jhoblitt
jhoblitt approved these changes Oct 9, 2024
View reviewed changes
Copy link
Member

@jhoblitt jhoblitt left a comment

Choose a reason for hiding this comment

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

This looks reasonable. I will add the oods-* accounts to #623

@cbarria cbarria merged commit 4125f27 into master Oct 9, 2024
8 checks passed
@cbarria cbarria deleted the IT-5301_policy_s3_butler branch October 9, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhoblitt jhoblitt jhoblitt approved these changes

@gseriche gseriche gseriche approved these changes

@csilva-cl csilva-cl Awaiting requested review from csilva-cl

@dtapiacl dtapiacl Awaiting requested review from dtapiacl

@igonzalezcl igonzalezcl Awaiting requested review from igonzalezcl

@badenerb badenerb Awaiting requested review from badenerb

Assignees

@cbarria cbarria

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cbarria @jhoblitt @gseriche