From 77a3a682f358c8e9a40c5b50e980b5e9ec5f6d60 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Han?= <seb@redhat.com>
Date: Mon, 20 Aug 2018 15:53:03 +0200
Subject: [PATCH] iscsi group name preserve backward compatibility
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Recently we renamed the group_name for iscsi iscsigws where previously
it was named iscsi-gws. Existing deployments with a host file section
with iscsi-gws must continue to work.

This commit adds the old group name as a backoward compatility, no error
from Ansible should be expected, if the hostgroup is not found nothing
is played.

Close: https://bugzilla.redhat.com/show_bug.cgi?id=1619167
Signed-off-by: Sébastien Han <seb@redhat.com>
---
 infrastructure-playbooks/purge-iscsi-gateways.yml | 4 +++-
 infrastructure-playbooks/rolling_update.yml       | 1 +
 site-docker.yml.sample                            | 5 ++++-
 site.yml.sample                                   | 5 ++++-
 4 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/infrastructure-playbooks/purge-iscsi-gateways.yml b/infrastructure-playbooks/purge-iscsi-gateways.yml
index fa86d126e..369901430 100644
--- a/infrastructure-playbooks/purge-iscsi-gateways.yml
+++ b/infrastructure-playbooks/purge-iscsi-gateways.yml
@@ -20,7 +20,9 @@
         igw_purge_type: "{{ purge_config }}"
 
 - name: Removing the gateway configuration
-  hosts: iscsigws
+  hosts:
+    - iscsigws
+    - iscsi-gws # for backward compatibility only!
   become: yes
   vars:
     - igw_purge_type: "{{hostvars['localhost']['igw_purge_type']}}"
diff --git a/infrastructure-playbooks/rolling_update.yml b/infrastructure-playbooks/rolling_update.yml
index 92a0e63a4..c9438d713 100644
--- a/infrastructure-playbooks/rolling_update.yml
+++ b/infrastructure-playbooks/rolling_update.yml
@@ -697,6 +697,7 @@
 
   hosts:
     - "{{ iscsi_gw_group_name|default('iscsigws') }}"
+    - iscsi-gws # for backward compatibility only!
 
   serial: 1
   become: True
diff --git a/site-docker.yml.sample b/site-docker.yml.sample
index 6eae75683..a76d1e7b4 100644
--- a/site-docker.yml.sample
+++ b/site-docker.yml.sample
@@ -12,6 +12,7 @@
   - rbdmirrors
   - clients
   - iscsigws
+  - iscsi-gws # for backward compatibility only!
   - mgrs
 
   gather_facts: false
@@ -331,7 +332,9 @@
             status: "Complete"
             end: "{{ lookup('pipe', 'date +%Y%m%d%H%M%SZ') }}"
 
-- hosts: iscsigws
+- hosts:
+    - iscsigws
+    - iscsi-gws # for backward compatibility only!
   gather_facts: false
   become: True
   pre_tasks:
diff --git a/site.yml.sample b/site.yml.sample
index f5782acec..fdba38d93 100644
--- a/site.yml.sample
+++ b/site.yml.sample
@@ -13,6 +13,7 @@
   - clients
   - mgrs
   - iscsigws
+  - iscsi-gws # for backward compatibility only!
 
   gather_facts: false
   any_errors_fatal: true
@@ -358,7 +359,9 @@
             status: "Complete"
             end: "{{ lookup('pipe', 'date +%Y%m%d%H%M%SZ') }}"
 
-- hosts: iscsigws
+- hosts:
+    - iscsigws
+    - iscsi-gws # for backward compatibility only!
   gather_facts: false
   become: True
   pre_tasks: