Skip to content

Nightly Windows CI failure #2146

New issue
New issue
Open
Open
Nightly Windows CI failure#2146
Labels
Stale

Description

@github-actions
github-actions
bot
opened on Jul 21, 2026

The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/29806012131

A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

Activity

  1. github-actions commented on Jul 23, 2026

    @github-actions
    github-actionsBot
    on Jul 23, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/29984270832

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  2. Tobbe commented on Jul 23, 2026

    @Tobbe
    Tobbe
    on Jul 23, 2026
    Member

    They both passed on a re-run

  3. github-actions commented on Jul 26, 2026

    @github-actions
    github-actionsBot
    on Jul 26, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/30190743916

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  4. lisa-assistant commented on Jul 26, 2026

    @lisa-assistant
    lisa-assistant
    on Jul 26, 2026
    Collaborator

    Implementation Plan

    The nightly Windows run on 2026-07-26 (run #30190743916) had two failures:

    Failure 1 — Background jobs E2E: "Expected 1 cron job report, but found 2"

    File: tasks/e2e-background-jobs/run.mts (around line 520) and tasks/e2e-background-jobs/fixtures.mts (line 79)

    Root cause: The cron job is scheduled with cron: '* * * * * *' (every second). By the time confirmJobsRan checks the project directory, the cron job has already fired more than once, leaving multiple report-*.txt files.

    Fix options:

    Option A (preferred): In confirmJobsRan, record which report files exist before calling jobsWorkoff, then assert that exactly 1 new file appeared after:

    // before jobsWorkoff:
const existingReports = new Set(fs.readdirSync(projectPath).filter(f => /^report-.*\.txt$/.test(f)))

// run jobsWorkoff...

// in confirmJobsRan:
const allReports = fs.readdirSync(projectPath).filter(f => /^report-.*\.txt$/.test(f))
const newReports = allReports.filter(f => !existingReports.has(f))
if (newReports.length !== 1) { ... }

    Option B (simpler): Change the assertion from !== 1 to < 1 — i.e., require at least one report file rather than exactly one. Less precise but eliminates the race condition entirely.

    Option C (also viable): Delete all report-*.txt files at the start of confirmJobsRan before running the assertion. But this could hide bugs where no report is written.

    Option A is the most correct: it measures what the test actually cares about (the cron job ran at least once during the workoff).

    Failure 2 — Smoke tests: Nx Cloud client bundle not initialized

    Log: This usually means multiple copies of the Nx Cloud client bundle are being loaded by Nx

    This is an intermittent Nx infrastructure issue (see nrwl/nx#35209). The job failed at the Set up job step before even checking out the code, so it's not caused by any Cedar commit.

    Fix: No Cedar code change needed. Options:

    • Add a retry on the smoke-tests job in .github/workflows/nightly-windows.yml using continue-on-error: true + a retry workflow, or use the nick-fields/retry action for the nx run call
    • Or bump the Nx version to one that has fixed #35209

    Summary of changes needed

    File Change
    tasks/e2e-background-jobs/run.mts Pass pre-existing report files into confirmJobsRan and diff against them
    tasks/e2e-background-jobs/run.mts line ~112 Capture report file snapshot before jobsWorkoff()
    .github/workflows/nightly-windows.yml (optional) Add retry on the Nx Cloud transient failure

  5. lisa-assistant commented on Jul 26, 2026

    @lisa-assistant
    lisa-assistant
    on Jul 26, 2026
    Collaborator

    Confidence: 7/10

    The E2E fix is clear — the cron schedule (* * * * * *) is inherently racy. Fixing the assertion logic (Option A) is mechanical once you understand the test flow. The score isn't higher because:

    1. The Nx Cloud failure is infrastructure-level and has no Cedar fix
    2. The smoke tests failure log doesn't show which suite actually failed — the job aborted at setup, so there may be an underlying issue that only manifests on Windows and only when the Nx cache warms up in a particular order

  6. github-actions commented on Jul 28, 2026

    @github-actions
    github-actionsBot
    on Jul 28, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/30333589337

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  7. Tobbe commented on Jul 28, 2026

    @Tobbe
    Tobbe
    on Jul 28, 2026
    Member

    Once again passed on a re-run

  8. Tobbe commented on Jul 28, 2026

    @Tobbe
    Tobbe
    on Jul 28, 2026
    Member

    The proposed Background Jobs E2E test fix was implemented in #2245

  9. github-actions commented on Jul 29, 2026

    @github-actions
    github-actionsBot
    on Jul 29, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/30427201144

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  10. github-actions commented on Jul 31, 2026

    @github-actions
    github-actionsBot
    on Jul 31, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/30609564488

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  11. github-actions commented on Aug 3, 2026

    @github-actions
    github-actionsBot
    on Aug 3, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/30790726804

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  12. github-actions commented on Aug 4, 2026

    @github-actions
    github-actionsBot
    on Aug 4, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/30882766089

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  13. github-actions commented on Aug 6, 2026

    @github-actions
    github-actionsBot
    on Aug 6, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/31076119277

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  14. github-actions commented on Aug 7, 2026

    @github-actions
    github-actionsBot
    on Aug 7, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/31149346860

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  15. github-actions commented on Aug 8, 2026

    @github-actions
    github-actionsBot
    on Aug 8, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/31239552284

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  16. 17 remaining items

  17. github-actions commented on Sep 22, 2026

    @github-actions
    github-actionsBot
    on Sep 22, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/35705553139

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  18. github-actions commented on Sep 23, 2026

    @github-actions
    github-actionsBot
    on Sep 23, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/35838116913

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  19. Tobbe
    added a commit that references this issue on Sep 25, 2026

    chore(ci): drop Windows-specific Node pin, track 24 on all platforms (#…

    6923b19

  20. github-actions commented on Sep 25, 2026

    @github-actions
    github-actionsBot
    on Sep 25, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/36115186377

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  21. lisa-assistant
    added a commit that references this issue on Sep 25, 2026

    fix(ci): force check-latest on Windows Node setup to avoid stale cach…

    7f720ba
  22. Tobbe
    added 2 commits that reference this issue on Sep 26, 2026

    chore(ci): drop Windows-specific Node pin, track 24 on all platforms (#…

    8e0593e

    fix(ci): force check-latest on Windows Node setup to avoid stale cach…

    398c64c

  23. github-actions commented on Sep 28, 2026

    @github-actions
    github-actionsBot
    on Sep 28, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/36405970479

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  24. github-actions commented on Sep 29, 2026

    @github-actions
    github-actionsBot
    on Sep 29, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/36551935555

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  25. github-actions commented on Oct 4, 2026

    @github-actions
    github-actionsBot
    on Oct 4, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/37193450037

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  26. github-actions commented on Nov 5, 2026

    @github-actions
    github-actionsBot
    on Nov 5, 2026 – with GitHub Actions
    Author

    This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

  27. github-actions
    added
    Stale
    on Nov 5, 2026

  28. github-actions commented on Nov 6, 2026

    @github-actions
    github-actionsBot
    on Nov 6, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/42206438672

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  29. github-actions commented on Nov 7, 2026

    @github-actions
    github-actionsBot
    on Nov 7, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/42363149212

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

  30. github-actions commented on Nov 10, 2026

    @github-actions
    github-actionsBot
    on Nov 10, 2026 – with GitHub Actions
    Author

    The nightly Windows run failed: https://github.com/cedarjs/cedar/actions/runs/42806018139

    A change that the Windows PR filter didn't flag may have broken Windows support. See the run for which suites failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Stale

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions