Skip to content

QVAC-20608 test[skiplog]: stabilize sdk e2e bootstrap downloads#2557

Merged
Victor-Rodzko merged 2 commits into
mainfrom
fix/sdk-e2e-bootstrap-downloads
Jun 15, 2026
Merged

QVAC-20608 test[skiplog]: stabilize sdk e2e bootstrap downloads#2557
Victor-Rodzko merged 2 commits into
mainfrom
fix/sdk-e2e-bootstrap-downloads

Conversation

@Victor-Rodzko

@Victor-Rodzko Victor-Rodzko commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🎯 What problem does this PR solve?

  • SDK e2e bootstrap could start too many registry model downloads at once, making transient REQUEST_TIMEOUT failures more likely on slower runners and mobile networks.
  • Desktop CI cache config could shadow the e2e registry retry/timeout fixture, so bootstrap was not guaranteed to use the intended registry stability settings.

📝 How does it solve it?

  • Adds a bounded bootstrap download queue with explicit desktop/mobile defaults and a second retry pass for first-pass download failures.
  • Keeps desktop CI-owned cacheDirectory merged with e2e-owned registry retry/timeout defaults before SDK initialization.
  • Makes desktop/mobile download targets explicit and ignores the generated merged desktop config file.

🧪 How was it tested?

  • npm run build in packages/sdk/e2e
  • Local macOS smoke: reports/local-local-1781203047990 (100/100 passed)
  • Local iOS smoke: reports/local-local-1781203876456 (84 passed, 16 expected skipped, 0 failed)
  • GitHub Actions run 27399817010: desktop macOS/Linux/Windows bootstrap applied the generated config and pre-cached 39 constants with concurrency=6; mobile Android/iOS applied the mobile config and pre-cached 25 constants with concurrency=4. No bootstrap REQUEST_TIMEOUT, HypercoreError, or final download failures were found.
  • GitHub Actions run 27403773340: Linux/Windows desktop and Android/iOS mobile bootstrap completed with the same bounded concurrency. Android had one registry REQUEST_TIMEOUT that recovered on internal retry attempt 1/5; Windows/iOS failures were later test failures, and the macOS desktop job was cancelled before bootstrap logs were available.

@Victor-Rodzko Victor-Rodzko changed the title QVAC-20608 fix: stabilize sdk e2e bootstrap downloads QVAC-20608 test[skiplog]: stabilize sdk e2e bootstrap downloads Jun 12, 2026
@Victor-Rodzko Victor-Rodzko added the verified Authorize secrets / label-gate in PR workflows label Jun 12, 2026
Co-authored-by: Cursor <cursoragent@cursor.com>
@Victor-Rodzko Victor-Rodzko force-pushed the fix/sdk-e2e-bootstrap-downloads branch from 7573b8d to 8329a4a Compare June 12, 2026 11:17
@Victor-Rodzko Victor-Rodzko marked this pull request as ready for review June 12, 2026 11:17
@Victor-Rodzko Victor-Rodzko requested review from a team as code owners June 12, 2026 11:17
@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

QVAC E2E — android — ✅ all tests passed (86/100, 2380s)

Config: suite=smoke · filter=(none) · exclude=(none)
View run · Artifacts: reports · Device Farm logs

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

QVAC E2E — ios — ✅ all tests passed (84/100, 1142s)

Config: suite=smoke · filter=(none) · exclude=(none)
View run · Artifacts: reports · Device Farm logs

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

QVAC E2E — windows — ✅ all tests passed (100/100, 691s)

Config: suite=smoke · filter=(none) · exclude=(none)
View run · Artifacts: reports

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

QVAC E2E — linux — ✅ all tests passed (100/100, 489s)

Config: suite=smoke · filter=(none) · exclude=(none)
View run · Artifacts: reports

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

QVAC E2E — macos — ✅ all tests passed (98/100, 336s)

Config: suite=smoke · filter=(none) · exclude=(none)
View run · Artifacts: reports

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Tier-based Approval Status

**PR Tier:** TIER1

**Current Status:** ✅ APPROVED

**Requirements:**
- 1 Team Member approval ✅ (2/1)
- 1 Team Lead OR Management approval ✅ (1/1)



---
*This comment is automatically updated when reviews change.*

@Victor-Rodzko

Copy link
Copy Markdown
Contributor Author

/review

@Victor-Rodzko Victor-Rodzko merged commit 8c6f545 into main Jun 15, 2026
25 checks passed
@Victor-Rodzko Victor-Rodzko deleted the fix/sdk-e2e-bootstrap-downloads branch June 15, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e-tested Test suite has run on this PR. Does not indicate tests pass/fail - see results in comments. test-e2e-smoke Triggers smoke e2e test suite [Currently SDK-only] verified Authorize secrets / label-gate in PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants