Build succeeded.
I think default value are better defined in the playbook/role directly. But if you prefer to have it set in the zuul job configuration, then you need to add it to the getTMTJobVars function like this: https://pagure.io/fork/tdecacqu/fedora-zuul-jobs-config/c/2e96abec6e495ce026526493b0de33a4847bb79b?branch=test-type-fmf
getTMTJobVars
@tdecacqu the thing is the playbook/role will run both STI and tmt tests so we need the param in the zuul job configuration
STI
tmt
@tdecacqu
But if you prefer to have it set in the zuul job configuration, then you need to add it to the getTMTJobVars function like this: https://pagure.io/fork/tdecacqu/fedora-zuul-jobs-config/c/2e96abec6e495ce026526493b0de33a4847bb79b?branch=test-type-fmf
ah neat, that is what we were not able to do
@guyinger pls incorporate ^
rebased onto bb997aa6e0bc9ab8166075a8a2886100ba3bd314
Please run the "dhall-to-yaml --generated-comment --file jobs.dhall | zuulfmt > jobs.yaml" command to get the jobs.yaml updated based on the dhall definition.
rebased onto 8ddaa1338f115ed2c44b939bceffb700c64405f4
Thanks let's merge !
Metadata Update from @fbo: - Pull-request tagged with: gateit
Build succeeded (gate pipeline).
Pull-Request has been merged by zuul