Removal of dockerSDK check for other container engines

Typo fix and adding condition on not checking docker SDK version
when container engine is not docker

This is a followup to Ic30b67daa2e215524096ad1f4385c569e3d41b95
Signed-off-by: Ivan Halomi <i.halomi@partner.samsung.com>
Change-Id: Iafa24db06ad46bcfe250451ed98bc3c48d8a5138
This commit is contained in:
Ivan Halomi 2022-11-28 11:08:25 +01:00
parent 68376d6f31
commit 652b9cfe0e
2 changed files with 4 additions and 2 deletions

View File

@ -3,7 +3,9 @@
command: "{{ ansible_facts.python.executable }} -c \"import docker; print(docker.__version__)\""
register: result
changed_when: false
when: inventory_hostname in groups['baremetal']
when:
- inventory_hostname in groups['baremetal']
- kolla_container_engine == 'docker'
failed_when: result is failed or result.stdout is version(docker_py_version_min, '<')
# NOTE(osmanlicilegi): ansible_version.full includes patch number that's useless

View File

@ -44,7 +44,7 @@
- name: Check the configs
become: true
command: "{{ kolla_container_engine }}exec -u root {{ item.name }} /usr/local/bin/kolla_set_configs --check"
command: "{{ kolla_container_engine }} exec -u root {{ item.name }} /usr/local/bin/kolla_set_configs --check"
changed_when: false
failed_when: false
register: check_results