Skip to content

Commit

Permalink
datetime and timezone unneeded here (ts_utc in util), and corrected c…
Browse files Browse the repository at this point in the history
…omment in hpss
  • Loading branch information
Anthony Bartoletti committed Dec 20, 2024
1 parent 0540e93 commit d7bae50
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion zstash/create.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
import sqlite3
import sys
from typing import Any, List, Tuple
from datetime import datetime, timezone

from six.moves.urllib.parse import urlparse

Expand Down
2 changes: 1 addition & 1 deletion zstash/hpss.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ def hpss_transfer(
logger.info(f"{ts_utc()}: DIVING: hpss calls globus_transfer(name={name})")
globus_status = globus_transfer(endpoint, url_path, name, transfer_type, non_blocking)
logger.info(f"{ts_utc()}: SURFACE hpss globus_transfer(name={name}) returns {globus_status}")
# NOTE: Here, the status could be "TIMEOUT_RETRIES_EXHAUSTED", meaning a very long transfer
# NOTE: Here, the status could be "EXHAUSTED_TIMEOUT_RETRIES", meaning a very long transfer
# or perhaps transfer is hanging. We should decide whether to ignore it, or cancel it, but
# we'd need the task_id to issue a cancellation. Perhaps we should have globus_transfer
# return a tuple (task_id, status).
Expand Down

0 comments on commit d7bae50

Please sign in to comment.