[all PATCH] gitlab: add CI job for validating DCO signoff

Andrea Bolognani abologna at redhat.com
Wed Apr 22 17:39:46 UTC 2020

On Wed, 2020-04-22 at 18:06 +0100, Daniel P. Berrangé wrote:
> On Wed, Apr 22, 2020 at 07:01:50PM +0200, Andrea Bolognani wrote:
> > I still don't understand why we would want to single out those
> > branches and not run the DCO check on them. What harm would it
> > cause? It takes around a minute to run it, which is significantly
> > less than the other jobs running during the prebuild stage...
> The check-dco script doesn't actually work if run against the
> main libvirt repo, as it ends up trying to use itself as a
> reference and failing to figure out which commits need checking.
> Of course that's a bug that's fixable, but in general I think it
> is better to not runthe job at all and thus eliminate any risk
> of false failures.

It seems to work fine here:


The corresponding commit is


where I changed the local require-dco.py script to use the very same
branch I pushed to as the base branch for DCO checking.

I believe you fixed the issue you mention above with

  commit 769ff77c9c5afaec97350a4931e5ca123b6af6d2
  Author: Daniel P. Berrangé <berrange at redhat.com>
  Date:   Fri Mar 27 14:38:49 2020 +0000

    scripts: avoid error in DCO check on empty branches

    If the DCO check is run on an empty branch (ie one which has no commits
    different from master), it throws an error due to trying to interpret
    the empty string as a git commit SHA.

    Reviewed-by: Andrea Bolognani <abologna at redhat.com>
    Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>


So I see no reason not to just run the check on all branches.

If you remove the except: part, and of course the corresponding
comment as well, then

  Reviewed-by: Andrea Bolognani <abologna at redhat.com>

Andrea Bolognani / Red Hat / Virtualization

More information about the libvir-list mailing list