Skip to content

Commit 8eb6ddd

Browse files
Pearl1594nvazquez
andauthored
Fix inactive VPCs deletion (#11618)
Co-authored-by: Nicolas Vazquez <[email protected]>
1 parent 0526ec0 commit 8eb6ddd

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

plugins/network-elements/netris/src/main/java/org/apache/cloudstack/service/NetrisApiClientImpl.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1131,16 +1131,17 @@ private VPCResponseObjectOK deleteVpcInternal(VPCListing vpcResource) {
11311131
public boolean deleteVpc(DeleteNetrisVpcCommand cmd) {
11321132
String suffix = String.valueOf(cmd.getId());
11331133
String vpcName = NetrisResourceObjectUtils.retrieveNetrisResourceObjectName(cmd, NetrisResourceObjectUtils.NetrisObjectType.VPC);
1134-
VPCListing vpcResource = getVpcByNameAndTenant(vpcName);
1135-
if (vpcResource == null) {
1136-
logger.error("Could not find the Netris VPC resource with name {} and tenant ID {}", vpcName, tenantId);
1137-
return false;
1138-
}
11391134
String snatRuleName = NetrisResourceObjectUtils.retrieveNetrisResourceObjectName(cmd, NetrisResourceObjectUtils.NetrisObjectType.SNAT, suffix);
11401135
NatGetBody existingNatRule = netrisNatRuleExists(snatRuleName);
11411136
boolean ruleExists = Objects.nonNull(existingNatRule);
11421137
if (ruleExists) {
1143-
deleteNatRule(snatRuleName, existingNatRule.getId(), vpcResource.getName());
1138+
deleteNatRule(snatRuleName, existingNatRule.getId(), vpcName);
1139+
}
1140+
1141+
VPCListing vpcResource = getVpcByNameAndTenant(vpcName);
1142+
if (vpcResource == null) {
1143+
logger.warn("The Netris VPC resource with name {} and tenant ID {} does not exist, cannot be removed", vpcName, tenantId);
1144+
return true;
11441145
}
11451146

11461147
String vpcAllocationName = NetrisResourceObjectUtils.retrieveNetrisResourceObjectName(cmd, NetrisResourceObjectUtils.NetrisObjectType.IPAM_ALLOCATION, cmd.getCidr());

0 commit comments

Comments
 (0)