Project

General

Profile

Actions

Bug #15422

closed

Update provenance/used_by endpoint to support containers and document

Added by Tom Morris about 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
07/12/2019
Due date:
% Done:

100%

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

Subtasks 1 (0 open1 closed)

Task #15437: Review 15422-collections-prov-docResolvedEric Biagiotti07/12/2019

Actions
Actions #1

Updated by Tom Morris about 3 years ago

  • Target version changed from 2019-07-03 Sprint to 2019-07-17 Sprint
Actions #2

Updated by Peter Amstutz about 3 years ago

  • Assigned To set to Peter Amstutz
Actions #3

Updated by Peter Amstutz about 3 years ago

The provenance/used_by methods haven't been updated for containers, meaning they don't return particularly useful results. We should either postpone or expand the scope of this story to include updating adding traversal of containers in the provenance graph.

Actions #4

Updated by Tom Morris about 3 years ago

  • Subject changed from Document collections provenance method to Update provenance/used_by endpoint to support containers and document
  • Story points set to 1.0
Actions #5

Updated by Peter Amstutz about 3 years ago

15422-collections-prov-doc @ b26a7ca6d495377344fc425e0c8e93275abef575

  • Update provenance and used_by feature on API server to traverse containers/container requests
  • Update workbench rendering
    • handle containers/container requests
    • use left-right layout for collections (works better visually when there are large fanouts)
    • rework container request rendering so it isn't so much of a special case
    • don't render extra items that are not part of the provenance graph.

https://ci.curoverse.com/view/Developer/job/developer-run-tests/1391/

Actions #6

Updated by Peter Amstutz about 3 years ago

  • Status changed from New to In Progress
Actions #7

Updated by Eric Biagiotti about 3 years ago

Actions #8

Updated by Peter Amstutz about 3 years ago

Eric Biagiotti wrote:

  • Looks like the test run has actual broken workbench tests.

Fixed. Thanks for pointing that out.

  • In the documentation, can we be more specific about what the attributes belong to? i.e. instead of (via "output_uuid" or “log_uuid”) maybe (via the “output_uuid” or “log_uuid" attributes of the container request)? It also would be more clear if the attributes were highlighted.

Done.

No. That's because when searching "up" (provenance) it searches by uuid but when it is searching "down" (used by) it searches by portable data hash. If that line was outside the if statement, the collection would be added twice, once by uuid and again by pdh.

15422-collections-prov-doc @ 508f13840841afc5938f7210a999ff58f002b29d

https://ci.curoverse.com/view/Developer/job/developer-run-tests/1399/

Actions #9

Updated by Eric Biagiotti about 3 years ago

Thanks, all the tests pass now. This LGTM!

Actions #10

Updated by Peter Amstutz about 3 years ago

  • Status changed from In Progress to Resolved
Actions #11

Updated by Peter Amstutz over 2 years ago

  • Release set to 22
Actions

Also available in: Atom PDF