Skip to content

test(compute-v1): Address flaky tests#34623

Closed
aandreassa wants to merge 1 commit into
mainfrom
compute-accept-fix
Closed

test(compute-v1): Address flaky tests#34623
aandreassa wants to merge 1 commit into
mainfrom
compute-accept-fix

Conversation

@aandreassa

@aandreassa aandreassa commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Fixes intermittent test failures caused by Quota exceeded for quota metric 'Read requests' on Compute Engine API calls.

Changes:

  1. Reduce LRO polling frequency: Increased max_delay from 1 to 5 seconds in samples/quickstart.rb to prevent tests from hitting the API once per second while waiting for resources (firewalls, instances).
  2. Increase pagination page size: Set max_results: 100 instead of 10 in PaginationSmokeTest for aggregated lists. This ensures all zones are retrieved in a single request rather than paginating through 10+ pages, reducing request overhead.

@aandreassa aandreassa force-pushed the compute-accept-fix branch from d2d5373 to e31f42b Compare June 17, 2026 22:39
@aandreassa aandreassa changed the title chore(compute-v1): trigger PR build with comment test(compute-v1): Address flaky tests Jun 17, 2026
@aandreassa aandreassa force-pushed the compute-accept-fix branch from e31f42b to 0b72d97 Compare June 18, 2026 00:55
@aandreassa aandreassa added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 18, 2026
@aandreassa

Copy link
Copy Markdown
Contributor Author

Still figuring this out!

@aandreassa aandreassa closed this Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants