Make 'disable ssl for dashboard task' idempotent.

This should reduce number of 'changed' tasks during convergence test.

Signed-off-by: George Shuklin <george.shuklin@gmail.com>
pull/5705/head
George Shuklin 2020-07-13 13:40:17 +03:00 committed by Guillaume Abrioux
parent 55cd6e83e4
commit 73d4bb6bd6
1 changed files with 12 additions and 3 deletions

View File

@ -5,10 +5,19 @@
when: containerized_deployment | bool when: containerized_deployment | bool
- name: disable SSL for dashboard - name: disable SSL for dashboard
command: "{{ container_exec_cmd }} ceph --cluster {{ cluster }} config set mgr mgr/dashboard/ssl false"
delegate_to: "{{ groups[mon_group_name][0] }}"
run_once: true
when: dashboard_protocol == "http" when: dashboard_protocol == "http"
delegate_to: "{{ groups[mon_group_name][0] }}"
block:
- name: get SSL status for dashboard
run_once: true
command: "{{ container_exec_cmd }} ceph --cluster {{ cluster }} config get mgr mgr/dashboard/ssl"
changed_when: false
register: current_ssl_for_dashboard
- name: disable SSL for dashboard
run_once: true
command: "{{ container_exec_cmd }} ceph --cluster {{ cluster }} config set mgr mgr/dashboard/ssl false"
when: current_ssl_for_dashboard.stdout == "true"
- name: with SSL for dashboard - name: with SSL for dashboard
when: dashboard_protocol == "https" when: dashboard_protocol == "https"