[ARVADOS] updated: c12be4d7070ee3de978c1426c68071fb53f1dabd
Git user
git at public.curoverse.com
Tue Jun 14 10:11:10 EDT 2016
Summary of changes:
services/nodemanager/arvnodeman/computenode/driver/__init__.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
via c12be4d7070ee3de978c1426c68071fb53f1dabd (commit)
from 4d89b138dff0f1bcf1c867d529aff599fcc0cda2 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit c12be4d7070ee3de978c1426c68071fb53f1dabd
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date: Tue Jun 14 09:57:54 2016 -0400
9353: Don't retry all BaseHTTPError, only retry InvalidInstanceID.NotFound.
diff --git a/services/nodemanager/arvnodeman/computenode/driver/__init__.py b/services/nodemanager/arvnodeman/computenode/driver/__init__.py
index 63e4c00..db799bc 100644
--- a/services/nodemanager/arvnodeman/computenode/driver/__init__.py
+++ b/services/nodemanager/arvnodeman/computenode/driver/__init__.py
@@ -25,7 +25,7 @@ class BaseComputeNodeDriver(RetryMixin):
Subclasses must implement arvados_create_kwargs, sync_node,
node_fqdn, and node_start_time.
"""
- CLOUD_ERRORS = NETWORK_ERRORS + (cloud_types.LibcloudError, BaseHTTPError)
+ CLOUD_ERRORS = NETWORK_ERRORS + (cloud_types.LibcloudError,)
@RetryMixin._retry()
def _create_driver(self, driver_class, **auth_kwargs):
@@ -211,6 +211,9 @@ class BaseComputeNodeDriver(RetryMixin):
# libcloud compute drivers typically raise bare Exceptions to
# represent API errors. Return True for any exception that is
# exactly an Exception, or a better-known higher-level exception.
+ if (exception is BaseHTTPError and
+ self.message and self.message.startswith("InvalidInstanceID.NotFound")):
+ return True
return (isinstance(exception, cls.CLOUD_ERRORS) or
type(exception) is Exception)
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list