X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/05b52b297b30d075ef2409a123f7d096c1156cf8..73812c5a32581a7f72922dcd2e448e4d4ba4d918:/services/nodemanager/arvnodeman/computenode/__init__.py diff --git a/services/nodemanager/arvnodeman/computenode/__init__.py b/services/nodemanager/arvnodeman/computenode/__init__.py index 6e46bc0f4c..3921e096d8 100644 --- a/services/nodemanager/arvnodeman/computenode/__init__.py +++ b/services/nodemanager/arvnodeman/computenode/__init__.py @@ -3,6 +3,7 @@ from __future__ import absolute_import, print_function import calendar +import functools import itertools import re import time @@ -43,6 +44,66 @@ def arvados_node_missing(arvados_node, fresh_time): else: return not timestamp_fresh(arvados_timestamp(arvados_node["last_ping_at"]), fresh_time) +class RetryMixin(object): + """Retry decorator for an method that makes remote requests. + + Use this function to decorate method, and pass in a tuple of exceptions to + catch. If the original method raises a known cloud driver error, or any of + the given exception types, this decorator will either go into a + sleep-and-retry loop with exponential backoff either by sleeping (if + self._timer is None) or by scheduling retries of the method (if self._timer + is a timer actor.) + + """ + def __init__(self, retry_wait, max_retry_wait, + logger, cloud, timer=None): + self.min_retry_wait = retry_wait + self.max_retry_wait = max_retry_wait + self.retry_wait = retry_wait + self._logger = logger + self._cloud = cloud + self._timer = timer + + @staticmethod + def _retry(errors=()): + def decorator(orig_func): + @functools.wraps(orig_func) + def retry_wrapper(self, *args, **kwargs): + while True: + try: + ret = orig_func(self, *args, **kwargs) + except Exception as error: + if not (isinstance(error, errors) or + self._cloud.is_cloud_exception(error)): + self.retry_wait = self.min_retry_wait + raise + + self._logger.warning( + "Client error: %s - waiting %s seconds", + error, self.retry_wait, exc_info=error) + + if self._timer: + start_time = time.time() + # reschedule to be called again + self._timer.schedule(start_time + self.retry_wait, + getattr(self._later, + orig_func.__name__), + *args, **kwargs) + else: + # sleep on it. + time.sleep(self.retry_wait) + + self.retry_wait = min(self.retry_wait * 2, + self.max_retry_wait) + if self._timer: + # expect to be called again by timer so don't loop + return + else: + self.retry_wait = self.min_retry_wait + return ret + return retry_wrapper + return decorator + class ShutdownTimer(object): """Keep track of a cloud node's shutdown windows.