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

Required storage roles / permissions #764

Open
maingoh opened this issue Dec 2, 2024 · 4 comments
Open

Required storage roles / permissions #764

maingoh opened this issue Dec 2, 2024 · 4 comments

Comments

@maingoh
Copy link

maingoh commented Dec 2, 2024

What are the required minimal roles for the state store bucket ? I am especially interested in GCS roles.

@wjf3121
Copy link
Contributor

wjf3121 commented Dec 3, 2024

Hi @maingoh, assuming the bucket is DATA_BUCKET and the dataDirectory is DATA_PREFIX, then the minimum permissions you need to grant to the role are:

  • roles/storage.objectViewer to the entire bucket (conditioned by resource.name.startsWith("projects/_/buckets/DATA_BUCKET") || resource.name == "projects/_/buckets/DATA_BUCKET")
  • roles/storage.objectAdmin to under the prefix of the bucket (conditioned by resource.name.startsWith("projects/_/buckets/DATA_BUCKET/objects/DATA_PREFIX")

Let me know if this answers you question, thanks!

@maingoh
Copy link
Author

maingoh commented Dec 4, 2024

Thank you for the fast answer ! I will try this and let you know if I encounter any issue

@maingoh
Copy link
Author

maingoh commented Dec 4, 2024

If the bucket is dedicated to risingwave it could be simplified to roles/storage.objectAdmin (with eventually a condition to resource.name.startsWith("projects/_/buckets/DATA_BUCKET") || resource.name == "projects/_/buckets/DATA_BUCKET" ?

I found this documentation, maybe we could improve it, or even here ?

@wjf3121
Copy link
Contributor

wjf3121 commented Dec 4, 2024

If the bucket is dedicated to risingwave it could be simplified to roles/storage.objectAdmin (with eventually a condition to resource.name.startsWith("projects//buckets/DATA_BUCKET") || resource.name == "projects//buckets/DATA_BUCKET" ?

We haven't tested this case before but it looks correct to me.

I found this documentation, maybe we could improve it, or even here ?

We can surely do that. I will talk to the doc owner. Thanks for the suggestion!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants