PR#60: propose zuul config - fedora-infra/ansible

fedora-infra / ansible

#60 propose zuul config

Merged 4 years ago by nphilipp. Opened 4 years ago by fbo.

fedora-infra/ fbo/ansible zuul-conf into master

Propose the zuul config - project pipelines to run ansible-review-diff

Fabien Boucher • 4 years ago

578fe5c

.zuul.yaml

file added

		`@@ -0,0 +1,5 @@`
		`+ ---`
		`+ - project:`
		`+ check:`
		`+ jobs:`
		`+ - ansible-review-diff`

fbo commented 4 years ago

Depends-on: https://pagure.io/fedora-zuul-jobs/pull-request/60

Propose the zuul config - project pipelines to run ansible-review-diff

Co-Authored-By: Nils Philippsen <nils@redhat.com>
Co-Authored-By: Pierre-Yves Chibon <pingou@pingoured.fr>

The ansible-review-diff is defined in https://pagure.io/fedora-zuul-jobs

This commit tells Zuul to run ansible-review-diff on all new and updated
Pull Requests.

This change was initially fedora-infra/ansible/pull-request/54

zuul commented 4 years ago

Build succeeded.

ansible-review-diff : SUCCESS in 1m 48s

rebased onto 81da68237df19bdec30bd6c12c7d8af8d5a513c5

4 years ago

rebased onto 81da68237df19bdec30bd6c12c7d8af8d5a513c5

4 years ago

zuul commented 4 years ago

Build succeeded.

ansible-review-diff : SUCCESS in 1m 16s

pingou commented 4 years ago

IIUC #54 had all three jobs -diff, -base and -everything enabled.
Do we want to add them all here or in another PR?

nphilipp commented 4 years ago

@pingou No, #54 only had the -diff job enabled. The -base job is "abstract" and inherited by -everything and -diff. Apart from the runtime (~15 mins), the -everything job will fail all the time until we fix "all the things". When we've done that, maybe there's a way to occasionally run it on the side from time to time, just in case the -diff job doesn't catch everything.

fbo commented 4 years ago