Skip to content

Fix 9 flaky tests identified in master nightly runs#68893

Open
Copilot wants to merge 10 commits intomasterfrom
copilot/identify-flaky-tests
Open

Fix 9 flaky tests identified in master nightly runs#68893
Copilot wants to merge 10 commits intomasterfrom
copilot/identify-flaky-tests

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 4, 2026

  • Add pytest.mark.timeout(300) to test_state_running
  • Fix test_reauth: reduce default master-down wait from 150s to 60s (saves ~90s); add fast recon settings (recon_default=100ms, recon_max=1000ms) to minion so reconnect cycles are faster
  • Fix queue conftest: reduce parameterization from 4 to 2 sets (remove unlimited variants that don't reliably trigger queueing and were causing assertion failures); cuts queue test runtime by ~50%

Copilot AI assigned Copilot and dwoz Apr 4, 2026
@dwoz dwoz added the test:full Run the full test suite label Apr 4, 2026
Copilot AI requested a review from dwoz April 4, 2026 05:31
Copilot finished work on behalf of dwoz April 4, 2026 05:31
@dwoz dwoz removed the test:full Run the full test suite label Apr 4, 2026
@dwoz dwoz added the test:full Run the full test suite label Apr 4, 2026
Copilot finished work on behalf of dwoz April 4, 2026 06:47
Copilot AI and others added 7 commits April 4, 2026 21:24
…ener cleanup, orchestrate race, queue timeout

Agent-Logs-Url: https://github.com/saltstack/salt/sessions/655b4e54-fa1b-4a03-88fd-542afc927254

Co-authored-by: dwoz <1527763+dwoz@users.noreply.github.com>
… race

The compound match policy G@testgrain:foo uses match.compound_matches
with greedy=False (no uncached minions). When the x509 minion just
started, its grains may not yet be in the master's cache, causing
"minion not permitted to use specified signing policy". Retry up to 5
times with a 3s sleep to allow the grain cache to populate.

Agent-Logs-Url: https://github.com/saltstack/salt/sessions/ff893826-902c-49d0-a5db-adf1de40e546

Co-authored-by: dwoz <1527763+dwoz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants