Skip to content

ci: raise the wasm headless tests timeout to 15 minutes - #3506

Merged
jpraynaud merged 1 commit into
mainfrom
jpraynaud/fix-flakiness-wasm-client-ci-test
Aug 31, 2026
Merged

ci: raise the wasm headless tests timeout to 15 minutes#3506
jpraynaud merged 1 commit into
mainfrom
jpraynaud/fix-flakiness-wasm-client-ci-test

Conversation

@jpraynaud

@jpraynaud jpraynaud commented Aug 31, 2026

Copy link
Copy Markdown
Member

Content

This PR includes a fix for the flakiness of the wasm client CI test which fails with a Selenium timeout on the nightly runs.

Pre-submit checklist

  • Branch
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • All check jobs of the CI have succeeded
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

The certificate chain of 'testing-preview' grows daily and the 5 chain
verifications of the test now exceed the 10 minutes limit.
@jpraynaud jpraynaud self-assigned this Aug 31, 2026
@jpraynaud
jpraynaud requested a review from Alenar as a code owner August 31, 2026 07:37
Copilot AI lite review requested due to automatic review settings August 31, 2026 07:37
@jpraynaud
jpraynaud requested a review from turmelclem as a code owner August 31, 2026 07:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces flakiness in the WASM client headless browser CI by increasing the Selenium explicit wait timeout so slower nightly runs don’t fail with a WebDriverWait timeout.

Changes:

  • Increased the headless browser test WebDriverWait timeout from 10 minutes (600s) to 15 minutes (900s).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@turmelclem turmelclem left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@github-actions

Copy link
Copy Markdown

Test Results

     5 files  ± 0     221 suites  ±0   59m 43s ⏱️ - 1h 5m 36s
 3 505 tests  - 57   3 505 ✅  - 57  0 💤 ±0  0 ❌ ±0 
11 612 runs   - 66  11 612 ✅  - 66  0 💤 ±0  0 ❌ ±0 

Results for commit 93ea64e. ± Comparison against base commit df38c0c.

This pull request removes 57 tests.
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::index_out_of_bounds
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::index_too_large_for_circuit_range
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::indices_not_increasing
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::leaf_merkle_path_mismatch
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::leaf_swap_keep_merkle_path
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::leaf_wrong_verification_key
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::merkle_path_corrupt_sibling
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::merkle_path_flip_position
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::merkle_path_length_long
mithril-stm ‑ circuits::halo2::tests::golden::cases::negative::slow::merkle_path_length_short
…

@jpraynaud
jpraynaud deployed to testing-2-preview August 31, 2026 07:58 — with GitHub Actions Active
@jpraynaud
jpraynaud deployed to testing-preview August 31, 2026 07:58 — with GitHub Actions Active
@jpraynaud
jpraynaud merged commit b780e13 into main Aug 31, 2026
94 of 95 checks passed
@jpraynaud
jpraynaud deleted the jpraynaud/fix-flakiness-wasm-client-ci-test branch August 31, 2026 09:03
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.

4 participants