Skip to content

GCP User IAM workflow modified to post Terraform plan as a comment on PRs - #36375

Merged
pabloem merged 6 commits into
apache:masterfrom
ksobrenat32:infra-iam-dry-run-on-pr
Oct 7, 2025
Merged

GCP User IAM workflow modified to post Terraform plan as a comment on PRs#36375
pabloem merged 6 commits into
apache:masterfrom
ksobrenat32:infra-iam-dry-run-on-pr

Conversation

@ksobrenat32

Copy link
Copy Markdown
Contributor

This PR updates .github/workflows/beam_Infrastructure_UsersPermissions.yml to improve the review of IAM changes. The main change is adding a comment on each PR, commenting the changes plan generated by Terraform. Apply command has now been restricted only on pushes to master branch.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions github-actions Bot added the build label Oct 3, 2025
@ksobrenat32

Copy link
Copy Markdown
Contributor Author

R: @pabloem

@github-actions

github-actions Bot commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment assign set of reviewers

@pabloem

pabloem commented Oct 4, 2025

Copy link
Copy Markdown
Member

this looks good to me.- have you tested it? we could test it in this PR direcly by changing users.yml

@ksobrenat32

Copy link
Copy Markdown
Contributor Author

this looks good to me.- have you tested it? we could test it in this PR direcly by changing users.yml

Haven't tested it yet, maybe we could migrate my user to the viewer role?

@pabloem

pabloem commented Oct 6, 2025

Copy link
Copy Markdown
Member

Haven't tested it yet, maybe we could migrate my user to the viewer role?

sure that works! - you can make the change in this PR and we should see the comment appear

@ksobrenat32
ksobrenat32 force-pushed the infra-iam-dry-run-on-pr branch from cebd433 to 64fa9fd Compare October 7, 2025 19:39
@ksobrenat32
ksobrenat32 force-pushed the infra-iam-dry-run-on-pr branch from 64fa9fd to 5b92d0a Compare October 7, 2025 19:41
@pabloem

pabloem commented Oct 7, 2025

Copy link
Copy Markdown
Member

lgtm!

@pabloem
pabloem merged commit 77973ca into apache:master Oct 7, 2025
4 checks passed
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.

2 participants