From 757fc291c1a3a793e9d4acdfe90e27c87ac7c53e Mon Sep 17 00:00:00 2001
From: Etienne CARRIERE <etienne.carriere@gmail.com>
Date: Thu, 2 Jul 2015 07:35:19 +0200
Subject: [PATCH] Rework on Exception management

---
 network/f5/bigip_virtual_server.py | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/network/f5/bigip_virtual_server.py b/network/f5/bigip_virtual_server.py
index 09e9bf0beb5..84cfccf2958 100644
--- a/network/f5/bigip_virtual_server.py
+++ b/network/f5/bigip_virtual_server.py
@@ -176,16 +176,23 @@ def vs_exists(api, vs):
 
 def vs_create(api,name,destination,port,pool):
     _profiles=[[{'profile_context': 'PROFILE_CONTEXT_TYPE_ALL', 'profile_name': 'tcp'}]]
+    created = False
+    # a bit of a hack to handle concurrent runs of this module.
+    # even though we've checked the vs doesn't exist,
+    # it may exist by the time we run create_vs().
+    # this catches the exception and does something smart
+    # about it!
     try:
         api.LocalLB.VirtualServer.create(
             definitions = [{'name': [name], 'address': [destination], 'port': port, 'protocol': 'PROTOCOL_TCP'}],
             wildmasks = ['255.255.255.255'],
             resources = [{'type': 'RESOURCE_TYPE_POOL', 'default_pool_name': pool}],
             profiles = _profiles)
-        result = True 
-        desc = 0
-    except Exception, e :
-        print e.args
+        created = True
+        return created
+    except bigsudsOperationFailed, e :
+        if "already exists" not in str(e):
+            raise Exception('Error on creating Virtual Server : %s' % e)
 
 def vs_remove(api,name):
     api.LocalLB.VirtualServer.delete_virtual_server(virtual_servers = [name ])
@@ -351,16 +358,12 @@ def main():
                     # about it!
                     try:
                         vs_create(api,name,destination,port,pool)
-                        result = {'changed': True}
-                    except bigsuds.OperationFailed, e:
-                        if "already exists" in str(e):
-                            update = True
-                        else:
-                            raise Exception('Error on creating Virtual Server : %s' % e)
-                    else:
                         set_profiles(api,name,all_profiles)
                         set_snat(api,name,snat)
                         set_description(api,name,description)
+                        result = {'changed': True}
+                    except bigsuds.OperationFailed, e:
+                            raise Exception('Error on creating Virtual Server : %s' % e)
                 else:
                     # check-mode return value
                     result = {'changed': True}