[libvirt] [jenkins-ci PATCH] jobs: Shorten name for python-distutils jobs

Andrea Bolognani abologna at redhat.com
Wed Feb 14 11:47:04 UTC 2018


Instead of passing the full name of the Python binary as 'python',
only pass the major version as 'pyver' and build everything else
based on that.

Doing so allows us to make a few things, most notably job names,
slightly shorter and nicer.

Signed-off-by: Andrea Bolognani <abologna at redhat.com>
---
I wouldn't bother changing this if the previous change to Python
jobs had already been deployed, but since it hasn't yet might as
well have nicer names instead :)

 jobs/python-distutils.yaml   | 22 +++++++++++-----------
 projects/libvirt-python.yaml | 16 ++++++++--------
 projects/virt-manager.yaml   | 14 +++++++-------
 3 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/jobs/python-distutils.yaml b/jobs/python-distutils.yaml
index 122c759..ff68c29 100644
--- a/jobs/python-distutils.yaml
+++ b/jobs/python-distutils.yaml
@@ -1,11 +1,11 @@
 
 - job-template:
     id: python-distutils-build-job
-    name: '{name}-{branch}-{python}-build'
+    name: '{name}-{branch}-py{pyver}-build'
     project-type: matrix
-    description: '{title} Build ({python})'
+    description: '{title} Build (Python {pyver})'
     command_pre_build: ''
-    workspace: '{name}-{branch}-{python}'
+    workspace: '{name}-{branch}-py{pyver}'
     child-workspace: '.'
     block-downstream: true
     block-upstream: true
@@ -43,8 +43,8 @@
           {global_env}
           {local_env}
           {command_pre_build}
-          {python} ./setup.py build
-          {python} ./setup.py install --prefix=$VIRT_PREFIX
+          python{pyver} ./setup.py build
+          python{pyver} ./setup.py install --prefix=$VIRT_PREFIX
     publishers:
       - email:
           recipients: '{obj:spam}'
@@ -54,9 +54,9 @@
 
 - job-template:
     id: python-distutils-check-job
-    name: '{name}-{branch}-{python}-check'
+    name: '{name}-{branch}-py{pyver}-check'
     project-type: matrix
-    description: '{title} Check ({python})'
+    description: '{title} Check (Python {pyver})'
     workspace: '{name}-{branch}'
     child-workspace: '.'
     block-downstream: true
@@ -83,7 +83,7 @@
       - shell: |
           {global_env}
           {local_env}
-          {python} ./setup.py test
+          python{pyver} ./setup.py test
     publishers:
       - email:
           recipients: '{obj:spam}'
@@ -92,9 +92,9 @@
 
 - job-template:
     id: python-distutils-rpm-job
-    name: '{name}-{branch}-{python}-rpm'
+    name: '{name}-{branch}-py{pyver}-rpm'
     project-type: matrix
-    description: '{title} RPM ({python})'
+    description: '{title} RPM (Python {pyver})'
     workspace: '{name}-{branch}'
     child-workspace: '.'
     block-downstream: true
@@ -122,7 +122,7 @@
           {global_env}
           {local_env}
           sed -i -e 's/BuildRequires: libvirt.*devel.*//' *.spec.in
-          {python} ./setup.py rpm
+          python{pyver} ./setup.py rpm
     publishers:
       - email:
           recipients: '{obj:spam}'
diff --git a/projects/libvirt-python.yaml b/projects/libvirt-python.yaml
index e065c5d..1c29321 100644
--- a/projects/libvirt-python.yaml
+++ b/projects/libvirt-python.yaml
@@ -14,10 +14,10 @@
     title: Libvirt Python
     jobs:
       - python-distutils-build-job:
-          python: python2
+          pyver: 2
           parent_jobs: 'libvirt-master-build'
       - python-distutils-build-job:
-          python: python3
+          pyver: 3
           parent_jobs: 'libvirt-master-build'
           machines:
             - libvirt-debian-8
@@ -28,11 +28,11 @@
             - libvirt-freebsd-10
             - libvirt-freebsd-11
       - python-distutils-check-job:
-          python: python2
-          parent_jobs: 'libvirt-python-master-{python}-build'
+          pyver: 2
+          parent_jobs: 'libvirt-python-master-py{pyver}-build'
       - python-distutils-check-job:
-          python: python3
-          parent_jobs: 'libvirt-python-master-{python}-build'
+          pyver: 3
+          parent_jobs: 'libvirt-python-master-py{pyver}-build'
           machines:
             - libvirt-debian-8
             - libvirt-debian-9
@@ -42,8 +42,8 @@
             - libvirt-freebsd-10
             - libvirt-freebsd-11
       - python-distutils-rpm-job:
-          python: python2
-          parent_jobs: 'libvirt-python-master-{python}-check'
+          pyver: 2
+          parent_jobs: 'libvirt-python-master-py{pyver}-check'
           machines:
             - libvirt-centos-6
             - libvirt-centos-7
diff --git a/projects/virt-manager.yaml b/projects/virt-manager.yaml
index b5b0df5..8f3112e 100644
--- a/projects/virt-manager.yaml
+++ b/projects/virt-manager.yaml
@@ -11,18 +11,18 @@
     title: Virtual Machine Manager
     jobs:
       - python-distutils-build-job:
-          python: python3
+          pyver: 3
           parent_jobs:
-            - 'libvirt-python-master-{python}-build'
+            - 'libvirt-python-master-py{pyver}-build'
             - 'libosinfo-master-build'
           command_pre_build: |
-            {python} ./setup.py configure --prefix=$VIRT_PREFIX
+            python{pyver} ./setup.py configure --prefix=$VIRT_PREFIX
       - python-distutils-check-job:
-          python: python3
-          parent_jobs: 'virt-manager-master-{python}-build'
+          pyver: 3
+          parent_jobs: 'virt-manager-master-py{pyver}-build'
       - python-distutils-rpm-job:
-          python: python3
-          parent_jobs: 'virt-manager-master-{python}-check'
+          pyver: 3
+          parent_jobs: 'virt-manager-master-py{pyver}-check'
           machines:
             - libvirt-fedora-26
             - libvirt-fedora-27
-- 
2.14.3




More information about the libvir-list mailing list