Repository navigation
ci: retry yarn install on transient registry errors - #35556
chirag-madlani wants to merge 6 commits into
Conversation
yarn v1 does not retry HTTP 5xx on tarball fetches, so a single registry 502 aborts the install and fails the job (e.g. RDF Playwright run 38749383300 attempt 1). Wrap every hard-failing yarn install in the same 3-attempt loop playwright-e2e-reusable.yml already uses.
|
✅ SSO Login Nightly — PASSED View workflow run · Commit
Totals: 118 passed · 0 failed · 0 flaky · 0 skipped · duration 23m 29s |
…ight-workflows-merge
Code Review ✅ Approved 1 closed / 1 findings🟡 Medium risk · Adding three-attempt Yarn install retries across 11 CI workflows could prolong job failures and consume runner time during persistent registry outages. Adds a 3-attempt retry loop with backoff to every ✅ 1 closed✅ Quality: CI-wide yarn retry budget and network timeout are unrecorded
VerificationRules No rules evaluated Functional validation Not enabledAuto-approval Not enabledFor more verification details, comment OptionsDisplay: compact → Showing compact verification results. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source |
Describe your changes:
CI-only change, no linked issue (
skip-pr-checks). Fixes the RDF Playwright failure in run 38749383300 attempt 1, whereregistry.yarnpkg.comreturned502 Bad Gatewayfor one tarball and yarn v1 aborted the install.yarn v1 does not retry HTTP 5xx on tarball fetches.
playwright-e2e-reusable.ymlalready wraps its installs in a 3-attempt retry loop (15s/30s backoff,--network-timeout 100000); this PR applies the same loop to every otheryarn installwhose failure fails the job — keeping each step's existing flags:playwright-knowledge-graph-postgresql-e2e.yml,playwright-search-nightly.yml,playwright-sso-login-nightly.yml,playwright-visual.ymlrelease-branch-validate.yml,security-scan.yml,update-playwright-e2e-docs.yml,ui-sonarcloud-nightly.yml,yarn-coverage.ymltypescript-type-generation.yml(root + UI installs)playwright-e2e-reusable.yml— the dockerized core-components install in the plan jobLeft as-is: the report-deps install in
playwright-postgresql-e2e.yml(continue-on-error, can't fail the job) and the Mavenfrontend-maven-plugininstalls (pom change, out of scope).Type of change:
High-level design:
N/A — same retry block copied into each workflow step; no behaviour change on success.
Tests:
Use cases covered
yarn installretries up to 3 times instead of failing the job.Unit tests
Not applicable — workflow-only change.
Backend integration tests
Ingestion integration tests
Playwright (UI) tests
Manual testing performed
yaml.safe_load.actionlinton the edited files reports nothing on the changed steps (only pre-existing shellcheck notes elsewhere).UI screen recording:
Not applicable — CI workflow change.
Checklist:
Fixes <issue-number>: <short explanation>— N/A, CI fix withskip-pr-checks.Fixes #<issue-number>above. — N/A, CI fix withskip-pr-checks.🤖 Generated with Claude Code


Confidence Score: 5/5
The PR appears safe to merge.Summary
Adds bounded retries to dependency installs across 11 CI workflows.
Reviews (4) · Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." · Reviewed by Greptile