CSTACKEX-36 Added TODOs

Author: Gupta, Surya

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

@@ -60,7 +60,7 @@
 import java.util.HashMap;
 import java.util.Map;
 
-public abstract class OntapPrimaryDatastoreDriver implements PrimaryDataStoreDriver {
+public class OntapPrimaryDatastoreDriver implements PrimaryDataStoreDriver {
 
     private static final Logger s_logger = LogManager.getLogger(OntapPrimaryDatastoreDriver.class);
 

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/lifecycle/OntapPrimaryDatastoreLifecycle.java

@@ -200,6 +200,7 @@ public boolean attachCluster(DataStore dataStore, ClusterScope scope) {
         if (!isProtocolSupportedByAllHosts(hostsToConnect, protocol, hostsIdentifier)) {
             throw new CloudRuntimeException("Not all hosts in the cluster support the protocol: " + protocol.toString());
         }
+        //TODO - check if no host to connect then also need to create access group without initiators
         if (hostsIdentifier != null && hostsIdentifier.size() > 0) {
             AccessGroup accessGroupRequest = utils.createAccessGroupRequestByProtocol(storagePool, scope.getScopeId(), details, hostsIdentifier);
             strategy.createAccessGroup(accessGroupRequest);