Skip to content

Commit

Permalink
Merge pull request openshift#10608 from tzumainn/openstack-restart-do…
Browse files Browse the repository at this point in the history
…cker-storage-setup

Restart docker after openstack storage setup
  • Loading branch information
openshift-merge-robot authored Nov 8, 2018
2 parents 85de160 + 4728696 commit 18a6b6b
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
2 changes: 2 additions & 0 deletions roles/openshift_openstack/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@ openshift_openstack_lbaasv2_provider: Octavia
openshift_openstack_use_vm_load_balancer: false
openshift_openstack_api_lb_listeners_timeout: 500000

openshift_docker_service_name: "docker"

# container-storage-setup
openshift_openstack_container_storage_setup:
docker_dev: "/dev/sdb"
Expand Down
11 changes: 11 additions & 0 deletions roles/openshift_openstack/tasks/container-storage-setup.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,3 +35,14 @@
# TODO(shadower): Find out which CentOS version supports overlayfs2
when:
- ansible_distribution == "CentOS"

- name: restart docker after storage configuration
become: yes
systemd:
name: "{{ openshift_docker_service_name }}"
state: restarted
register: l_docker_restart_docker_in_storage_setup_result
until: not (l_docker_restart_docker_in_storage_setup_result is failed)
retries: 3
delay: 30
when: not openshift_use_crio_only|default(None)

0 comments on commit 18a6b6b

Please sign in to comment.