Actions
Idea #9019
closed
[API] Relax Job.ensure_no_collection_uuids_in_script_params
Status:
Closed
Priority:
Normal
Assigned To:
-
Category:
-
Target version:
-
Story points:
-
Description
Job su92l-8i9sb-gp7g016r63yx38u failed with:
2016-04-20_15:34:31 su92l-8i9sb-gp7g016r63yx38u 45929 log collection is 980840acc5a5fab3cd13327450f663d8+85 Error creating job for component GETEvidenceReportHTML: Script parameters must use portable_data_hash instead of collection uuid Job submission was: {"job":{"script":"run-GETEvidenceReportHTML","script_parameters":{"INPUT_COLLECTION":"0df6a8a8ea11d25e11bd307d3cb06bb2+205","REPORT_ID":"su92l-4zz18-ucz7noj94usyh6f (WGC069884D.vcf.gz)"},"script_version":"master","repository":"https://github.com/abeconnelly/GETEvidenceReportHTML","runtime_constraints":{"docker_image":"arvados/jobs","min_nodes":1},"owner_uuid":"su92l-j7d0g-skv48ygo1i3cvi5","submit_id":"instance su92l-d1hrv-rtk4n266kwogmv5 rand 1pt93cbhak90n","state":"Queued"},"find_or_create":true}
The value in REPORT_ID should not trigger this check.
Updated by
Updated by Brett Smith almost 8 years ago
- Subject changed from [Crunch] script parameters check for PDH is too agressive to [API] Relax Job.ensure_no_collection_uuids_in_script_params
- Tracker changed from Bug to Idea
Ward, would it work for you if the validation failed when a script_parameter matched ^[collection uuid](/|$)?
Updated by Brett Smith almost 8 years ago
This validation was a transition measure as we gave collections "normal" UUIDs. Now that that's basically settled, we should get rid of it. If we want to help users write more reproducible pipelines, that should be a feature in Workbench, like a red flag for work units that refer to collections by UUID.
Updated by
Actions