Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
fix: mtime inventory for google storage was accidentally setting a fl…
…oat instead of a proper mtime object (#1484)
  • Loading branch information
johanneskoester committed Mar 16, 2022
1 parent 013a6e8 commit 7c762c7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions snakemake/remote/GS.py
Expand Up @@ -171,7 +171,7 @@ async def inventory(self, cache: snakemake.io.IOCache):
iterate over the entire bucket once (and then not need to again).
This includes:
- cache.exist_remote
- cache_mtime
- cache.mtime
- cache.size
"""
if cache.remaining_wait_time <= 0:
Expand All @@ -184,7 +184,7 @@ async def inventory(self, cache: snakemake.io.IOCache):
# By way of being listed, it exists. mtime is a datetime object
name = "{}/{}".format(blob.bucket.name, blob.name)
cache.exists_remote[name] = True
cache.mtime[name] = blob.updated.timestamp()
cache.mtime[name] = snakemake.io.Mtime(remote=blob.updated.timestamp())
cache.size[name] = blob.size

cache.remaining_wait_time -= time.time() - start_time
Expand Down

0 comments on commit 7c762c7

Please sign in to comment.