[libvirt] [jenkins-ci PATCH 0/2] Run 'make syntax-check' for libvirt-dbus
This requires https://www.redhat.com/archives/libvir-list/2018-August/msg00863.html to be merged first. Andrea Bolognani (2): guests: Install flake8 for libvirt-dbus projects: Run 'make syntax-check' for libvirt-dbus guests/vars/mappings.yml | 7 +++++++ guests/vars/projects/libvirt-dbus.yml | 1 + projects/libvirt-dbus.yaml | 11 ++++++++++- 3 files changed, 18 insertions(+), 1 deletion(-) -- 2.17.1
It is required to run 'make syntax-check'. Signed-off-by: Andrea Bolognani <abologna@redhat.com> --- guests/vars/mappings.yml | 7 +++++++ guests/vars/projects/libvirt-dbus.yml | 1 + 2 files changed, 8 insertions(+) diff --git a/guests/vars/mappings.yml b/guests/vars/mappings.yml index 21948fb..59a4b01 100644 --- a/guests/vars/mappings.yml +++ b/guests/vars/mappings.yml @@ -100,6 +100,13 @@ mappings: default: ebtables FreeBSD: + flake8: + deb: flake8 + pkg: py36-flake8 + rpm: python3-flake8 + CentOS7: + Debian8: + fuse: deb: libfuse-dev pkg: fusefs-libs diff --git a/guests/vars/projects/libvirt-dbus.yml b/guests/vars/projects/libvirt-dbus.yml index ebb346e..22052de 100644 --- a/guests/vars/projects/libvirt-dbus.yml +++ b/guests/vars/projects/libvirt-dbus.yml @@ -1,5 +1,6 @@ --- packages: + - flake8 - glib2 - python3 - python3-dbus -- 2.17.1
Now that we have the necessary tools installed, we can run the extra checks for every build. Signed-off-by: Andrea Bolognani <abologna@redhat.com> --- projects/libvirt-dbus.yaml | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/projects/libvirt-dbus.yaml b/projects/libvirt-dbus.yaml index c12a769..f01ea9e 100644 --- a/projects/libvirt-dbus.yaml +++ b/projects/libvirt-dbus.yaml @@ -15,7 +15,7 @@ - libvirt-fedora-rawhide - libvirt-freebsd-10 - libvirt-freebsd-11 - - autotools-check-job: + - autotools-syntax-check-job: parent_jobs: 'libvirt-dbus-master-build' machines: - libvirt-debian-9 @@ -24,6 +24,15 @@ - libvirt-fedora-rawhide - libvirt-freebsd-10 - libvirt-freebsd-11 + - autotools-check-job: + parent_jobs: 'libvirt-dbus-master-syntax-check' + machines: + - libvirt-debian-9 + - libvirt-fedora-27 + - libvirt-fedora-28 + - libvirt-fedora-rawhide + - libvirt-freebsd-10 + - libvirt-freebsd-11 - autotools-rpm-job: parent_jobs: 'libvirt-dbus-master-check' machines: '{rpm_machines}' -- 2.17.1
On Tue, Aug 14, 2018 at 05:35:32PM +0200, Andrea Bolognani wrote:
This requires
https://www.redhat.com/archives/libvir-list/2018-August/msg00863.html
to be merged first.
Andrea Bolognani (2): guests: Install flake8 for libvirt-dbus projects: Run 'make syntax-check' for libvirt-dbus
guests/vars/mappings.yml | 7 +++++++ guests/vars/projects/libvirt-dbus.yml | 1 + projects/libvirt-dbus.yaml | 11 ++++++++++- 3 files changed, 18 insertions(+), 1 deletion(-)
Reviewed-by: Ján Tomko <jtomko@redhat.com> Jano
participants (2)
-
Andrea Bolognani -
Ján Tomko