fix: Remove code that is expected to never run

Author: dlipovetsky

pkg/webhook/preflight/nutanix/storagecontainer.go

@@ -35,17 +35,6 @@ func (c *storageContainerCheck) Run(ctx context.Context) preflight.CheckResult {
 		Allowed: true,
 	}
 
-	if c.csiSpec == nil {
-		result.Allowed = false
-		result.InternalError = true
-		result.Causes = append(result.Causes, preflight.Cause{
-			Message: "Nutanix CSI Provider configuration is missing",
-			Field:   c.field,
-		})
-
-		return result
-	}
-
 	if c.csiSpec.StorageClassConfigs == nil {
 		result.Allowed = false
 		result.Causes = append(result.Causes, preflight.Cause{

pkg/webhook/preflight/nutanix/storagecontainer_test.go

@@ -264,20 +264,6 @@ func TestStorageContainerCheck(t *testing.T) {
 		expectedError        bool
 		expectedCauseMessage string
 	}{
-		{
-			name: "nil CSI spec",
-			machineSpec: &carenv1.NutanixMachineDetails{
-				Cluster: capxv1.NutanixResourceIdentifier{
-					Type: capxv1.NutanixIdentifierName,
-					Name: ptr.To(clusterName),
-				},
-			},
-			csiSpec:              nil,
-			nclient:              nil,
-			expectedAllowed:      false,
-			expectedError:        true,
-			expectedCauseMessage: "Nutanix CSI Provider configuration is missing",
-		},
 		{
 			name: "nil storage class configs",
 			machineSpec: &carenv1.NutanixMachineDetails{