2016-03-24 21:21:07 +08:00
|
|
|
---
|
2017-12-16 02:43:23 +08:00
|
|
|
# NOTE(leseb): using failed_when to handle the case when the image is not present yet
|
|
|
|
- name: "inspecting {{ ceph_docker_registry}}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }} image before pulling"
|
|
|
|
command: "docker inspect {{ ceph_docker_registry}}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}"
|
|
|
|
changed_when: false
|
|
|
|
failed_when: false
|
|
|
|
register: container_inspect_before_pull
|
|
|
|
|
2018-01-08 17:00:25 +08:00
|
|
|
- name: set_fact repodigest_before_pulling
|
|
|
|
set_fact:
|
|
|
|
repodigest_before_pulling: "{{ (container_inspect_before_pull.stdout | from_json)[0].RepoDigests[0] }}"
|
|
|
|
when:
|
|
|
|
- container_inspect_before_pull.rc == 0
|
|
|
|
|
2017-12-16 02:43:23 +08:00
|
|
|
- name: "pulling {{ ceph_docker_registry}}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }} image"
|
2017-10-18 03:09:03 +08:00
|
|
|
command: "timeout {{ docker_pull_timeout }} docker pull {{ ceph_docker_registry}}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}"
|
2016-03-24 21:21:07 +08:00
|
|
|
changed_when: false
|
2017-10-18 03:09:03 +08:00
|
|
|
register: docker_image
|
|
|
|
until: docker_image.rc == 0
|
|
|
|
retries: "{{ docker_pull_retry }}"
|
|
|
|
delay: 10
|
2017-09-15 06:48:53 +08:00
|
|
|
when:
|
|
|
|
- (ceph_docker_dev_image is undefined or not ceph_docker_dev_image)
|
2016-03-24 21:21:07 +08:00
|
|
|
|
2017-12-16 02:43:23 +08:00
|
|
|
- name: "inspecting {{ ceph_docker_registry}}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }} image after pulling"
|
|
|
|
command: "docker inspect {{ ceph_docker_registry}}/{{ ceph_docker_image }}:{{ ceph_docker_image_tag }}"
|
|
|
|
failed_when: false
|
|
|
|
register: container_inspect_after_pull
|
2018-01-08 17:00:25 +08:00
|
|
|
|
|
|
|
- name: set_fact repodigest_after_pulling
|
|
|
|
set_fact:
|
|
|
|
repodigest_after_pulling: "{{ (container_inspect_after_pull.stdout | from_json)[0].RepoDigests[0] }}"
|
|
|
|
when:
|
|
|
|
- container_inspect_after_pull.rc == 0
|
|
|
|
|
|
|
|
- name: set_fact is_image_updated
|
|
|
|
set_fact:
|
|
|
|
is_image_updated: "{{ repodigest_before_pulling == repodigest_after_pulling }}"
|
|
|
|
changed_when: true
|
2017-12-16 02:43:23 +08:00
|
|
|
notify:
|
2018-01-08 17:00:25 +08:00
|
|
|
- restart ceph mons
|
|
|
|
- restart ceph osds
|
|
|
|
- restart ceph mdss
|
|
|
|
- restart ceph rgws
|
|
|
|
- restart ceph mgrs
|
|
|
|
- restart ceph rbdmirrors
|
|
|
|
when:
|
|
|
|
- container_inspect_before_pull.rc == 0
|
|
|
|
- repodigest_before_pulling == repodigest_after_pulling
|
2017-12-16 02:43:23 +08:00
|
|
|
|
2016-03-24 21:21:07 +08:00
|
|
|
- name: export local ceph dev image
|
syntax: change local_action syntax
Use a nicer syntax for `local_action` tasks.
We used to have oneliner like this:
```
local_action: wait_for port=22 host={{ hostvars[inventory_hostname]['ansible_default_ipv4']['address'] }} state=started delay=10 timeout=500 }}
```
The usual syntax:
```
local_action:
module: wait_for
port: 22
host: "{{ hostvars[inventory_hostname]['ansible_default_ipv4']['address'] }}"
state: started
delay: 10
timeout: 500
```
is nicer and kind of way to keep consistency regarding the whole
playbook.
This also fix a potential issue about missing quotation :
```
Traceback (most recent call last):
File "/tmp/ansible_wQtWsi/ansible_module_command.py", line 213, in <module>
main()
File "/tmp/ansible_wQtWsi/ansible_module_command.py", line 185, in main
rc, out, err = module.run_command(args, executable=executable, use_unsafe_shell=shell, encoding=None, data=stdin)
File "/tmp/ansible_wQtWsi/ansible_modlib.zip/ansible/module_utils/basic.py", line 2710, in run_command
File "/usr/lib64/python2.7/shlex.py", line 279, in split
return list(lex) File "/usr/lib64/python2.7/shlex.py", line 269, in next
token = self.get_token()
File "/usr/lib64/python2.7/shlex.py", line 96, in get_token
raw = self.read_token()
File "/usr/lib64/python2.7/shlex.py", line 172, in read_token
raise ValueError, "No closing quotation"
ValueError: No closing quotation
```
writing `local_action: shell echo {{ fsid }} | tee {{ fetch_directory }}/ceph_cluster_uuid.conf`
can cause trouble because it's complaining with missing quotes, this fix solves this issue.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1510555
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
2018-01-31 16:23:28 +08:00
|
|
|
local_action:
|
|
|
|
module: command
|
|
|
|
docker save -o "/tmp/{{ ceph_docker_username }}-{{ ceph_docker_imagename }}-{{ ceph_docker_image_tag }}.tar" "{{ ceph_docker_username }}/{{ ceph_docker_imagename }}:{{ ceph_docker_image_tag }}"
|
2017-09-15 06:48:53 +08:00
|
|
|
when:
|
|
|
|
- (ceph_docker_dev_image is defined and ceph_docker_dev_image)
|
2016-03-24 21:21:07 +08:00
|
|
|
run_once: true
|
|
|
|
|
|
|
|
- name: copy ceph dev image file
|
|
|
|
copy:
|
2016-07-09 09:38:13 +08:00
|
|
|
src: "/tmp/{{ ceph_docker_username }}-{{ ceph_docker_imagename }}-{{ ceph_docker_image_tag }}.tar"
|
|
|
|
dest: "/tmp/{{ ceph_docker_username }}-{{ ceph_docker_imagename }}-{{ ceph_docker_image_tag }}.tar"
|
2017-09-15 06:48:53 +08:00
|
|
|
when:
|
|
|
|
- (ceph_docker_dev_image is defined and ceph_docker_dev_image)
|
2016-03-24 21:21:07 +08:00
|
|
|
|
|
|
|
- name: load ceph dev image
|
2016-07-09 09:38:13 +08:00
|
|
|
command: "docker load -i /tmp/{{ ceph_docker_username }}-{{ ceph_docker_imagename }}-{{ ceph_docker_image_tag }}.tar"
|
2017-09-15 06:48:53 +08:00
|
|
|
when:
|
|
|
|
- (ceph_docker_dev_image is defined and ceph_docker_dev_image)
|
2016-03-24 21:21:07 +08:00
|
|
|
|
|
|
|
- name: remove tmp ceph dev image file
|
2016-07-09 09:38:13 +08:00
|
|
|
command: "rm /tmp/{{ ceph_docker_username }}-{{ ceph_docker_imagename }}-{{ ceph_docker_image_tag }}.tar"
|
2017-09-15 06:48:53 +08:00
|
|
|
when:
|
|
|
|
- (ceph_docker_dev_image is defined and ceph_docker_dev_image)
|