Project

General

Profile

Actions

Bug #4756

closed

[API] Add migration to change collection uuids to portable_data_hash in old job script_parameters

Added by Tim Pierce almost 10 years ago. Updated over 7 years ago.

Status:
Rejected
Priority:
Normal
Assigned To:
-
Category:
API
Target version:
Story points:
2.0

Description

Following #3036 and #4269, collection uuids are forbidden from job script parameters. We have some unknown number of jobs in the database which reference input collections by uuid, and should add a migration to keep them consistent with new jobs.

The migration for this change will need to be careful to use the correct portable_data_hash for a collection as it existed at the time the job was created.

  • If the collection has not been modified since the job was recorded, then it should be safe to use the current portable_data_hash (this should cover most cases).
  • If, however, the collection has been modified more recently than the job, it will be necessary to process the logs for that collection to ensure that we use the portable_data_hash that was in effect at the time the job was processed.

Related issues

Related to Arvados - Idea #3036: [API] Use regular uuids instead of content hashes to identify collectionsResolvedPeter Amstutz08/07/2014Actions
Related to Arvados - Bug #4269: [API] Disallow collection UUIDs in script_parameters, only allow portable data hashResolvedTim Pierce12/04/2014Actions
Actions #1

Updated by Brett Smith almost 10 years ago

  • Target version deleted (Bug Triage)

Per discussion in sprint retrospective, not a bug. Moving to backlog for Tom's prioritization into a future sprint.

Actions #2

Updated by Tom Clegg almost 10 years ago

  • Target version set to Deferred
Actions #3

Updated by Tom Clegg over 7 years ago

  • Status changed from New to Rejected
Actions

Also available in: Atom PDF