[libvirt PATCH 001/351] meson: ci: increase git clone depth to 1000

Peter Krempa pkrempa at redhat.com
Wed Jul 22 08:35:43 UTC 2020


On Wed, Jul 22, 2020 at 10:21:43 +0200, Ján Tomko wrote:
> On a Thursday in 2020, Pavel Hrdina wrote:
> > Having limit set to 100 is in most cases perfect but sometimes there can
> > be a larger series that will have more than 100 patches and it will make
> > the check-dco job fail.
> > 
> > Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> > ---
> > .gitlab-ci.yml      | 2 +-
> > ci/cirrus/build.yml | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> > index 702198ec8e4..c997dc6df25 100644
> > --- a/.gitlab-ci.yml
> > +++ b/.gitlab-ci.yml
> > @@ -1,5 +1,5 @@
> > variables:
> > -  GIT_DEPTH: 100
> > +  GIT_DEPTH: 1000
> > 
> > stages:
> >   - sanity_checks
> > diff --git a/ci/cirrus/build.yml b/ci/cirrus/build.yml
> > index 893e13d7241..49e90b6d67b 100644
> > --- a/ci/cirrus/build.yml
> > +++ b/ci/cirrus/build.yml
> > @@ -14,7 +14,7 @@ build_task:
> >   install_script:
> >     - @INSTALL_COMMAND@ @PKGS@
> >   clone_script:
> > -    - git clone --depth 100 "$CI_REPOSITORY_URL" .
> > +    - git clone --depth 1000 "$CI_REPOSITORY_URL" .
> 
> Does cirrus run the dco-check too or is this just for consistency?
> 
> >     - git fetch origin "$CI_COMMIT_REF_NAME"
> >     - git reset --hard "$CI_COMMIT_SHA"
> >   build_script:
> 
> Reviewed-by: Ján Tomko <jtomko at redhat.com>

Do we really want to merge this in the end?




More information about the libvir-list mailing list