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

fix(ci): Switch CI to a GCP zone with available instances #6365

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

teor2345
Copy link
Contributor

Motivation

GCP CI fails with:

A c2d-standard-16 VM instance is currently unavailable in the us-central1-a
zone. Consider trying your request in the us-central1-f, us-central1-b, us-central1-c
zone(s), which currently has capacity to accommodate your request. Alternatively,
you can try your request again with a different VM hardware configuration or
at a later time. For more information, see the troubleshooting documentation.

https://github.com/ZcashFoundation/zebra/actions/runs/4429186789/jobs/7772459849#step:10:94

Solution

  • Switch the only workflow that uses c2d-standard-16 machines to us-central1-f

Review

This is an urgent CI failure fix.

Reviewer Checklist

  • Will the PR name make sense to users?
    • Does it need extra CHANGELOG info? (new features, breaking changes, large changes)
  • Are the PR labels correct?
  • Does the code do what the ticket and PR says?
    • Does it change concurrent code, unsafe code, or consensus rules?
  • How do you know it works? Does it have tests?

Follow Up Work

We'll also need to change the corresponding variable in PR #6357.

@teor2345 teor2345 added C-bug Category: This is a bug A-infrastructure Area: Infrastructure changes A-devops Area: Pipelines, CI/CD and Dockerfiles P-Critical 🚑 I-integration-fail Continuous integration fails, including build and test failures C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG labels Mar 21, 2023
@teor2345 teor2345 requested a review from a team as a code owner March 21, 2023 02:11
@teor2345 teor2345 self-assigned this Mar 21, 2023
@teor2345 teor2345 requested review from arya2 and removed request for a team March 21, 2023 02:11
Copy link
Member

@gustavovalverde gustavovalverde left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit de0767e into main Mar 21, 2023
@mergify mergify bot deleted the ci-switch-zone branch March 21, 2023 11:19
@oxarbitrage oxarbitrage mentioned this pull request Mar 23, 2023
36 tasks
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 A-infrastructure Area: Infrastructure changes C-bug Category: This is a bug C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG 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