From cb784c601d2063b95fb7d2514e39518137164e12 Mon Sep 17 00:00:00 2001 From: Ramana Raja Date: Mon, 3 Dec 2018 19:55:42 +0530 Subject: [PATCH] rolling_update: fail if less than 3 MONs ... for non-containerized deployments as well. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1655470 Signed-off-by: Ramana Raja --- infrastructure-playbooks/rolling_update.yml | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/infrastructure-playbooks/rolling_update.yml b/infrastructure-playbooks/rolling_update.yml index 58246eeab..98ad833cb 100644 --- a/infrastructure-playbooks/rolling_update.yml +++ b/infrastructure-playbooks/rolling_update.yml @@ -85,15 +85,11 @@ - name: set mon_host_count set_fact: mon_host_count={{ groups[mon_group_name] | length }} - - debug: msg="WARNING - upgrading a ceph cluster with only one monitor node ({{ inventory_hostname }})" - when: mon_host_count | int == 1 - - - name: fail when single containerized monitor + - name: fail when less than three monitors fail: - msg: "Upgrades of a single monitor are not supported, also running 1 monitor is not recommended always use 3." + msg: "Upgrade of cluster with less than three monitors is not supported." when: - - containerized_deployment - - mon_host_count | int == 1 + - mon_host_count | int < 3 - name: stop ceph mon - shortname systemd: @@ -186,9 +182,9 @@ - name: set mon_host_count set_fact: mon_host_count={{ groups[mon_group_name] | length }} - - name: select a running monitor if multiple monitors + - name: select a running monitor set_fact: - mon_host: "{{ groups[mon_group_name] | difference([inventory_hostname]) | last if mon_host_count > 1 else groups[mon_group_name] | last }}" + mon_host: "{{ groups[mon_group_name] | difference([inventory_hostname]) | last }}" - name: non container | waiting for the monitor to join the quorum... command: ceph --cluster "{{ cluster }}" -s --format json -- 2.39.5