CSTACKEX-112 Resolve Review Comments

Author: Gupta, Surya

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/provider/OntapPrimaryDatastoreProvider.java

@@ -28,6 +28,7 @@
 import org.apache.cloudstack.storage.driver.OntapPrimaryDatastoreDriver;
 import org.apache.cloudstack.storage.lifecycle.OntapPrimaryDatastoreLifecycle;
 import org.apache.cloudstack.storage.listener.OntapHostListener;
+import org.apache.cloudstack.storage.utils.Constants;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.springframework.stereotype.Component;
@@ -65,7 +66,7 @@ public HypervisorHostListener getHostListener() {
     @Override
     public String getName() {
         s_logger.trace("OntapPrimaryDatastoreProvider: getName: Called");
-        return "NetApp ONTAP";
+        return Constants.NETAPP_ONTAP_PROVIDER;
     }
 
     @Override

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/utils/Constants.java

@@ -22,6 +22,7 @@
 
 public class Constants {
 
+    public static final String NETAPP_ONTAP_PROVIDER = "NetApp ONTAP";
     public static final String NFS = "nfs";
     public static final String ISCSI = "iscsi";
     public static final String SIZE = "size";