Repository: jclouds-labs
Updated Branches:
  refs/heads/2.0.x 73af05acd -> 6c1e9f2ce


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/6c1e9f2c
Tree: http://git-wip-us.apache.org/repos/asf/jclouds-labs/tree/6c1e9f2c
Diff: http://git-wip-us.apache.org/repos/asf/jclouds-labs/diff/6c1e9f2c

Branch: refs/heads/2.0.x
Commit: 6c1e9f2ce1b5953455328a2b4514b7a620f0a56a
Parents: 73af05a
Author: Svetoslav Neykov <[email protected]>
Authored: Thu Jul 6 10:51:27 2017 +0300
Committer: Svetoslav Neykov <[email protected]>
Committed: Fri Jul 7 10:38:50 2017 +0300

----------------------------------------------------------------------
 .../extensions/AzureComputeSecurityGroupExtension.java    | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds-labs/blob/6c1e9f2c/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

Reply via email to