Skip to content

fix: remove unused service and redundant IAM bindings in workload identity setup script#481

Merged
kschaab merged 1 commit intogoogle-github-actions:mainfrom
anguillanneuf:patch-1
Mar 19, 2026
Merged

fix: remove unused service and redundant IAM bindings in workload identity setup script#481
kschaab merged 1 commit intogoogle-github-actions:mainfrom
anguillanneuf:patch-1

Conversation

@anguillanneuf
Copy link
Contributor

@anguillanneuf anguillanneuf commented Mar 13, 2026

The original fix for #444 dated in January is incomplete.

This change removes an unused Google internal service cloudcode-pa.googleapis.com for users in the setup script for Workload Identity Federation.

It also removes redundant IAM bindings on the Workload Identity Pool's principal in Step 4. These IAM bindings are added in Step 5 on a connected service account to the Workload Identity Pool, different from its principal. GitHub Actions "impersonates" this connected service account, which has access to Google Cloud resources and services.

Lastly, it updates the description to correctly reflect the content of the shell script.

I have published this codelab that teaches about this setup.

remove an unused internal service cloudcode-pa.googleapis.com 


remove redundant IAM bindings on the Workload Identity Pool's principal from the original script

Signed-off-by: Tianzi Cai <tianzi@google.com>
@anguillanneuf
Copy link
Contributor Author

@kschaab for a review when you have a chance. Ty.

@kschaab kschaab merged commit 642deeb into google-github-actions:main Mar 19, 2026
16 checks passed
@anguillanneuf anguillanneuf deleted the patch-1 branch March 20, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants