Skip to content

Harden Windows x86-64 CI LLVM install against Chocolatey 504 timeouts#177

Merged
bqminh merged 2 commits into
masterfrom
copilot/fix-windows-x86-64-job
Jun 19, 2026
Merged

Harden Windows x86-64 CI LLVM install against Chocolatey 504 timeouts#177
bqminh merged 2 commits into
masterfrom
copilot/fix-windows-x86-64-job

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

The Windows x86-64 Actions job failed before compilation in the Install LLVM v14 step due to transient 504 Gateway Timeout responses from Chocolatey. This change makes that dependency install resilient to temporary feed outages without altering the build configuration.

  • Root cause addressed

    • choco install llvm --version=14.0.6 --allow-downgrade previously failed fast on a single network timeout.
  • Workflow change (.github/workflows/ci.yaml)

    • Replaced the one-shot install command with bounded retry logic in cmd.
    • Retries up to 3 times with a short delay between attempts.
    • Preserves failure behavior if all attempts fail.
  • Behavioral impact

    • Improves reliability of the Windows dependency bootstrap path.
    • No changes to compiler version, build flags, packaging, or benchmark/test steps.
set RETRIES=3
set WAIT_SECONDS=20
for /L %%i in (1,1,%RETRIES%) do (
  choco install llvm --version=14.0.6 --allow-downgrade
  if !ERRORLEVEL! EQU 0 goto :success
  if %%i LSS %RETRIES% timeout /t %WAIT_SECONDS% /nobreak >nul
)
exit /b 1
:success

Copilot AI changed the title [WIP] Fix failing GitHub Actions job "Windows x86-64" Harden Windows x86-64 CI LLVM install against Chocolatey 504 timeouts Jun 19, 2026
Copilot AI requested a review from bqminh June 19, 2026 02:06
@bqminh bqminh marked this pull request as ready for review June 19, 2026 02:07
Copilot AI review requested due to automatic review settings June 19, 2026 02:07

Copilot AI 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.

Pull request overview

This PR improves the resilience of the Windows x86-64 GitHub Actions workflow by adding bounded retry logic around the Chocolatey installation of LLVM 14.0.6, addressing transient Chocolatey feed/network failures (e.g., 504 timeouts) without changing the build/toolchain configuration itself.

Changes:

  • Replaced the single choco install llvm --version=14.0.6 --allow-downgrade invocation with a cmd retry loop (3 attempts).
  • Added a short wait (timeout) between retries plus clearer log output for attempts and final failure/success.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bqminh bqminh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

great

@bqminh bqminh merged commit 70d1d98 into master Jun 19, 2026
8 checks passed
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