QA » openQA Project » openQA Tests

CI has a weird behavior here. If a dev opens a pr just deleting a file (and removing it from the job_groups.yaml accordingly), the CI will not count the deleted file as changed, and will not try to run yamllint on it. Example: https://github.com/os-autoinst/opensuse-jobgroups/pull/240

But, if the dev adds a random change to any other yaml file to that pr, the CI counts both the changed file AND the deleted file as changed: https://github.com/os-autoinst/opensuse-jobgroups/pull/239

The culprit is probably the step Get modified job group yaml files in the static-check job of the CI (https://github.com/os-autoinst/opensuse-jobgroups/blob/316b798494090c1724896b83eda47f3a6a435638/.github/workflows/ci.yml#L29) - for some reason if there are only deleted files it doesn't count them, but if there are deleted AND changed files it counts them all as changed.

There are 2 solutions:
a) compliment the yamllint command with --ignore-nonexistent-files so that it doesn't fail if a file is missing. THat may sound a bit risky, but if a file gets accidentally deleted (and thus remains in the job_groups.yaml file) it will be caught in the next CI step (check for orphaned yaml files).
b) Try and make the Get modified job group yaml files step to always recognize deleted files and not add them to the changed files.

pr open: https://github.com/os-autoinst/opensuse-jobgroups/pull/242