Skip to content

xtask/tmt: increase SSH readiness window for composefs+grub+xfs integration VMs#2230

Closed
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-test-integration-job
Closed

xtask/tmt: increase SSH readiness window for composefs+grub+xfs integration VMs#2230
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-test-integration-job

Conversation

Copilot AI commented Jun 5, 2026

Copy link
Copy Markdown

The test-integration (centos-10, composefs, xfs, grub, bls, unsealed) Actions job was failing before plan execution because VM SSH readiness timed out (60 attempts) under this slower boot profile. This change makes SSH readiness retries profile-aware so this lane gets enough time to become reachable.

  • SSH retry policy

    • Added SSH_CONNECTIVITY_MAX_ATTEMPTS_EXTENDED = 120.
    • Kept existing default (60) for all other configurations.
  • Targeted profile selection

    • Added get_ssh_connectivity_attempts(&RunTmtArgs) to select extended retries only for:
      • composefs_backend = true
      • bootloader = grub
      • boot_type = bls
      • filesystem = xfs
      • seal_state != sealed
    • Wired run_tmt() to use the selected retry budget during VM SSH verification.
  • SSH verification API cleanup

    • Introduced verify_ssh_connectivity_with_attempts(...).
    • Retained verify_ssh_connectivity(...) as the default-path wrapper.
  • Focused unit coverage

    • Added tests for:
      • extended retry selection for composefs+grub+bls+xfs+unsealed
      • default retry selection for non-matching profiles
let ssh_attempts = get_ssh_connectivity_attempts(args);
verify_ssh_connectivity_with_attempts(sh, ssh_port, &key_path, ssh_attempts)?;

Copilot AI changed the title [WIP] Fix failing GitHub Actions job test-integration (centos-10, composefs, xfs, grub, bls, unsealed) xtask/tmt: increase SSH readiness window for composefs+grub+xfs integration VMs Jun 5, 2026
Copilot AI requested a review from Johan-Liebert1 June 5, 2026 07:04
@Johan-Liebert1

Copy link
Copy Markdown
Collaborator

:/

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.

2 participants