Skip to content

Fix flaky test: Use setSkipWorkerJobs#39106

Merged
gillespi314 merged 1 commit intomainfrom
38898-flaky-test
Feb 2, 2026
Merged

Fix flaky test: Use setSkipWorkerJobs#39106
gillespi314 merged 1 commit intomainfrom
38898-flaky-test

Conversation

@gillespi314
Copy link
Copy Markdown
Contributor

@gillespi314 gillespi314 commented Jan 30, 2026

Related issue: Resolves #38898

Looking at the logs from the issue, we see that the command to install fleetd is being sent twice, which indicates the worker job ran twice. Speculative fix to setSkipWorkerJobs was validated with RandoKiller™.

2026-01-27T16:57:48.2528634Z {"host_uuid":"09A54077-7060-43FB-9E97-7EC4F8EFD248","info":"sent command to install fleetd"}
2026-01-27T16:57:48.2574915Z {"host_uuid":"09A54077-7060-43FB-9E97-7EC4F8EFD248","info":"unable to find a bootstrap package for DEP enrolled device, skipping installation"}
2026-01-27T16:57:48.2735754Z     integration_mdm_test.go:944: [awaitTriggerProfileSchedule] Starting profile schedule trigger at 2026-01-27T16:57:48Z
2026-01-27T16:57:48.2742390Z     integration_mdm_test.go:948: [awaitTriggerProfileSchedule] Triggering profile schedule...
2026-01-27T16:57:48.2784612Z     integration_mdm_test.go:964: [awaitTriggerProfileSchedule] Waiting for 3 jobs to complete...
2026-01-27T16:57:48.2792241Z {"instanceID":"TestIntegrationsMDM","level":"debug","msg":"done, trigger received","schedule":"mdm_apple_profile_manager"}
2026-01-27T16:57:48.2809898Z {"host_uuid":"09A54077-7060-43FB-9E97-7EC4F8EFD248","info":"sent command to install fleetd"}

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.18%. Comparing base (39f9518) to head (d2db0b7).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #39106   +/-   ##
=======================================
  Coverage   66.18%   66.18%           
=======================================
  Files        2423     2423           
  Lines      194078   194078           
  Branches     8518     8518           
=======================================
+ Hits       128452   128453    +1     
  Misses      53947    53947           
+ Partials    11679    11678    -1     
Flag Coverage Δ
backend 67.95% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gillespi314 gillespi314 marked this pull request as ready for review January 30, 2026 22:09
@gillespi314 gillespi314 requested a review from a team as a code owner January 30, 2026 22:09
@gillespi314 gillespi314 merged commit 41a6658 into main Feb 2, 2026
50 checks passed
@gillespi314 gillespi314 deleted the 38898-flaky-test branch February 2, 2026 16:00
sgress454 added a commit that referenced this pull request Feb 25, 2026
…40536)

<!-- Add the related story/sub-task/bug number, like Resolves #123, or
remove if NA -->
**Related issue:** Resolves #40535

Fixes a flaky test (and possibly others) by turning off worker jobs
while the test is running. @gillespi314 did a similar fix recently in
#39106; this adds it to other
vulnerable tests including
`TestSetupExperienceFlowWithRequiredSoftwareVPP` which I recently got a
failure on in CI.
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.

Flaky test: TestIntegrationsMDM/TestSetupExperienceFlowUpdateScript

2 participants