Skip to content

dataflow: use us-central1-f for gpu test - #5413

Merged
davidcavazos merged 2 commits into
masterfrom
dataflow-gpu-zone
Feb 26, 2021
Merged

davidcavazos merged 2 commits into
masterfrom
dataflow-gpu-zone

Conversation

@davidcavazos

Copy link
Copy Markdown

Description

It won't fix the timeouts, but it should fix the quota issues.

Fixes #5275, #5395, #5397

Checklist

@davidcavazos
davidcavazos requested a review from a team as a code owner February 25, 2021 21:00
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Feb 25, 2021
@product-auto-label product-auto-label Bot added api: dataflow Issues related to the Dataflow API. samples Issues that are directly related to samples. labels Feb 25, 2021

@leahecole leahecole left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM if tests pass - will we eventually run into quota issues in this zone though?

@davidcavazos

Copy link
Copy Markdown
Author

The workaround was more around this comment. I missed that comment before merging the PR, and for some reason I thought I already made the changes in another PR, but apparently not.

@davidcavazos
davidcavazos enabled auto-merge (squash) February 26, 2021 17:06
@davidcavazos
davidcavazos merged commit 9b5289c into master Feb 26, 2021
@davidcavazos
davidcavazos deleted the dataflow-gpu-zone branch February 26, 2021 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: dataflow Issues related to the Dataflow API. cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants