PWX-32850: When preflight is unsupported we need to make sure we are …

…setting the pre-flight annotation to false. (#1195) Signed-off-by: Jose Rivera <jose@portworx.com>
libopenstorage · Aug 10, 2023 · cf05dbb · cf05dbb
1 parent 75fc9b7
commit cf05dbb
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/pkg/controller/storagecluster/storagecluster.go b/pkg/controller/storagecluster/storagecluster.go
@@ -742,12 +742,11 @@ func (c *Controller) syncStorageCluster(
 			return fmt.Errorf("preflight check failed for StorageCluster %v/%v: %v", cluster.Namespace, cluster.Name, err)
 		}
 	} else {
-		if _, exists := cluster.Annotations[pxutil.AnnotationPreflightCheck]; !exists {
-			if cluster.Annotations == nil {
-				cluster.Annotations = make(map[string]string)
-			}
-			cluster.Annotations[pxutil.AnnotationPreflightCheck] = "false"
+		// Always disable preflight if not supported.
+		if cluster.Annotations == nil {
+			cluster.Annotations = make(map[string]string)
 		}
+		cluster.Annotations[pxutil.AnnotationPreflightCheck] = "false"
 	}
 
 	if err := c.miscCleanUp(cluster); err != nil {