Merge "Specify 'become' for all tasks that use kolla_docker module"

ansible/roles/haproxy/tasks/config.yml

@@ -94,6 +94,7 @@
     - Ensuring latest haproxy config is used
 
 - name: Check haproxy containers
+  become: true
   kolla_docker:
     action: "compare_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/haproxy/tasks/pull.yml

 ---
 - name: Pulling haproxy images
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"

ansible/roles/haproxy/tasks/upgrade.yml

@@ -4,6 +4,7 @@
 - set_fact: secondary_addresses={{ hostvars[inventory_hostname]['ansible_' + api_interface].get('ipv4_secondaries', []) | map(attribute='address') | list }}
 
 - name: Stopping all slave keepalived containers
+  become: true
   kolla_docker:
     action: "stop_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/heat/handlers/main.yml

@@ -7,6 +7,7 @@
     heat_conf: "{{ heat_confs.results|selectattr('item.key', 'equalto', service_name)|first }}"
     policy_overwriting: "{{ heat_policy_overwriting.results|selectattr('item.key', 'equalto', service_name)|first }}"
     heat_api_container: "{{ check_heat_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"
@@ -30,6 +31,7 @@
     heat_conf: "{{ heat_confs.results|selectattr('item.key', 'equalto', service_name)|first }}"
     policy_overwriting: "{{ heat_policy_overwriting.results|selectattr('item.key', 'equalto', service_name)|first }}"
     heat_api_cfn_container: "{{ check_heat_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"
@@ -53,6 +55,7 @@
     heat_conf: "{{ heat_confs.results|selectattr('item.key', 'equalto', service_name)|first }}"
     policy_overwriting: "{{ heat_policy_overwriting.results|selectattr('item.key', 'equalto', service_name)|first }}"
     heat_engine_container: "{{ check_heat_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/heat/tasks/bootstrap_service.yml

@@ -2,6 +2,7 @@
 - name: Running Heat bootstrap container
   vars:
     heat_api: "{{ heat_services['heat-api'] }}"
+  become: true
   kolla_docker:
     action: "start_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/heat/tasks/config.yml

@@ -95,6 +95,7 @@
     - Restart heat-engine container
 
 - name: Check heat containers
+  become: true
   kolla_docker:
     action: "compare_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/heat/tasks/pull.yml

 ---
 - name: Pulling heat images
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"

ansible/roles/horizon/handlers/main.yml

@@ -3,6 +3,7 @@
   vars:
     service_name: "horizon"
     service: "{{ horizon_services[service_name] }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/horizon/tasks/bootstrap_service.yml

@@ -2,6 +2,7 @@
 - name: Running Horizon bootstrap container
   vars:
     horizon: "{{ horizon_services['horizon'] }}"
+  become: true
   kolla_docker:
     action: "start_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/horizon/tasks/config.yml

@@ -136,6 +136,7 @@
 - name: Check horizon container
   vars:
     horizon: "{{ horizon_services['horizon'] }}"
+  become: true
   kolla_docker:
     action: "compare_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/horizon/tasks/pull.yml

 ---
 - name: Pulling horizon images
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"

ansible/roles/influxdb/handlers/main.yml

@@ -5,6 +5,7 @@
     service: "{{ influxdb_services[service_name] }}"
     config_json: "{{ influxdb_config_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     influxdb_container: "{{ check_influxdb_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/influxdb/tasks/config.yml

@@ -46,6 +46,7 @@
     - Restart influxdb container
 
 - name: Check influxdb containers
+  become: true
   kolla_docker:
     action: "compare_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/influxdb/tasks/pull.yml

 ---
 - name: Pulling influxdb image
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"

ansible/roles/ironic/handlers/main.yml

@@ -7,6 +7,7 @@
     ironic_conf: "{{ ironic_confs.results|selectattr('item.key', 'equalto', service_name)|first }}"
     policy_json: "{{ ironic_policy_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     ironic_api_container: "{{ check_ironic_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"
@@ -30,6 +31,7 @@
     ironic_conf: "{{ ironic_confs.results|selectattr('item.key', 'equalto', service_name)|first }}"
     policy_json: "{{ ironic_policy_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     ironic_conductor_container: "{{ check_ironic_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"
@@ -53,6 +55,7 @@
     config_json: "{{ ironic_config_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     policy_json: "{{ ironic_policy_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     ironic_inspector_container: "{{ check_ironic_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"
@@ -75,6 +78,7 @@
     service: "{{ ironic_services[service_name] }}"
     config_json: "{{ ironic_config_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     ironic_pxe_container: "{{ check_ironic_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"
@@ -119,6 +123,7 @@
     service: "{{ ironic_services[service_name] }}"
     config_json: "{{ ironic_config_jsons.results|selectattr('item.key', 'equalto', service_name)|first }}"
     ironic_dnsmasq_container: "{{ check_ironic_containers.results|selectattr('item.key', 'equalto', service_name)|first }}"
+  become: true
   kolla_docker:
     action: "recreate_or_restart_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/ironic/tasks/bootstrap.yml

@@ -54,6 +54,7 @@
 - name: Running Ironic-PXE bootstrap container
   vars:
     ironic_pxe: "{{ ironic_services['ironic-pxe'] }}"
+  become: true
   kolla_docker:
     action: "start_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/ironic/tasks/bootstrap_service.yml

@@ -2,6 +2,7 @@
 - name: Running Ironic bootstrap container
   vars:
     ironic_api: "{{ ironic_services['ironic-api'] }}"
+  become: true
   kolla_docker:
     action: "start_container"
     common_options: "{{ docker_common_options }}"
@@ -22,6 +23,7 @@
 - name: Running Ironic Inspector bootstrap container
   vars:
     ironic_inspector: "{{ ironic_services['ironic-inspector'] }}"
+  become: true
   kolla_docker:
     action: "start_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/ironic/tasks/config.yml

@@ -251,6 +251,7 @@
     - "Restart {{ item.key }} container"
 
 - name: Check ironic containers
+  become: true
   kolla_docker:
     action: "compare_container"
     common_options: "{{ docker_common_options }}"

ansible/roles/ironic/tasks/pull.yml

 ---
 - name: Pulling ironic images
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"

ansible/roles/iscsi/tasks/pull.yml

 ---
 - name: Pulling iscsid image
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"
@@ -7,6 +8,7 @@
   when: inventory_hostname in groups['iscsid']
 
 - name: Pulling tgtd image
+  become: true
   kolla_docker:
     action: "pull_image"
     common_options: "{{ docker_common_options }}"