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

Upgrade to google-github-actions/setup-gcloud@v2 #1296

Closed
lpezet opened this issue Jul 15, 2024 · 4 comments
Closed

Upgrade to google-github-actions/setup-gcloud@v2 #1296

lpezet opened this issue Jul 15, 2024 · 4 comments
Labels
enhancement New feature or request Stale

Comments

@lpezet
Copy link
Contributor

lpezet commented Jul 15, 2024

TL;DR

If I'm not mistaken, Github workflow files (e.g. github-tf-apply.yaml) setup in 0-bootstrap (at least), are using v1 of google-github-actions/setup-gcloud. Any particular reasons it's not using v2?

Terraform Resources

No response

Detailed design

Files involved:
- https://github.com/terraform-google-modules/terraform-example-foundation/blob/master/build/github-tf-apply.yaml
- https://github.com/terraform-google-modules/terraform-example-foundation/blob/master/build/github-tf-pull-request.yaml

Additional information

Willing to submit PR if version upgrade OKed. I can't really test it though (besides running it against my own setup)...

@lpezet lpezet added the enhancement New feature or request label Jul 15, 2024
@eeaton
Copy link
Collaborator

eeaton commented Jul 16, 2024

Thanks for flagging @lpezet. No deliberate reason to stay with v1, I have other projects using github-actions/setup-gcloud@v2 that work just fine.

The github actions aren't part of the automated CI set we have that PR must pass before merging, but for a change like this I'd say just running it against your own setup should be sufficient. (I'll do the same before merging)

If you have the availability to make a PR that's great, otherwise I'll add it to the backlog for our next major round of updates, there are a few other issues raised against the github flow that need some review. Probably will configure Dependabot or similar to make sure we're automatically checking for updates to GHA like setup-gcloud

@daniel-cit
Copy link
Contributor

The code is using v1 because it was merged a few weeks before v2 was released.

This is a good opportunity to review all the versions in the github and gitlab build configuration.

@lpezet
Copy link
Contributor Author

lpezet commented Jul 16, 2024

@daniel-cit As you suggested, I went over other github actions and upgraded those.
I pasted what I could find from their changelogs in the PR description as well.
PR #1297

Copy link

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the Stale label Sep 14, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Stale
Projects
None yet
Development

No branches or pull requests

3 participants