#128 remove rpm-tests job and related playbook
Opened by guyinger. Modified
guyinger/fedora-zuul-jobs cleanup_old_check_sti_jobs  into  master

Download 128.patch
no initial comment

Zuul encountered a syntax error while parsing its configuration in the
repo fedora-zuul-jobs-config on branch master. The error was:

Job rpm-test not defined

The error appears in the following project stanza:

project:
name: rpms/selinux-policy
check:
jobs:
- rpm-test:
timeout: 21600

in "fedora-zuul-jobs-config/zuul.d/projects.yaml@master", line 101, column 3

you need to remove this first https://pagure.io/fedora-zuul-jobs-config/blob/master/f/zuul.d/projects.yaml#_101

you need to remove this first https://pagure.io/fedora-zuul-jobs-config/blob/master/f/zuul.d/projects.yaml#_101

Removed in this PR: https://pagure.io/fedora-zuul-jobs-config/pull-request/144

This change depends on a change with an invalid configuration.

Sorry not remove but change "rpm-test" to "rpm-sti-test". We should be conservative and keep the override of the timeout.

Well it seems we will have trouble to merge those two PRs 144 and 128.
I suggest to do it step by step as explained here: https://pagure.io/fedora-zuul-jobs/pull-request/127#comment-162135

Could you open a PR that just do the step 4 ?

Well it seems we will have trouble to merge those two PRs 144 and 128.
I suggest to do it step by step as explained here: https://pagure.io/fedora-zuul-jobs/pull-request/127#comment-162135

Could you open a PR that just do the step 4 ?

PR for step 4: https://pagure.io/fedora-zuul-jobs/pull-request/129

rebased onto b5e51e2c292980cc7925f788823a4c191410eb97

This change depends on a change that failed to merge.

Zuul encountered a syntax error while parsing its configuration in the
repo fedora-zuul-jobs on branch master. The error was:

Job rpm-test not defined

The error appears in the following project-template stanza:

project-template:
name: test-tests-def
check:
jobs:
- rpm-test

in "fedora-zuul-jobs/zuul.d/templates.yaml@master", line 63, column 3

We need to check if rpm-sti-test and/or rpm-tmt-test support this king of test (w/o the repo url passed to the API call)

@fbo, yep, that should be supported, you can omit environemnts[].artifacts, it is an optional field. If not passed, it will just run tests, without installing any packages from the repo.

@fbo, yep, that should be supported, you can omit environemnts[].artifacts, it is an optional field. If not passed, it will just run tests, without installing any packages from the repo.

Metadata