[ARVADOS] updated: 1.3.0-403-gd9e5d620d
Git user
git at public.curoverse.com
Thu Feb 28 16:48:58 EST 2019
Summary of changes:
sdk/cwl/arvados_cwl/arvcontainer.py | 2 +-
sdk/cwl/setup.py | 1 +
sdk/python/arvados/commands/keepdocker.py | 4 ++--
sdk/python/setup.py | 1 +
services/fuse/arvados_fuse/fresh.py | 2 +-
services/fuse/setup.py | 3 +++
6 files changed, 9 insertions(+), 4 deletions(-)
via d9e5d620d8e68b349bb79b9749a74110603a89d4 (commit)
via ffef38539dd36e224abb75f0480cff1deb319124 (commit)
from d0584563200b0fe69e508b2fc5b0ddb223ebccc0 (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 d9e5d620d8e68b349bb79b9749a74110603a89d4
Author: Eric Biagiotti <ebiagiotti at veritasgenetics.com>
Date: Thu Feb 28 15:18:55 2019 -0500
14885: Adds python2 conditional pytz import
Arvados-DCO-1.1-Signed-off-by: Eric Biagiotti <ebiagiotti at veritasgenetics.com>
diff --git a/sdk/cwl/setup.py b/sdk/cwl/setup.py
index 20a601802..d97e7428d 100644
--- a/sdk/cwl/setup.py
+++ b/sdk/cwl/setup.py
@@ -43,6 +43,7 @@ setup(name='arvados-cwl-runner',
],
extras_require={
':os.name=="posix" and python_version<"3"': ['subprocess32 >= 3.5.1'],
+ ':python_version<"3"': ['pytz'],
},
data_files=[
('share/doc/arvados-cwl-runner', ['LICENSE-2.0.txt', 'README.rst']),
diff --git a/sdk/python/setup.py b/sdk/python/setup.py
index f4b4012a0..ffca23495 100644
--- a/sdk/python/setup.py
+++ b/sdk/python/setup.py
@@ -57,6 +57,7 @@ setup(name='arvados-python-client',
],
extras_require={
':os.name=="posix" and python_version<"3"': ['subprocess32 >= 3.5.1'],
+ ':python_version<"3"': ['pytz'],
},
classifiers=[
'Programming Language :: Python :: 2',
diff --git a/services/fuse/setup.py b/services/fuse/setup.py
index f2e385fa7..9b4b997cd 100644
--- a/services/fuse/setup.py
+++ b/services/fuse/setup.py
@@ -45,6 +45,9 @@ setup(name='arvados_fuse',
'ciso8601 >= 2.0.0',
'setuptools'
],
+ extras_require={
+ ':python_version<"3"': ['pytz'],
+ },
test_suite='tests',
tests_require=['pbr<1.7.0', 'mock>=1.0', 'PyYAML'],
zip_safe=False
commit ffef38539dd36e224abb75f0480cff1deb319124
Author: Eric Biagiotti <ebiagiotti at veritasgenetics.com>
Date: Thu Feb 28 15:17:31 2019 -0500
14885: Removes as_naive datetime parsing distinction
parse_datetime_as_naive should only be used in instances where we want to parse a string that we know contains timezone info AND we do not want to include it in the resulting datetime object. Otherwise, parse_datetime is preferred for parsing strings with or without timezone info.
Arvados-DCO-1.1-Signed-off-by: Eric Biagiotti <ebiagiotti at veritasgenetics.com>
diff --git a/sdk/cwl/arvados_cwl/arvcontainer.py b/sdk/cwl/arvados_cwl/arvcontainer.py
index 461f5b66a..1261e94a5 100644
--- a/sdk/cwl/arvados_cwl/arvcontainer.py
+++ b/sdk/cwl/arvados_cwl/arvcontainer.py
@@ -342,7 +342,7 @@ class ArvadosContainer(JobBase):
if record["output_uuid"]:
if self.arvrunner.trash_intermediate or self.arvrunner.intermediate_output_ttl:
# Compute the trash time to avoid requesting the collection record.
- trash_at = ciso8601.parse_datetime_as_naive(record["modified_at"]) + datetime.timedelta(0, self.arvrunner.intermediate_output_ttl)
+ trash_at = ciso8601.parse_datetime(record["modified_at"]) + datetime.timedelta(0, self.arvrunner.intermediate_output_ttl)
aftertime = " at %s" % trash_at.strftime("%Y-%m-%d %H:%M:%S UTC") if self.arvrunner.intermediate_output_ttl else ""
orpart = ", or" if self.arvrunner.trash_intermediate and self.arvrunner.intermediate_output_ttl else ""
oncomplete = " upon successful completion of the workflow" if self.arvrunner.trash_intermediate else ""
diff --git a/sdk/python/arvados/commands/keepdocker.py b/sdk/python/arvados/commands/keepdocker.py
index ec2a9942a..e596e6691 100644
--- a/sdk/python/arvados/commands/keepdocker.py
+++ b/sdk/python/arvados/commands/keepdocker.py
@@ -230,12 +230,12 @@ def docker_link_sort_key(link):
Docker metadata links to sort them from least to most preferred.
"""
try:
- image_timestamp = ciso8601.parse_datetime_as_naive(
+ image_timestamp = ciso8601.parse_datetime(
link['properties']['image_timestamp'])
except (KeyError, ValueError):
image_timestamp = EARLIEST_DATETIME
try:
- created_timestamp = ciso8601.parse_datetime_as_naive(link['created_at'])
+ created_timestamp = ciso8601.parse_datetime(link['created_at'])
except ValueError:
created_timestamp = None
return (image_timestamp, created_timestamp)
diff --git a/services/fuse/arvados_fuse/fresh.py b/services/fuse/arvados_fuse/fresh.py
index 2e7a2a818..acebe2b1b 100644
--- a/services/fuse/arvados_fuse/fresh.py
+++ b/services/fuse/arvados_fuse/fresh.py
@@ -12,7 +12,7 @@ def convertTime(t):
if not t:
return 0
try:
- return calendar.timegm(ciso8601.parse_datetime_as_naive(t).timetuple())
+ return calendar.timegm(ciso8601.parse_datetime(t).timetuple())
except (TypeError, ValueError):
return 0
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list