Skip to content

Commit

Permalink
After download is complete remove the remote job dir
Browse files Browse the repository at this point in the history
  • Loading branch information
sverhoeven committed Mar 8, 2024
1 parent 565e23b commit 5b3bc23
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 0 additions & 2 deletions src/bartender/filesystems/abstract.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,6 @@ async def delete(self, description: JobDescription) -> None:
Args:
description: Remote directory to delete.
"""
# after download or cancellation you might want to
# delete the remote job directory

async def close(self) -> None:
"""Close filesystem."""
2 changes: 1 addition & 1 deletion src/bartender/filesystems/queue.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ async def perform_download(

await filesystem.download(localized_description, description)

# TODO for non-local file system should also remove remote files?
await filesystem.delete(localized_description)

if job_id is not None:
await job_dao.update_job_state(job_id, state)
Expand Down

0 comments on commit 5b3bc23

Please sign in to comment.