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

feat: Allow to use workload federation principals in state bucket permissions #21

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

zonorti
Copy link

@zonorti zonorti commented Apr 30, 2024

To use workload federation this validation needs to be extended;

More information on federated principals

It seems like this validation is the only part that was blocking.

@zonorti zonorti requested review from mariux and a team as code owners April 30, 2024 11:35
zonorti added a commit to klanggames/terraform-google-storage-bucket-iam that referenced this pull request Apr 30, 2024
variables.tf Outdated Show resolved Hide resolved
Co-authored-by: Zied Elouaer <86047070+zied-elouaer@users.noreply.github.com>
@zonorti
Copy link
Author

zonorti commented May 3, 2024

Attempted to load application default credentials since neither credentials nor access_token was set in the provider block.

Unit tests failing, but it doesn't look related to PR contents.

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

Successfully merging this pull request may close these issues.

2 participants