Uploaded image for project: 'OpenShift Windows Containers'
  1. OpenShift Windows Containers
  2. WINC-1709

Fix OCP-60944: Add Flexy bastion discovery and SSH validation for platform 'none'

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Unresolved
    • Icon: Undefined Undefined
    • None
    • None
    • None
    • None
    • None
    • False
    • Hide

      None

      Show
      None
    • False
    • None
    • None
    • None
    • None

      Problem

      Test OCP-60944 (WICD service reconciliation) fails with SSH timeout errors on platform 'none' clusters because:

      • Code only checked QE_BASTION_PUBLIC_ADDRESS environment variable
      • Jenkins/Flexy sets BASTION instead (different naming convention)
      • Users get cryptic 8-minute timeout errors with no helpful guidance

      Solution

      1. Support Flexy Environment Variables

      • Added support for both BASTION and QE_BASTION_PUBLIC_ADDRESS
      • Added support for both BASTION_SSH_USER and QE_BASTION_SSH_USER
      • Now works automatically in Jenkins/Flexy environments

      2. Improved Error Messages

      • Detailed bastion discovery summary for platform 'none'
      • Actionable setup instructions when bastion not found
      • Better developer experience

      Testing Results

      Tested locally with BASTION env var set:

      • ✅ Test passes successfully (2m46s)
      • ✅ Bastion detection working
      • ✅ SSH connections routing through bastion correctly
      • ✅ WICD reconciliation verified on both Windows nodes

      Impact

      • No more SSH timeout failures in Jenkins/Flexy
      • Better user experience with actionable error messages
      • No configuration changes needed for existing Jenkins/Flexy setups

      Files Changed

      • test/extended/winc/utils.go: Bastion discovery logic (27 additions, 5 deletions)

      Related Work

      • Complements PR #29256 (bastion helper scripts for deployment)
      • Both PRs together provide complete bastion support for WINC tests

              Unassigned Unassigned
              rrasouli Aharon Rasouli
              None
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: