[ARVADOS] updated: 9640930c3934344ab64c92a5c86c1f7488a4de42

Git user git at public.curoverse.com
Fri Nov 11 16:23:35 EST 2016


Summary of changes:
 sdk/cli/bin/arv-run-pipeline-instance | 2 ++
 1 file changed, 2 insertions(+)

       via  9640930c3934344ab64c92a5c86c1f7488a4de42 (commit)
      from  d0bc46ce842e7fdb71c25cc32caef0afd209c9ec (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 9640930c3934344ab64c92a5c86c1f7488a4de42
Author: Tom Clegg <tom at curoverse.com>
Date:   Fri Nov 11 16:19:58 2016 -0500

    3486: Fix ignored "required: false" in pipeline template. refs #3486 refs #10470

diff --git a/sdk/cli/bin/arv-run-pipeline-instance b/sdk/cli/bin/arv-run-pipeline-instance
index bcb11d1..960d784 100755
--- a/sdk/cli/bin/arv-run-pipeline-instance
+++ b/sdk/cli/bin/arv-run-pipeline-instance
@@ -380,6 +380,8 @@ class WhRunPipelineInstance
           value = params[parametername.to_s]
         elsif parameter.has_key?(:default)
           value = parameter[:default]
+        elsif [false, 'false', 0, '0'].index(parameter[:required])
+          value = nil
         else
           errors << [componentname, parametername, "required parameter is missing"]
           next

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list