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 <[email protected]>
Gerrit-Reviewer: Holger Freyther <[email protected]>
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: lynxis lazus <[email protected]>

Reply via email to