Fixed an issue with trying to parse VirtualIPs for all nodes

Project: http://git-wip-us.apache.org/repos/asf/libcloud/repo
Commit: http://git-wip-us.apache.org/repos/asf/libcloud/commit/c9ce29ed
Tree: http://git-wip-us.apache.org/repos/asf/libcloud/tree/c9ce29ed
Diff: http://git-wip-us.apache.org/repos/asf/libcloud/diff/c9ce29ed

Branch: refs/heads/trunk
Commit: c9ce29eded3eb762553d2ea68cef1574a09c717f
Parents: b7eb06f
Author: Michael Bennett <[email protected]>
Authored: Wed Oct 29 10:50:30 2014 -0400
Committer: Michael Bennett <[email protected]>
Committed: Wed Nov 19 13:03:16 2014 -0500

----------------------------------------------------------------------
 libcloud/compute/drivers/azure.py | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/libcloud/blob/c9ce29ed/libcloud/compute/drivers/azure.py
----------------------------------------------------------------------
diff --git a/libcloud/compute/drivers/azure.py 
b/libcloud/compute/drivers/azure.py
index 501b083..f5e0524 100644
--- a/libcloud/compute/drivers/azure.py
+++ b/libcloud/compute/drivers/azure.py
@@ -286,7 +286,10 @@ class AzureNodeDriver(NodeDriver):
 
         data = self._parse_response(response, HostedService)
 
-        vips = [vip.address for vip in data.deployments[0].virtual_ips]
+        vips = None
+
+        if data.deployments[0].virtual_ips is not None:
+            vips = [vip.address for vip in data.deployments[0].virtual_ips]
 
         try:
             return [
@@ -735,22 +738,17 @@ class AzureNodeDriver(NodeDriver):
                 _deployment_name,
                 node.id
             )
-            path += '?comp=media'  # forces deletion of attached disks
-
-            self._perform_delete(path)
-
-            return True
         else:
             path = self._get_deployment_path_using_name(
                 ex_cloud_service_name,
                 _deployment_name
             )
 
-            path += '?comp=media'
+        path += '?comp=media'
 
-            self._perform_delete(path)
+        self._perform_delete(path)
 
-            return True
+        return True
 
     def create_cloud_service(self,
                              ex_cloud_service_name=None,

Reply via email to