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

Add bucket IAM policy read #2872

Merged
merged 2 commits into from
Feb 9, 2025

Conversation

karpok78
Copy link
Contributor

@karpok78 karpok78 commented Feb 8, 2025

Allow the Project factory read only SA to retrieve buckets IAM policy for buckets created by the PF


Checklist

I applicable, I acknowledge that I have:

  • Read the contributing guide
  • Ran terraform fmt on all modified files
  • Regenerated the relevant README.md files using tools/tfdoc.py
  • Made sure all relevant tests pass

Allow the Project factory read only SA to retrieve buckets IAM policy for buckets created by the PF
@ludoo ludoo enabled auto-merge (squash) February 8, 2025 15:50
@ludoo
Copy link
Collaborator

ludoo commented Feb 8, 2025

@juliocc project module is failing universe test...

@juliocc
Copy link
Collaborator

juliocc commented Feb 8, 2025

@juliocc project module is failing universe test...

Ah that's my fault. Let me take a look

@ludoo ludoo merged commit e4f55fb into GoogleCloudPlatform:fast-dev Feb 9, 2025
14 checks passed
@karpok78 karpok78 deleted the pf_bucket_iam_ro branch February 20, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants