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

Moving tests to regions with higher IP quota #9109

Merged
merged 6 commits into from
Feb 14, 2023
Merged

Conversation

rsamborski
Copy link
Member

@rsamborski rsamborski commented Feb 10, 2023

Description

Moving tests to regions with higher IN-USE-ADDRESSES-per-project-region quota

Fixes #9084

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

@product-auto-label product-auto-label bot added samples Issues that are directly related to samples. api: compute Issues related to the Compute Engine API. labels Feb 10, 2023
@rsamborski rsamborski marked this pull request as ready for review February 10, 2023 11:02
@rsamborski rsamborski requested review from m-strzelczyk and a team as code owners February 10, 2023 11:02
@rsamborski rsamborski enabled auto-merge February 10, 2023 13:34
@rsamborski rsamborski added this pull request to the merge queue Feb 10, 2023
@nicain nicain removed this pull request from the merge queue due to a manual request Feb 10, 2023
@nicain nicain added this pull request to the merge queue Feb 10, 2023
@nicain nicain removed this pull request from the merge queue due to a manual request Feb 10, 2023
@nicain nicain added this pull request to the merge queue Feb 10, 2023
@leahecole
Copy link
Collaborator

To other maintainers - leave this alone for now even if it seems stuck! @nicain is looking at it and has filed GH feedback with this PR as an example

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 10, 2023
@rsamborski
Copy link
Member Author

@leahecole Can we proceed with merging this one? I see it was automatically removed from the merge queue by the bot.

@kweinmeister kweinmeister merged commit d302dae into main Feb 14, 2023
@kweinmeister kweinmeister deleted the fix-compute-ip-quota branch February 14, 2023 13:50
telpirion pushed a commit that referenced this pull request Mar 13, 2023
* Moving tests to regions with higher IP quota

* Move back to europe-north1-a due to permissions

---------

Co-authored-by: Dan Lee <71398022+dandhlee@users.noreply.github.com>
Co-authored-by: Karl Weinmeister <11586922+kweinmeister@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: compute Issues related to the Compute Engine API. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

compute.api.create_instance_test: test_main failed
6 participants