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

Hypershift NodePool AllMachinesReady and AllNodesHealthy should have message conditions ordered

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done-Errata
    • Major
    • 4.14.0
    • 4.12
    • HyperShift
    • No
    • Hypershift Sprint 239
    • 1
    • Rejected
    • False
    • Hide

      None

      Show
      None

    Description

      Description of problem:

      
      NodePool conditions AllMachinesReady and AllNodesHealthy are used by Cluster Service to detect problems on customer nodes.
      
      Everytime a NodePool is updated, it triggers an update in a ManifestWork that is being processed by CS to build a user message about why a specific machinepool/nodepool is not healthy.
      
      The lack of a sorted message when there are more than one machines creates a bug that the NodePool is updated multiple time, when the state is the same.
      
      For example, CS may capture scenarios like this and consider them like the change is the same.
      
      Machine rosa-vws58-workshop-69b55d58b-mq44p: UnhealthyNode
      Machine rosa-vws58-workshop-69b55d58b-97n47: UnhealthyNode ,
      Machine rosa-vws58-workshop-69b55d58b-mq44p: NodeConditionsFailed
      Machine rosa-vws58-workshop-69b55d58b-97n47: Deleting ,
      
      Machine rosa-vws58-workshop-69b55d58b-97n47: UnhealthyNode
      Machine rosa-vws58-workshop-69b55d58b-mq44p: UnhealthyNode ,
      Machine rosa-vws58-workshop-69b55d58b-97n47: Deleting
      Machine rosa-vws58-workshop-69b55d58b-mq44p: NodeConditionsFailed ,
      
      Machine rosa-vws58-workshop-69b55d58b-mq44p: UnhealthyNode
      Machine rosa-vws58-workshop-69b55d58b-97n47: UnhealthyNode ,
      Machine rosa-vws58-workshop-69b55d58b-mq44p: NodeConditionsFailed
      Machine rosa-vws58-workshop-69b55d58b-97n47: Deleting ,
      

      Expected results:

      
      The HyperShift Operator should sort the messages where multiple machines/nodes are invovled:
      
      https://github.com/openshift/hypershift/blob/86af31a5a5cdee3da0d7f65f3bd550f4ec9cac55/hypershift-operator/controllers/nodepool/nodepool_controller.go#L2509
      
      

      Attachments

        Issue Links

          Activity

            People

              agarcial@redhat.com Alberto Garcia Lamela
              lponce@redhat.com Lucas Ponce
              Jie Zhao Jie Zhao
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: