Change in osmo-ci[master]: ansible: Wrap distro version number as string for ansible

Holger Freyther gerrit-no-reply at lists.osmocom.org
Fri Nov 9 02:00:47 UTC 2018


Holger Freyther has submitted this change and it was merged. ( https://gerrit.osmocom.org/11648 )

Change subject: ansible: Wrap distro version number as string for ansible
......................................................................

ansible: Wrap distro version number as string for ansible

Change-Id: I58591ee37c6ec1479778c3f90ed8004a7e2adf1e
---
M ansible/roles/osmocom-jenkins-slave/tasks/debian.yml
M ansible/roles/osmocom-jenkins-slave/tasks/main.yml
M ansible/roles/osmocom-jenkins-slave/tasks/osmocom_build_deps.yml
3 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Holger Freyther: Looks good to me, approved; Verified
  Pau Espin Pedrol: Looks good to me, but someone else must approve



diff --git a/ansible/roles/osmocom-jenkins-slave/tasks/debian.yml b/ansible/roles/osmocom-jenkins-slave/tasks/debian.yml
index 87a616c..30e4ece 100644
--- a/ansible/roles/osmocom-jenkins-slave/tasks/debian.yml
+++ b/ansible/roles/osmocom-jenkins-slave/tasks/debian.yml
@@ -40,4 +40,4 @@
     install_recommends: no
   with_items:
     - openjdk-8-jre-headless
-  when: ansible_distribution == 'Debian' and ansible_distribution_version >= 9
+  when: ansible_distribution == 'Debian' and ansible_distribution_version >= '9'
diff --git a/ansible/roles/osmocom-jenkins-slave/tasks/main.yml b/ansible/roles/osmocom-jenkins-slave/tasks/main.yml
index 98cae55..36bc424 100644
--- a/ansible/roles/osmocom-jenkins-slave/tasks/main.yml
+++ b/ansible/roles/osmocom-jenkins-slave/tasks/main.yml
@@ -27,7 +27,7 @@
 
 - name: install ttcn3 dependencies
   include_tasks: ttcn3-slave.yml
-  when: ttcn3_slave and ansible_distribution == 'Debian' and ansible_distribution_version >= 9
+  when: ttcn3_slave and ansible_distribution == 'Debian' and ansible_distribution_version >= '9'
 
 - name: copy .gitconfig
   copy:
diff --git a/ansible/roles/osmocom-jenkins-slave/tasks/osmocom_build_deps.yml b/ansible/roles/osmocom-jenkins-slave/tasks/osmocom_build_deps.yml
index 16e72ed..b2a2f2b 100644
--- a/ansible/roles/osmocom-jenkins-slave/tasks/osmocom_build_deps.yml
+++ b/ansible/roles/osmocom-jenkins-slave/tasks/osmocom_build_deps.yml
@@ -105,7 +105,7 @@
     install_recommends: no
   with_items:
     - liblua5.3-dev
-  when: ansible_distribution == 'Debian' and ansible_distribution_version >= 9
+  when: ansible_distribution == 'Debian' and ansible_distribution_version >= '9'
 
 - name: install liblimesuite-dev for stretch or later
   apt:
@@ -115,4 +115,4 @@
     install_recommends: no
   with_items:
     - liblimesuite-dev
-  when: ansible_distribution == 'Debian' and ansible_distribution_version >= 9
+  when: ansible_distribution == 'Debian' and ansible_distribution_version >= '9'

-- 
To view, visit https://gerrit.osmocom.org/11648
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I58591ee37c6ec1479778c3f90ed8004a7e2adf1e
Gerrit-Change-Number: 11648
Gerrit-PatchSet: 2
Gerrit-Owner: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181109/6b793694/attachment.html>


More information about the gerrit-log mailing list