Repository: jclouds-labs Updated Branches: refs/heads/master 2495dd9a6 -> 1450892fd
Fix SGE.removeSecurityGroup for when the SG doesn't exist Project: http://git-wip-us.apache.org/repos/asf/jclouds-labs/repo Commit: http://git-wip-us.apache.org/repos/asf/jclouds-labs/commit/1450892f Tree: http://git-wip-us.apache.org/repos/asf/jclouds-labs/tree/1450892f Diff: http://git-wip-us.apache.org/repos/asf/jclouds-labs/diff/1450892f Branch: refs/heads/master Commit: 1450892fd841b683fc5da5acb38a853b5e45605b Parents: 2495dd9 Author: Svetoslav Neykov <[email protected]> Authored: Thu Jul 6 10:51:27 2017 +0300 Committer: Svetoslav Neykov <[email protected]> Committed: Fri Jul 7 10:37:03 2017 +0300 ---------------------------------------------------------------------- .../extensions/AzureComputeSecurityGroupExtension.java | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jclouds-labs/blob/1450892f/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/compute/extensions/AzureComputeSecurityGroupExtension.java ---------------------------------------------------------------------- diff --git a/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/compute/extensions/AzureComputeSecurityGroupExtension.java b/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/compute/extensions/AzureComputeSecurityGroupExtension.java index 50cb75a..eaf820c 100644 --- a/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/compute/extensions/AzureComputeSecurityGroupExtension.java +++ b/azurecompute-arm/src/main/java/org/jclouds/azurecompute/arm/compute/extensions/AzureComputeSecurityGroupExtension.java @@ -190,7 +190,15 @@ public class AzureComputeSecurityGroupExtension implements SecurityGroupExtensio final ResourceGroupAndName resourceGroupAndName = ResourceGroupAndName.fromSlashEncoded(id); URI uri = api.getNetworkSecurityGroupApi(resourceGroupAndName.resourceGroup()) .delete(resourceGroupAndName.name()); - return resourceDeleted.apply(uri); + + // https://docs.microsoft.com/en-us/rest/api/network/virtualnetwork/delete-a-network-security-group + if (uri != null) { + // 202-Accepted if resource exists and the request is accepted. + return resourceDeleted.apply(uri); + } else { + // 204-No Content if resource does not exist. + return false; + } } @Override
