Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-76350

HCP: hypershift-operator ignores AWS AMI override in NodePool specification causing failures in AWS ISO (classified) regions

    • None
    • False
    • Hide

      None

      Show
      None
    • None
    • Important
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Description of problem:

      Hypershift hosted clusters in AWS ISO (classified) regions fail to honor AMI overrides in NodePool specifications causing node pool deployments to fail.  
      
      The offending code appears to be at https://github.com/openshift/hypershift/blob/main/hypershift-operator/controllers/nodepool/token.go#L137, which fails to take into account the AMI override defined in nodePool.Spec.Platform.AWS.AMI.  
      
      Additionally, there's no provision for handling the lack of release image metadata in the ISO regions at https://github.com/openshift/hypershift/blob/main/hypershift-operator/controllers/nodepool/nodepool_controller.go#L723-L726.
      
      An example of where the AMI override *is* being honored can be seen at https://github.com/openshift/hypershift/blob/main/hypershift-operator/controllers/nodepool/aws.go#L56-L57.
      
      This is preventing a customer from moving forward with their HCP implementation.

      Version-Release number of selected component (if applicable):

          

      How reproducible:

      Consistently fails when attempting to deploy in AWS ISO (classified) regions.    

      Steps to Reproduce:

          1.
          2.
          3.
          

      Actual results:

          

      Expected results:

          

      Additional info:

          

              hypershift-automation hypershift-team automation
              rh-ee-cwigal Chad Wigal
              None
              None
              Jim Ma Jim Ma
              None
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated: