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

Update to google-github-actions/setup-gcloud in workflows #2533

Merged
merged 5 commits into from
Jul 27, 2021

Conversation

teor2345
Copy link
Contributor

@teor2345 teor2345 commented Jul 26, 2021

Motivation

Google Cloud just got rid of some deprecated GitHub action names.

Warning: Thank you for using setup-gcloud Action. GoogleCloudPlatform/github-actions/setup-gcloud has been deprecated, please switch to google-github-actions/setup-gcloud.

https://github.com/ZcashFoundation/zebra/runs/3145563888#step:4:14

API Reference

See https://github.com/google-github-actions/setup-gcloud#use-google-github-actionssetup-gcloud

Solution

  • Use the new action names

Review

Anyone can review this PR.

This change is urgent because every main merge will fail until it is fixed.

I manually launched the Google Cloud workflows:

The first CD run tests creating the instance group & deploying. We can test deploying CD updates by merging to the main branch.

Reviewer Checklist

  • CI works before merge
  • CI works after merge on main

This change is Reviewable

@teor2345 teor2345 added C-bug Category: This is a bug A-devops Area: Pipelines, CI/CD and Dockerfiles P-High I-integration-fail Continuous integration fails, including build and test failures labels Jul 26, 2021
@teor2345 teor2345 added this to the 2021 Sprint 15 milestone Jul 26, 2021
@teor2345 teor2345 requested review from dconnolly and a team July 26, 2021 23:53
@teor2345 teor2345 self-assigned this Jul 26, 2021
@dconnolly
Copy link
Contributor

dconnolly commented Jul 27, 2021

Ran CD workflow again as there was an instance template collision? error? on the run teor kicked off
https://github.com/ZcashFoundation/zebra/actions/runs/1071821329

👍

image

Copy link
Contributor

@dconnolly dconnolly left a comment

Choose a reason for hiding this comment

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

Reviewed 5 of 5 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @teor2345)

@dconnolly dconnolly merged commit 5684667 into main Jul 27, 2021
@dconnolly dconnolly deleted the fix-gcloud-workflows branch July 27, 2021 16:13
mpguerra added a commit that referenced this pull request Jul 27, 2021
mpguerra added a commit that referenced this pull request Jul 29, 2021
* Draft CHANGELOG for Zebra 1.0.0-alpha.14

* Add PR #2533 to CHANGELOG

* Apply suggestions from code review

Co-authored-by: teor <[email protected]>

* Remove entry about updating the changelog

* move #2497

* add #2529

* Add a missing space

* Add #2458, #2525, #2486, #2542 and  #2539 to CHANGELOG

Co-authored-by: teor <[email protected]>
Co-authored-by: Deirdre Connolly <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles C-bug Category: This is a bug I-integration-fail Continuous integration fails, including build and test failures
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants