Holger Freyther has posted comments on this change. (
https://gerrit.osmocom.org/11648 )
Change subject: ansible: Wrap distro version number as string for ansible
......................................................................
Patch Set 2:
> why is this needed? which version of ansible are you using?
ansible 2.4.6.0
config file = None
configured module search path = [
'/usr/local/share/py36-ansible/plugins/modules']
ansible python module location =
/usr/local/lib/python3.6/site-packages/ansible
executable location = /usr/local/bin/ansible
python version = 3.6.7 (default, Oct 30 2018, 23:02:44) [GCC 4.2.1 Compatible
FreeBSD Clang 3.4.1 (tags/RELEASE_34/dot1-final 208032)]
Error summary (full error below):
AnsibleUnsafeText $OP int
The examples I found also compared it as string and it seems to work for the
debian9 and limesuite case.
Full error:
fatal: [build2-deb9build-ansible]: FAILED! => {"failed": true, "msg": "The
conditional check 'ansible_distribution == 'Debian' and
ansible_distribution_version >= 9' failed. The error was: Unexpected templating
type error occurred on ({% if ansible_distribution == 'Debian' and
ansible_distribution_version >= 9 %} True {% else %} False {% endif %}): '>='
not supported between instances of 'AnsibleUnsafeText' and 'int'\n\nThe error
appears to have been in
'/root/git/osmo-ci/ansible/roles/osmocom-jenkins-slave/tasks/debian.yml': line
35, column 3, but may\nbe elsewhere in the file depending on the exact syntax
problem.\n\nThe offending line appears to be:\n\n\n- name: install java for
stretch or later\n ^ here\n"}
--
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: comment
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]>
Gerrit-Comment-Date: Sun, 11 Nov 2018 11:36:57 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
