Changed if condition order

Signed-off-by: Niladri Halder <[email protected]>
openebs · May 1, 2021 · aa46f93 · aa46f93
1 parent 5145478
commit aa46f93
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/cmd/provisioner-localpv/app/provisioner.go b/cmd/provisioner-localpv/app/provisioner.go
@@ -124,15 +124,21 @@ func (p *Provisioner) Provision(opts pvController.ProvisionOptions) (*v1.Persist
 		size = pvc.Spec.Resources.Requests["storage"]
 	}
 	sendEventOrIgnore(pvc.Name, name, size.String(), stgType, analytics.VolumeProvision)
-	if stgType == "hostpath" {
-		return p.ProvisionHostPath(opts, pvCASConfig)
-	}
+
+	// StorageType: Device
 	if stgType == "device" {
 		return p.ProvisionBlockDevice(opts, pvCASConfig)
 	}
+
+	// EXCEPTION: Block VolumeMode
 	if *opts.PVC.Spec.VolumeMode == v1.PersistentVolumeBlock && stgType != "device" {
 		return nil, fmt.Errorf("PV with BlockMode is not supported with StorageType %v", stgType)
 	}
+
+	// StorageType: Hostpath
+	if stgType == "hostpath" {
+		return p.ProvisionHostPath(opts, pvCASConfig)
+	}
 	alertlog.Logger.Errorw("",
 		"eventcode", "local.pv.provision.failure",
 		"msg", "Failed to provision Local PV",