ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-02-07 13:23:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix check permissions fail in HTTPDriver (issue #394)

Browse Source
This commit is contained in:
allegroai 2021-07-23 16:00:23 +03:00
parent 900ee9de5a
commit 82009a5935
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
clearml/storage/helper.py
View File

@ -759,7 +759,7 @@ class StorageHelper(object):
return self._driver.delete_object(self._get_object(path)) return self._driver.delete_object(self._get_object(path))
def check_write_permissions(self, dest_path=None): def check_write_permissions(self, dest_path=None):
# create a temporary file, then de;ete it # create a temporary file, then delete it
base_url = dest_path or self._base_url base_url = dest_path or self._base_url
dest_path = base_url + '/.clearml.test' dest_path = base_url + '/.clearml.test'
# do not check http/s connection permissions # do not check http/s connection permissions
@ -1055,7 +1055,8 @@ class _HttpDriver(_Driver):
container = self._containers[obj.container_name] container = self._containers[obj.container_name]
res = container.session.delete(obj.url, headers=container.get_headers(obj.url)) res = container.session.delete(obj.url, headers=container.get_headers(obj.url))
if res.status_code != requests.codes.ok: if res.status_code != requests.codes.ok:
raise ValueError('Failed deleting object %s (%d): %s' % (obj.object_name, res.status_code, res.text)) log.warning('Failed deleting object %s (%d): %s' % (obj.object_name, res.status_code, res.text))
return False
return True return True
def get_object(self, container_name, object_name, *args, **kwargs): def get_object(self, container_name, object_name, *args, **kwargs):