some retry testing

allmightyspiff · allmightyspiff · commit 26e313fa762d · 2017-11-07T12:17:53.000-06:00
diff --git a/SoftLayer/managers/vs.py b/SoftLayer/managers/vs.py
@@ -13,10 +13,12 @@
 import time
 import warnings
 
+from SoftLayer.decoration import retry
 from SoftLayer import exceptions
 from SoftLayer.managers import ordering
 from SoftLayer import utils
 
+
 LOGGER = logging.getLogger(__name__)
 # pylint: disable=no-self-use
 
@@ -584,9 +586,17 @@ def create_instance(self, **kwargs):
         tags = kwargs.pop('tags', None)
         inst = self.guest.createObject(self._generate_create_dict(**kwargs))
         if tags is not None:
-            self.guest.setTags(tags, id=inst['id'])
+            self.set_tags(tags, guest_id=inst['id'])
         return inst
 
+    @retry(exceptions.SoftLayerAPIError, logger=LOGGER)
+    def set_tags(self, tags, guest_id):
+        """Sets tags on a guest with a retry decorator
+
+        Just calls guest.setTags, but lets if it fails from an APIError will retry
+        """
+        self.guest.setTags(tags, id=guest_id)
+
     def create_instances(self, config_list):
         """Creates multiple virtual server instances.
 
@@ -636,7 +646,7 @@ def create_instances(self, config_list):
 
         for instance, tag in zip(resp, tags):
             if tag is not None:
-                self.guest.setTags(tag, id=instance['id'])
+                self.set_tags(tag, guest_id=instance['id'])
 
         return resp
 
@@ -717,7 +727,7 @@ def edit(self, instance_id, userdata=None, hostname=None, domain=None,
             self.guest.setUserMetadata([userdata], id=instance_id)
 
         if tags is not None:
-            self.guest.setTags(tags, id=instance_id)
+            self.set_tags(tags, guest_id=instance_id)
 
         if hostname:
             obj['hostname'] = hostname
