Story #3709

[SDKs] arv-put should return Arvados UUID of collection by default instead of portable data hash.

Added by Peter Amstutz over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
SDKs
Target version:
Start date:
08/28/2014
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
1.0

Description

Users want to be able to find their collections, so we should give them the Arvados UUID of the collection object instead of the portable data hash. However, crunch (and possibly other tools) expect to get portable data hashes back from arv-put, so there needs to be a new command line flag for arv-put such as "--portable-data-hash" that continues to implement the current behavior, and tools need to be modified to use that.


Subtasks

Task #3754: Review 3709-arv-put-print-uuidResolvedPeter Amstutz

Task #3726: Change arv-putResolvedPeter Amstutz

Task #3727: Fix crunch-jobResolvedPeter Amstutz

Associated revisions

Revision 4fc443bf
Added by Peter Amstutz over 6 years ago

Merge branch '3709-arv-put-print-uuid' closes #3709

Revision f2d89d3e (diff)
Added by Peter Amstutz over 6 years ago

Fix arv-put tests refs #3709

History

#1 Updated by Peter Amstutz over 6 years ago

  • Description updated (diff)

#2 Updated by Peter Amstutz over 6 years ago

  • Subject changed from arv-put returns UUID of collection by default instead of portable data hash. to arv-put should return Arvados UUID of collection by default instead of portable data hash.

#3 Updated by Ward Vandewege over 6 years ago

  • Target version changed from Arvados Future Sprints to 2014-09-17 sprint

#4 Updated by Peter Amstutz over 6 years ago

  • Subject changed from arv-put should return Arvados UUID of collection by default instead of portable data hash. to [SDKs] arv-put should return Arvados UUID of collection by default instead of portable data hash.

#5 Updated by Peter Amstutz over 6 years ago

  • Category set to SDKs

#6 Updated by Peter Amstutz over 6 years ago

  • Story points set to 1.0

#7 Updated by Peter Amstutz over 6 years ago

  • Assigned To set to Peter Amstutz

#8 Updated by Peter Amstutz over 6 years ago

  • Status changed from New to In Progress

#9 Updated by Brett Smith over 6 years ago

Reviewing c59670e and it looks good to me. In case it's helpful for the future, you might find testing mydict.get(key) easier than key in mydict and mydict[key]. Thanks.

#10 Updated by Anonymous over 6 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 67 to 100

Applied in changeset arvados|commit:4fc443bfedf165307795e2b9e6c022008b5d7639.

Also available in: Atom PDF