Skip to content

Commit a64bb60

Browse files
Gupta, SuryaGupta, Surya
authored andcommitted
[CSTACKEX-132] VM create operation is stuck if desired protocol is not enabled on Host
1 parent 8ae97bf commit a64bb60

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/UnifiedSANStrategy.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,7 @@ public AccessGroup createAccessGroup(AccessGroup accessGroup) {
198198
// Check if all hosts support the protocol
199199
if (!validateProtocolSupport(accessGroup.getHostsToConnect(), protocol)) {
200200
String errMsg = "Not all hosts " + " support the protocol: " + protocol.name();
201+
logger.error(errMsg);
201202
throw new CloudRuntimeException(errMsg);
202203
}
203204

@@ -331,7 +332,9 @@ public void deleteAccessGroup(AccessGroup accessGroup) {
331332
private boolean validateProtocolSupport(List<HostVO> hosts, ProtocolType protocolType) {
332333
String protocolPrefix = OntapStorageConstants.IQN;
333334
for (HostVO host : hosts) {
335+
logger.info("validateProtocolSupportAndFetchHostsIdentifier: Validating host {} for protocol {} and IQN {}", host.getName(), protocolType.name(), host.getStorageUrl());
334336
if (host == null || host.getStorageUrl() == null || host.getStorageUrl().trim().isEmpty() || !host.getStorageUrl().startsWith(protocolPrefix)) {
337+
logger.info("validateProtocolSupport return false: Validating host {} for protocol {} and IQN {}", host.getName(), protocolType.name(), host.getStorageUrl());
335338
return false;
336339
}
337340
}

0 commit comments

Comments
 (0)