Feature #17936

Arv-put resume cache handling when having expired block signatures

Added by Lucas Di Pentima 2 months ago. Updated about 1 month ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
08/09/2021
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

With the recently added token expiration features, arv-put will start to complain more often about its resume cache not being valid anymore. In addition to this, arv-put is called from other Arvados tools like arv-keepdocker that may show arv-put's suggestions to users making the UX somewhat confusing.

It may be necessary to make arv-put automatically handle these situations, for example by removing the resume cache if it detects that there're expired block signatures on it.

Add a --batch mode to arv-put which automatically deletes the cache and retries on error.

Modify arv-keepdocker to use the --batch mode when it invokes arv-put.


Subtasks

Task #17973: Review 17936-arvput-batch-modeResolvedPeter Amstutz

Associated revisions

Revision 10bfcf75
Added by Lucas Di Pentima about 1 month ago

Merge branch '17936-arvput-batch-mode' into main. Closes #17936.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <>

History

#1 Updated by Lucas Di Pentima 2 months ago

  • Release set to 41

#2 Updated by Anonymous 2 months ago

Thanks google

#3 Updated by Peter Amstutz about 2 months ago

  • Target version changed from 2021-08-18 sprint to 2021-09-01 sprint

#4 Updated by Peter Amstutz about 2 months ago

  • Target version changed from 2021-09-01 sprint to 2021-08-18 sprint

#5 Updated by Peter Amstutz about 2 months ago

  • Description updated (diff)

#6 Updated by Peter Amstutz about 2 months ago

  • Assigned To set to Lucas Di Pentima

#7 Updated by Lucas Di Pentima about 2 months ago

  • Status changed from New to In Progress

#8 Updated by Lucas Di Pentima about 2 months ago

Updates at 8cb640565 - branch 17936-arvput-batch-mode
Test run: https://ci.arvados.org/job/developer-run-tests/2626/

  • Adds --batch argument to arv-put making it ignore the cache file if it's not valid.
  • Makes arv-keepdocker to always call arv-put with --batch so that users don't get confusing messages regarding invalid cached signatures from an indirectly called arv-put.
  • Adds tests.

#9 Updated by Peter Amstutz about 1 month ago

Lucas Di Pentima wrote:

Updates at 8cb640565 - branch 17936-arvput-batch-mode
Test run: https://ci.arvados.org/job/developer-run-tests/2626/

  • Adds --batch argument to arv-put making it ignore the cache file if it's not valid.
  • Makes arv-keepdocker to always call arv-put with --batch so that users don't get confusing messages regarding invalid cached signatures from an indirectly called arv-put.
  • Adds tests.

This LGTM.

#10 Updated by Lucas Di Pentima about 1 month ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF