ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-06-26 18:16:07 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix timeout for uploading images

Browse Source
This commit is contained in:
allegroai 2019-06-19 19:47:12 +03:00
parent cc536a8f3f
commit 77e2c32523
4 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
trains/backend_interface/task/task.py
View File

@ -74,6 +74,7 @@ class Task(IdObjectBase, AccessMixin, SetupUploadMixin):
:type force_create: bool
"""
task_id = self._resolve_task_id(task_id, log=log) if not force_create else None
self._edit_lock = RLock()
super(Task, self).__init__(id=task_id, session=session, log=log)
self._storage_uri = None
self._input_model = None
@ -85,7 +86,6 @@ class Task(IdObjectBase, AccessMixin, SetupUploadMixin):
self._parameters_allowed_types = (
six.string_types + six.integer_types + (six.text_type, float, list, dict, type(None))
)
self._edit_lock = RLock()
if not task_id:
# generate a new task

5
trains/storage/helper.py
View File

@ -965,6 +965,7 @@ class StorageHelper(object):
class _HttpDriver(object):
""" LibCloud http/https adapter (simple, enough for now) """
timeout = (5.0, None)
class _Container(object):
def __init__(self, name, retries=5, **kwargs):
@ -983,7 +984,7 @@ class _HttpDriver(object):
def upload_object_via_stream(self, iterator, container, object_name, extra=None, **kwargs):
url = object_name[:object_name.index('/')]
url_path = object_name[len(url)+1:]
res = container.session.post(container.name+url, files={url_path: iterator})
res = container.session.post(container.name+url, files={url_path: iterator}, timeout=self.timeout)
if res.status_code != requests.codes.ok:
raise ValueError('Failed uploading object %s (%d): %s' % (object_name, res.status_code, res.text))
return res
@ -998,7 +999,7 @@ class _HttpDriver(object):
container = self._containers[container_name]
# set stream flag before get request
container.session.stream = kwargs.get('stream', True)
res = container.session.get(''.join((container_name, object_name.lstrip('/'))))
res = container.session.get(''.join((container_name, object_name.lstrip('/'))), timeout=self.timeout)
if res.status_code != requests.codes.ok:
raise ValueError('Failed getting object %s (%d): %s' % (object_name, res.status_code, res.text))
return res

2
trains/task.py
View File

@ -89,6 +89,7 @@ class Task(_Task):
if private is not Task.__create_protection:
raise UsageError(
'Task object cannot be instantiated externally, use Task.current_task() or Task.get_task(...)')
self._lock = threading.RLock()
super(Task, self).__init__(**kwargs)
self._arguments = _Arguments(self)
@ -100,7 +101,6 @@ class Task(_Task):
self._dev_mode_periodic_flag = False
self._connected_parameter_type = None
self._detect_repo_async_thread = None
self._lock = threading.RLock()
# register atexit, so that we mark the task as stopped
self._at_exit_called = False
self.__register_at_exit(self._at_exit)

2
trains/utilities/async_manager.py
View File

@ -36,7 +36,7 @@ class AsyncManagerMixin(object):
if r.ready():
continue
t = time.time()
r.wait(timeout=remaining or 2.0)
r.wait(timeout=remaining)
count += 1
if max_num_uploads is not None and max_num_uploads - count <= 0:
break