#123 Generalize testing-farm-run-test role
Merged by zuul. Opened by guyinger.
guyinger/fedora-zuul-jobs generic-role-testing-farm-run-test  into  master

Download 123.patch

Generalize testing-farm-run-test role so it can be used for other Zuul projects outside of Fedora.

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@guyinger can you try to add the new secret as described on the Zuul wiki

@guyinger also trailing whitespaces are there :)

roles/testing-farm-run-test/tasks/main.yml
  1:50      error    trailing spaces  (trailing-spaces)

that is why linters job fails

Are you sure about zuul.project.url ? I don't see it in https://zuul-ci.org/docs/zuul/reference/jobs.html#var-zuul.project
I think we should expect a base_url var instead, that could be set by default to "https://src.fedoraproject.org" in role's default.yaml.

You should prepare a change to update the secret name for tmt/sti jobs in https://pagure.io/fedora-zuul-jobs-config/blob/master/f/zuul.d/jobs.dhall

rebased onto 5b9a1ccd2c3679bcbae42f4ba81d3affb69f2caf

Build succeeded.

@mvadkert I've changed the secret name so there's no need to add a new one instead https://pagure.io/fedora-zuul-jobs-config/pull-request/136#

@fbo thanks, I did the changes and created this PR for the secret name: https://pagure.io/fedora-zuul-jobs-config/pull-request/136#

rebased onto d4c23f1b98a66f6bf81174edd04a65743714e235

Build succeeded.

Metadata Update from @fbo:
- Pull-request tagged with: gateit

Build succeeded (gate pipeline).

Pull-Request has been merged by zuul

Metadata