Re: [libvirt PATCH] ci: Reduce number of stages
On Wed, Jun 10, 2020 at 01:51:29PM +0200, Peter Krempa wrote:
On Wed, Jun 10, 2020 at 13:33:01 +0200, Andrea Bolognani wrote:
[...]
> Building artifacts in a separate pipeline stage also doesn't have any
> advantages, and only delays further stages by a couple of minutes.
> The only job that really makes sense in its own stage is the DCO
> check, because it's extremely fast (less than 1 minute) and, if that
> fails, we can avoid kicking off all other jobs.
On the contrary I think that the DCO check should be made after builds
as that usually forces users to add a sign-off just to bypass that check
if they want to sanity check their series.
Missing signoff is quite common for new contributors, so it was put as
the first check so that get quick notification of this mistake.
Since the lack of a sign off can be effectively used as a mark for
an
patch that is not ready to be pushed, but a build-check is still needed.
This adds a pointless hurdle in using the CI and also removes one of the
meaningful uses to have a sign off checker.
That kind of usage of signoff is not really required in a merge request
workflow. You won't typically open the merge request in the first place
if code isn't ready, but if you did, then there's explicit "WIP" flag
for
merge requests to achieve this. Once libvirt.git uses merge request, we
will fully block all ability to push directly to git.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|