.gitlab-ci.d/static_checks.yml | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-)
The 'check-patch' and 'check-dco' jobs only need Python and git for
checking the patches, so it's not really necessary to use a container
here that has all the other build dependencies installed. By using a
lightweight Alpine container, we can improve the runtime here quite a
bit, cutting it down from ca. 1:30 minutes to ca. 45 seconds.
Suggested-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
v2: Use python:alpine for even quicker execution
.gitlab-ci.d/static_checks.yml | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/.gitlab-ci.d/static_checks.yml b/.gitlab-ci.d/static_checks.yml
index 5e955540d3..94858e3272 100644
--- a/.gitlab-ci.d/static_checks.yml
+++ b/.gitlab-ci.d/static_checks.yml
@@ -1,12 +1,13 @@
check-patch:
stage: build
- image: $CI_REGISTRY_IMAGE/qemu/centos8:latest
- needs:
- job: amd64-centos8-container
+ image: python:3.10-alpine
+ needs: []
script:
- .gitlab-ci.d/check-patch.py
variables:
GIT_DEPTH: 1000
+ before_script:
+ - apk -U add git perl
rules:
- if: '$CI_PROJECT_NAMESPACE == "qemu-project"'
when: never
@@ -15,12 +16,13 @@ check-patch:
check-dco:
stage: build
- image: $CI_REGISTRY_IMAGE/qemu/centos8:latest
- needs:
- job: amd64-centos8-container
+ image: python:3.10-alpine
+ needs: []
script: .gitlab-ci.d/check-dco.py
variables:
GIT_DEPTH: 1000
+ before_script:
+ - apk -U add git
rules:
- if: '$CI_PROJECT_NAMESPACE == "qemu-project" && $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH'
when: never
--
2.27.0
Thomas Huth <thuth@redhat.com> writes: > The 'check-patch' and 'check-dco' jobs only need Python and git for > checking the patches, so it's not really necessary to use a container > here that has all the other build dependencies installed. By using a > lightweight Alpine container, we can improve the runtime here quite a > bit, cutting it down from ca. 1:30 minutes to ca. 45 seconds. > > Suggested-by: Daniel P. Berrangé <berrange@redhat.com> > Signed-off-by: Thomas Huth <thuth@redhat.com> Queued to testing/next, thanks. -- Alex Bennée
Alex Bennée <alex.bennee@linaro.org> writes: > Thomas Huth <thuth@redhat.com> writes: > >> The 'check-patch' and 'check-dco' jobs only need Python and git for >> checking the patches, so it's not really necessary to use a container >> here that has all the other build dependencies installed. By using a >> lightweight Alpine container, we can improve the runtime here quite a >> bit, cutting it down from ca. 1:30 minutes to ca. 45 seconds. >> >> Suggested-by: Daniel P. Berrangé <berrange@redhat.com> >> Signed-off-by: Thomas Huth <thuth@redhat.com> > > Queued to testing/next, thanks. And it didn't apply because it was already merged - sorry about that ;-) -- Alex Bennée
On Mon, May 16, 2022 at 10:23:10AM +0200, Thomas Huth wrote: > The 'check-patch' and 'check-dco' jobs only need Python and git for > checking the patches, so it's not really necessary to use a container > here that has all the other build dependencies installed. By using a > lightweight Alpine container, we can improve the runtime here quite a > bit, cutting it down from ca. 1:30 minutes to ca. 45 seconds. > > Suggested-by: Daniel P. Berrangé <berrange@redhat.com> > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > v2: Use python:alpine for even quicker execution > > .gitlab-ci.d/static_checks.yml | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> With 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 :|
On Mon, May 16, 2022 at 4:26 AM Daniel P. Berrangé <berrange@redhat.com> wrote: > > On Mon, May 16, 2022 at 10:23:10AM +0200, Thomas Huth wrote: > > The 'check-patch' and 'check-dco' jobs only need Python and git for > > checking the patches, so it's not really necessary to use a container > > here that has all the other build dependencies installed. By using a > > lightweight Alpine container, we can improve the runtime here quite a > > bit, cutting it down from ca. 1:30 minutes to ca. 45 seconds. > > > > Suggested-by: Daniel P. Berrangé <berrange@redhat.com> > > Signed-off-by: Thomas Huth <thuth@redhat.com> > > --- > > v2: Use python:alpine for even quicker execution > > > > .gitlab-ci.d/static_checks.yml | 14 ++++++++------ > > 1 file changed, 8 insertions(+), 6 deletions(-) > > Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> > Looks good. Sorry for fooling you with my python container -- I am using it to ensure a very wide spectrum of version compatibility. Reviewed-by: John Snow <jsnow@redhat.com>
© 2016 - 2024 Red Hat, Inc.