Skip to content

Comments

chore: improve reliability of deploy integration tests#7951

Merged
serhalp merged 4 commits intomainfrom
ci/specify-integration-test-account
Feb 18, 2026
Merged

chore: improve reliability of deploy integration tests#7951
serhalp merged 4 commits intomainfrom
ci/specify-integration-test-account

Conversation

@serhalp
Copy link
Member

@serhalp serhalp commented Feb 18, 2026

Summary

The deploy command integration tests have become extremely unreliable recently. This PR makes them pass reliably.

create integration test sites in testing account

This mechanism already exists but wasn't being used here, so it was falling back to the first account the user has access to that is returned by the accounts API.

Integration tests in CI were often failing lately, and under the hood it appears to be because requests to the test sites we're creating on the fly were being rate limited.

The netlify-integration-testing account is properly configured to host our various integration test sites, e.g. to avoid being rate limited.

lower test concurrency

Running multiple entire build + deploy cycles concurrently results in a lot of network requests that may cause resource contention anyway, so lower the default concurrency from 5 to 3.

increase test timeout

The timeout was 90s, which is a pretty low threshold given that 36 of these tests run in each job, multiplied by 4 test matrix variations, multiplied by however many jobs happen to be running. We were even within the test giving a 5 min timeout to the underlying ntl deploy, but this could never be reached because the test itself stopped after 90s.

Bump this up to 5 mins.

This [mechanism already
exists](https://github.com/netlify/cli/blob/b80b98f85929803fc35a08458c9327dc7ef63de0/tests/integration/utils/create-live-test-site.ts#L22-L36)
but wasn't being used here, so it was falling back to the first account the user has access to that
is returned by the accounts API.

The `netlify-integration-testing` account is properly configured to host our various integration
test sites, e.g. to avoid being rate limited.
@github-actions
Copy link

github-actions bot commented Feb 18, 2026

📊 Benchmark results

Comparing with b80b98f

  • Dependency count: 1,073 (no change)
  • Package size: 325 MB (no change)
  • Number of ts-expect-error directives: 364 (no change)

@serhalp serhalp force-pushed the ci/specify-integration-test-account branch from dd737b2 to 25ec325 Compare February 18, 2026 15:21
@serhalp serhalp changed the title ci: create integration test sites in testing account chore: improve reliability of deploy integration tests Feb 18, 2026
@serhalp serhalp marked this pull request as ready for review February 18, 2026 15:51
@serhalp serhalp requested a review from a team as a code owner February 18, 2026 15:51
@serhalp serhalp enabled auto-merge (squash) February 18, 2026 16:45
@serhalp serhalp merged commit f876f57 into main Feb 18, 2026
154 of 160 checks passed
@serhalp serhalp deleted the ci/specify-integration-test-account branch February 18, 2026 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants