Project

General

Profile

Actions

Bug #10259

closed

[CWL] Submitted pipelines don't finish

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

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Story points:
0.5

Description

Pipelines submitted from arvados-cwl-runner don't finish, because they are submitted as RunningOnClient. Fix this so that pipelines submitted this way are RunningOnServer so that arv-run-pipeline-instance will ensure that the pipeline is marked as completed.


Subtasks 3 (0 open3 closed)

Task #10286: Review 10259-cwl-pipeline-nullResolvedPeter Amstutz10/13/2016Actions
Task #10271: Review 10259-cwl-pipeline-valueResolvedPeter Amstutz10/13/2016Actions
Task #10260: Review 10259-cwl-pipelineResolvedRadhika Chippada10/13/2016Actions
Actions #1

Updated by Peter Amstutz over 7 years ago

  • Description updated (diff)
  • Assigned To set to Peter Amstutz
  • Story points set to 0.5
Actions #2

Updated by Peter Amstutz over 7 years ago

  • Status changed from New to In Progress
Actions #3

Updated by Radhika Chippada over 7 years ago

LGTM

Actions #4

Updated by Peter Amstutz over 7 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:82cd0096558b94f8ff1e23d6e3f8b8dfec7df7b9.

Actions #5

Updated by Peter Amstutz over 7 years ago

  • Status changed from Resolved to In Progress
Actions #6

Updated by Lucas Di Pentima over 7 years ago

10259-cwl-pipeline-value @ 1825696494802d264c74ed2af58cc39b6ff8c7f8 LGTM

Actions #7

Updated by Peter Amstutz over 7 years ago

One more branch @ 10259-cwl-pipeline-null. Passes 100% CWL conformance tests for me on arvbox.

Actions #8

Updated by Lucas Di Pentima over 7 years ago

I think I spotted a small typo that don't get to be tested:

  • sdk/cli/bin/arv-run-pipeline-instance:381: I think has_key(...) should be has has_key?(...) as used at line 369.
Actions #9

Updated by Lucas Di Pentima over 7 years ago

Apart from that typo, lgtm.

Actions #10

Updated by Peter Amstutz over 7 years ago

  • Status changed from In Progress to Resolved

I think the fallout from this ticket has been resolved, so marking resolved.

Actions

Also available in: Atom PDF