[ovirt-devel] Re: GitHub Actions-based CI for vdsm
On Fri, Nov 26, 2021 at 7:44 PM Marcin Sobczyk <msobczyk(a)redhat.com> wrote:
Hi All,
I've been working on adding GitHub Actions-based CI to vdsm today.
Feel free to check out the patches here:
https://gerrit.ovirt.org/q/topic:%22github-actions-ci%22+(status:open%20O...
Awesome!
Some comments:
- the linters work fine already, we can start using them
- RPM building works too in general. I think the RPM versions are not
right yet, so I'll look into this. After the 'rpm' job is done we get a
zipfile with all the built RPMs inside. In the future we may want to run
'createrepo_c' on this dir as well, so we'll have a ready-to-be-used
repository in that zip.
We already have code to do this in automation/common.sh - can we
reuse it or extract it to new script?
- 'tests' are working too, but we have a couple of failures
we'd need to
look at. This job, unlike the others, runs in GitHub's Ubuntu VM inside
which we use a privileged container for running the tests.
The failure seems to be missing TRAVIS_CI=1 environment variable.
This should of course change to something moer generic like CONTAINER_CI=1
and the helper modules tests/testing.py should be updated.
- didn't try 'tests-storage' yet
That's fine, the storage team will handle this.
- Not to waist precious free CI minutes and storage, we run linters
first, tests after that, but only under the condition that the linters
didn't fail, and finally we build RPMs, but this time under the
condition that the tests didn't fail.
This is very bad idea. We want to run all the jobs in parallel so we
can get quick feedback from all jobs at the same run. There is no real
dependency between the jobs, and we don't waste resources by running
them in parallel.
Thanks for this work!
Nir