Skip to content

Syncd SA changes#132

Closed
haroonc wants to merge 2 commits intoevalsfrom
main
Closed

Syncd SA changes#132
haroonc wants to merge 2 commits intoevalsfrom
main

Conversation

@haroonc
Copy link
Copy Markdown
Contributor

@haroonc haroonc commented Apr 12, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates IAM role assignments for Cloud Build by correcting the Developer Connect Service Agent email format and transitioning to service-account-level role bindings via a new AddServiceAccountRoleBinding method. It also adds logic to grant the Cloud Build Service Agent impersonation rights for the user-specified service account. Feedback was provided to improve variable naming and error message consistency in the setPermissionsForCloudBuildSA function.

@yeshwanth1993 yeshwanth1993 self-requested a review April 12, 2026 18:36
@haroonc haroonc closed this Apr 12, 2026
@haroonc haroonc reopened this Apr 12, 2026
@haroonc haroonc closed this Apr 12, 2026
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