Skip to content

fix: changes default region for dataproc quickstart test to us-central2 #9764

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

Merged
merged 7 commits into from
Apr 21, 2023

Conversation

nicain
Copy link
Contributor

@nicain nicain commented Apr 21, 2023

Fixes #9461

Seeing intermittent failures:

google.api_core.exceptions.InvalidArgument: 400 The resource 'projects/<project_id>/regions/us-central1/subnetworks/default' is not ready

The current hypothesis is that the subnet being used as a test fixture for this test is overused by multiple other tests, and becoming unavailable when this test is trying to run. This change offers a short-term fix, by moving the testing to another region and therefore another default subnet.

@nicain nicain added the snippet-bot:force-run Force snippet-bot runs its logic label Apr 21, 2023
@product-auto-label product-auto-label bot added samples Issues that are directly related to samples. api: dataproc Issues related to the Dataproc API. labels Apr 21, 2023
@snippet-bot snippet-bot bot removed the snippet-bot:force-run Force snippet-bot runs its logic label Apr 21, 2023
@nicain nicain marked this pull request as ready for review April 21, 2023 20:07
@nicain nicain requested a review from a team as a code owner April 21, 2023 20:07
Copy link
Collaborator

@leahecole leahecole left a comment

Choose a reason for hiding this comment

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

LGTM if passes - if not, feel free to assign to me

@nicain nicain force-pushed the dataproc_default_region branch from c8c7189 to a6ffefc Compare April 21, 2023 21:00
…t disk_config to smaller values for several tests
@nicain nicain force-pushed the dataproc_default_region branch from 399f7de to 17e7e35 Compare April 21, 2023 22:12
@nicain nicain added the snippet-bot:force-run Force snippet-bot runs its logic label Apr 21, 2023
@snippet-bot snippet-bot bot removed the snippet-bot:force-run Force snippet-bot runs its logic label Apr 21, 2023
@nicain nicain merged commit 8bfa093 into main Apr 21, 2023
@nicain nicain deleted the dataproc_default_region branch April 21, 2023 22:35
@leahecole leahecole linked an issue Apr 24, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: dataproc Issues related to the Dataproc API. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dataproc.snippets.quickstart.quickstart_test: test_quickstart failed dataproc.snippets.update_cluster_test: test_update_cluster failed
2 participants