[libvirt] [jenkins-ci PATCH] Stop building osinfo-db on CentOS 7

Fabiano Fidêncio posted 1 patch 5 years, 1 month ago
Failed in applying to current master (apply log)
jenkins/projects/osinfo-db.yaml | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
[libvirt] [jenkins-ci PATCH] Stop building osinfo-db on CentOS 7
Posted by Fabiano Fidêncio 5 years, 1 month ago
osinfo-db tests are Python3 only.

Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
---
 jenkins/projects/osinfo-db.yaml | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/jenkins/projects/osinfo-db.yaml b/jenkins/projects/osinfo-db.yaml
index 87b6748..25a6621 100644
--- a/jenkins/projects/osinfo-db.yaml
+++ b/jenkins/projects/osinfo-db.yaml
@@ -1,7 +1,16 @@
 ---
 - project:
     name: osinfo-db
-    machines: '{all_machines}'
+    # osinfo-db tests are Python 3 only, so they can't be built on
+    # CentOS 7;
+    machines:
+      - libvirt-debian-8
+      - libvirt-debian-9
+      - libvirt-fedora-28
+      - libvirt-fedora-29
+      - libvirt-fedora-rawhide
+      - libvirt-freebsd-11
+      - libvirt-freebsd-12
     title: osinfo database
     archive_format: xz
     git_url: '{git_urls[osinfo-db][default]}'
-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [jenkins-ci PATCH] Stop building osinfo-db on CentOS 7
Posted by Andrea Bolognani 5 years, 1 month ago
On Tue, 2019-03-26 at 11:29 +0100, Fabiano Fidêncio wrote:
[...]
>  jenkins/projects/osinfo-db.yaml | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)

You need to update the corresponding file living under
guests/playbooks/build/ at the same time.

[...]
>  - project:
>      name: osinfo-db
> -    machines: '{all_machines}'
> +    # osinfo-db tests are Python 3 only, so they can't be built on
> +    # CentOS 7;

If tests are the only thing that doesn't work without Python 3,
which I seem to understand is the case, then we can still build
on CentOS 7 and simply skip the tests there; if not, then we we
need to stop building libosinfo and any project depending on it
on CentOS 7 as well, no?

> +    machines:
> +      - libvirt-debian-8
> +      - libvirt-debian-9
> +      - libvirt-fedora-28
> +      - libvirt-fedora-29
> +      - libvirt-fedora-rawhide
> +      - libvirt-freebsd-11
> +      - libvirt-freebsd-12

According to the other patch I just reviewed, the one updating
build requirements for osinfo-db, we now need python3-libxml2;
however

  python3-libxml2:
    default: python3-libxml2
    FreeBSD: py36-libxml2
    CentOS7:
    Debian8:
    Ubuntu16:

so there are more target platforms that we need to exclude.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list