Add ceph_stable_releases array to ceph-common/defaults/main.yml

rework the various `when' tests to use that array.

diff --git a/roles/ceph-common/defaults/main.yml b/roles/ceph-common/defaults/main.yml
index f58e88e304b7139a019d339d8a28267d2e1ae404..4945b392921867fe0357e24eb4ccbf673156a5f4 100644 (file)
--- a/roles/ceph-common/defaults/main.yml
+++ b/roles/ceph-common/defaults/main.yml
@@ -45,6 +45,16 @@ ceph_stable: false # use ceph stable branch
 ceph_stable_key: https://download.ceph.com/keys/release.asc\r
 ceph_stable_release: infernalis # ceph stable release\r
 \r
+####################\r
+## Stable Releases #\r
+####################\r
+ceph_stable_releases:\r
+  - dumpling\r
+  - emperor\r
+  - firefly\r
+  - giant\r
+  - hammer\r
+\r
 # Use the option below to specify your applicable package tree, eg. when using non-LTS Ubuntu versions\r
 # # for a list of available Debian distributions, visit http://ceph.com/debian-{{ ceph_stable_release }}/dists/\r
 # for more info read: https://github.com/ceph/ceph-ansible/issues/305\r

diff --git a/roles/ceph-mds/tasks/pre_requisite.yml b/roles/ceph-mds/tasks/pre_requisite.yml
index 1a4bc967c1d12b0498b0b30cd311f0c3af117e13..3ef77dda4fea0597bb2d093e0c633db0a84ccb29 100644 (file)
--- a/roles/ceph-mds/tasks/pre_requisite.yml
+++ b/roles/ceph-mds/tasks/pre_requisite.yml
@@ -6,12 +6,7 @@
     owner: ceph
     group: ceph
     mode: 0755
-  when:
-    ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer'
+  when: ceph_stable_release not in ceph_stable_releases
 
 - name: create bootstrap-mds directory (before infernalis release)
   file:
@@ -20,13 +15,7 @@
     owner: root
     group: root
     mode: 0755
-  when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+  when: cephx and ceph_stable_release in ceph_stable_releases
 
 - name: copy mds bootstrap key (for or after infernalis release)
   copy:
@@ -36,12 +25,7 @@
     group: ceph
     mode: 0600
   when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: copy mds bootstrap key (before infernalis release)
   copy:
@@ -51,12 +35,7 @@
     group: root
     mode: 0600
   when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    cephx and ceph_stable_release in ceph_stable_releases
 
 - name: create mds directory (for or after infernalis release)
   file:
@@ -66,12 +45,7 @@
     group: ceph
     mode: 0755
   when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: create mds directory (before infernalis release)
   file:
@@ -81,12 +55,7 @@
     group: root
     mode: 0755
   when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    cephx and ceph_stable_release in ceph_stable_releases
 
 - name: create mds keyring
   command: ceph --cluster ceph --name client.bootstrap-mds --keyring /var/lib/ceph/bootstrap-mds/ceph.keyring auth get-or-create mds.{{ ansible_hostname }} osd 'allow rwx' mds 'allow' mon 'allow profile mds' -o /var/lib/ceph/mds/ceph-{{ ansible_hostname }}/keyring
@@ -103,11 +72,7 @@
     group: ceph
   when:
     cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ceph_stable_release not in ceph_stable_releases
 
 - name: set mds key permissions (before infernalis)
   file:
@@ -116,12 +81,7 @@
     owner: root
     group: root
   when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    cephx and ceph_stable_release in ceph_stable_releases
 
 - name: activate metadata server with upstart (for or after infernalis release)
   file:
@@ -136,11 +96,7 @@
   changed_when: false
   when:
     ansible_distribution == "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ceph_stable_release not in ceph_stable_releases
 
 - name: activate metadata server with upstart (before infernalis release)
   file:
@@ -154,12 +110,7 @@
     - upstart
   changed_when: false
   when:
-    ansible_distribution == "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ansible_distribution == "Ubuntu" and ceph_stable_release in ceph_stable_releases
 
 - name: activate metadata server with sysvinit (for or after infernalis release)
   file:
@@ -174,11 +125,7 @@
   changed_when: false
   when:
     ansible_distribution != "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ceph_stable_release not in ceph_stable_releases
 
 - name: activate metadata server with sysvinit (before infernalis release)
   file:
@@ -192,12 +139,7 @@
     - sysvinit
   changed_when: false
   when:
-    ansible_distribution != "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ansible_distribution != "Ubuntu" and ceph_stable_release in ceph_stable_releases
 
 - name: start and add that the metadata service to the init sequence (ubuntu)
   service:
@@ -215,13 +157,7 @@
     enabled: yes
     args: mds.{{ ansible_hostname }}
   changed_when: false
-  when: ansible_distribution != "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
-
+  when: ansible_distribution != "Ubuntu" and ceph_stable_release in ceph_stable_releases
 
 - name: start and add that the metadata service to the init sequence (for or after infernalis)
   service:
@@ -229,9 +165,4 @@
     state: started
     enabled: yes
   changed_when: false
-  when: ansible_distribution != "Ubuntu" and not
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+  when: ansible_distribution != "Ubuntu" and ceph_stable_release not in ceph_stable_releases

diff --git a/roles/ceph-mon/tasks/deploy_monitors.yml b/roles/ceph-mon/tasks/deploy_monitors.yml
index 3b18c42bc407cfa21a7abc2b90959a5079274a50..1982e4105e5754ab0879917fb72737c70171eba9 100644 (file)
--- a/roles/ceph-mon/tasks/deploy_monitors.yml
+++ b/roles/ceph-mon/tasks/deploy_monitors.yml
@@ -25,12 +25,7 @@
     mode: 0600
     owner: ceph
     group: ceph
-  when:
-    ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer'
+  when: ceph_stable_release not in ceph_stable_releases
 
 - name: create monitor directory (for or after infernalis release)
   file:
@@ -39,12 +34,7 @@
     owner: ceph
     group: ceph
     mode: 0755
-  when:
-    ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer'
+  when: ceph_stable_release not in ceph_stable_releases
 
 - name: set initial monitor key permissions (before infernalis release)
   file:
@@ -52,12 +42,7 @@
     mode: 0600
     owner: root
     group: root
-  when:
-    ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer'
+  when: ceph_stable_release in ceph_stable_releases
 
 - name: create monitor directory (before infernalis release)
   file:
@@ -66,31 +51,16 @@
     owner: root
     group: root
     mode: 0755
-  when:
-    ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer'
+  when: ceph_stable_release in ceph_stable_releases
 
 - name: ceph monitor mkfs (for or after infernalis release)
   command: ceph-mon --setuser ceph --setgroup ceph --mkfs -i {{ ansible_hostname }} --fsid {{ fsid }} --keyring /var/lib/ceph/tmp/keyring.mon.{{ ansible_hostname }}
   args:
     creates: /var/lib/ceph/mon/ceph-{{ ansible_hostname }}/keyring
-  when:
-    ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer'
+  when: ceph_stable_release not in ceph_stable_releases
 
 - name: ceph monitor mkfs (before infernalis release)
   command: ceph-mon --mkfs -i {{ ansible_hostname }} --fsid {{ fsid }} --keyring /var/lib/ceph/tmp/keyring.mon.{{ ansible_hostname }}
   args:
     creates: /var/lib/ceph/mon/ceph-{{ ansible_hostname }}/keyring
-  when:
-    ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer'
+  when: ceph_stable_release in ceph_stable_releases

diff --git a/roles/ceph-mon/tasks/start_monitor.yml b/roles/ceph-mon/tasks/start_monitor.yml
index cc5e93b0014c4684f168ac40234cef4a6471ca50..d77e9fca0534df76810af63a6f099d5cc0a127de 100644 (file)
--- a/roles/ceph-mon/tasks/start_monitor.yml
+++ b/roles/ceph-mon/tasks/start_monitor.yml
@@ -9,13 +9,8 @@
   with_items:
     - done
     - upstart
-  when:
-    ansible_distribution == "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+  when: 
+    ansible_distribution == "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
   changed_when: false
 
@@ -31,11 +26,7 @@
     - upstart
   when:
     ansible_distribution == "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ceph_stable_release in ceph_stable_releases
   changed_when: false
 
 - name: start and add that the monitor service to the init sequence (ubuntu)
@@ -53,11 +44,7 @@
   changed_when: false
   when:
     ansible_distribution != "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ceph_stable_release in ceph_stable_releases
 
 - name: start and add that the monitor service to the init sequence (for or after infernalis)
   service:
@@ -66,12 +53,8 @@
       enabled: yes
   changed_when: false
   when:
-    ansible_distribution != "Ubuntu" and not
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ansible_distribution != "Ubuntu" and
+    ceph_stable_release not in ceph_stable_releases
 
 - name: collect admin and bootstrap keys
   command: ceph-create-keys --id {{ ansible_hostname }}

diff --git a/roles/ceph-osd/tasks/pre_requisite.yml b/roles/ceph-osd/tasks/pre_requisite.yml
index 718c79fc19e93e63bc514797941d1ecd02fe1b97..97d3f0367c9f36563a82df43d63e5419fa74d86d 100644 (file)
--- a/roles/ceph-osd/tasks/pre_requisite.yml
+++ b/roles/ceph-osd/tasks/pre_requisite.yml
@@ -18,13 +18,7 @@
     owner: ceph
     group: ceph
     mode: 0755
-  when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+  when: cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: create bootstrap-osd directory (before infernalis release)
   file:
@@ -33,13 +27,7 @@
     owner: root
     group: root
     mode: 0755
-  when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+  when: cephx and ceph_stable_release in ceph_stable_releases
 
 - name: copy osd bootstrap key (for or after infernalis release)
   copy:
@@ -48,13 +36,7 @@
     owner: ceph
     group: ceph
     mode: 0600
-  when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+  when: cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: copy osd bootstrap key (before infernalis release)
   copy:
@@ -63,10 +45,4 @@
     owner: root
     group: root
     mode: 600
-  when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+  when: cephx and ceph_stable_release in ceph_stable_releases

diff --git a/roles/ceph-restapi/tasks/pre_requisite.yml b/roles/ceph-restapi/tasks/pre_requisite.yml
index 13aa693f395f4c3586ff82cb251e7b56e25287f5..4da21012ba8047b86734a915f65146cc7b44d2f8 100644 (file)
--- a/roles/ceph-restapi/tasks/pre_requisite.yml
+++ b/roles/ceph-restapi/tasks/pre_requisite.yml
@@ -6,12 +6,7 @@
     owner: ceph
     group: ceph
     mode: 0755
-  when:
-    ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer'
+  when: ceph_stable_release not in ceph_stable_releases
 
 - name: create ceph rest api directory (before infernalis release)
   file:
@@ -20,12 +15,7 @@
     owner: root
     group: root
     mode: 0755
-  when:
-    ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer'
+  when: ceph_stable_release in ceph_stable_releases
 
 - name: copy ceph rest api keyring (for or after infernalis release)
   copy:
@@ -35,12 +25,7 @@
     group: ceph
     mode: 600
   when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: copy ceph rest api keyring (before infernalis release)
   copy:
@@ -50,12 +35,7 @@
     group: root
     mode: 600
   when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    cephx and ceph_stable_release in ceph_stable_releases
 
 - name: activate ceph rest api with upstart (for or after infernalis release)
   file:
@@ -69,12 +49,7 @@
     - upstart
   changed_when: false
   when:
-    ansible_distribution == "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ansible_distribution == "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
 - name: activate ceph rest api with upstart (before infernalis release)
   file:
@@ -88,12 +63,7 @@
     - upstart
   changed_when: false
   when:
-    ansible_distribution == "Ubuntu" and
-    (ceph_stable_release != 'dumpling' or
-    ceph_stable_release != 'emperor' or
-    ceph_stable_release != 'firefly' or
-    ceph_stable_release != 'giant' or
-    ceph_stable_release != 'hammer')
+    ansible_distribution == "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
 - name: activate ceph rest api with sysvinit (for or after infernalis release))
   file:
@@ -106,12 +76,7 @@
     - done
     - sysvinit
   when:
-    ansible_distribution != "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ansible_distribution != "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
 - name: activate ceph rest api with sysvinit (before infernalis release))
   file:
@@ -124,12 +89,7 @@
     - done
     - sysvinit
   when:
-    ansible_distribution != "Ubuntu" and
-    (ceph_stable_release != 'dumpling' or
-    ceph_stable_release != 'emperor' or
-    ceph_stable_release != 'firefly' or
-    ceph_stable_release != 'giant' or
-    ceph_stable_release != 'hammer')
+    ansible_distribution != "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
 # NOTE (leseb): will uncomment this when this https://github.com/ceph/ceph/pull/4144 lands
 #- name: start and add that the Ceph REST API service to the init sequence (Ubuntu)

diff --git a/roles/ceph-rgw/tasks/pre_requisite.yml b/roles/ceph-rgw/tasks/pre_requisite.yml
index 89d47e365fed907b88b35529d968fc0bef03d195..9a415f255adb51a3e24a70c57b6a2e8246b623aa 100644 (file)
--- a/roles/ceph-rgw/tasks/pre_requisite.yml
+++ b/roles/ceph-rgw/tasks/pre_requisite.yml
@@ -9,12 +9,7 @@
   with_items:
     - /var/lib/ceph/bootstrap-rgw
     - /var/lib/ceph/radosgw/ceph-rgw.{{ ansible_hostname }}
-  when:
-    ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer'
+  when: ceph_stable_release not in ceph_stable_releases
 
 - name: create rados gateway directories (before infernalis release)
   file:
@@ -26,12 +21,7 @@
   with_items:
     - /var/lib/ceph/bootstrap-rgw
     - /var/lib/ceph/radosgw/ceph-rgw.{{ ansible_hostname }}
-  when:
-    ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer'
+  when: ceph_stable_release in ceph_stable_releases
 
 - name: copy rados gateway bootstrap key (for or after infernalis release)
   copy:
@@ -40,13 +30,7 @@
     owner: ceph
     group: ceph
     mode: 0600
-  when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+  when: cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: copy rados gateway bootstrap key (before infernalis release)
   copy:
@@ -55,13 +39,7 @@
     owner: root
     group: root
     mode: 0600
-  when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+  when: cephx and ceph_stable_release in ceph_stable_releases
 
 - name: create rados gateway keyring
   command: ceph --cluster ceph --name client.bootstrap-rgw --keyring /var/lib/ceph/bootstrap-rgw/ceph.keyring auth get-or-create client.rgw.{{ ansible_hostname }} osd 'allow rwx' mon 'allow rw' -o /var/lib/ceph/radosgw/ceph-rgw.{{ ansible_hostname }}/keyring
@@ -76,13 +54,7 @@
     mode: 0600
     owner: ceph
     group: ceph
-  when:
-    cephx and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+  when: cephx and ceph_stable_release not in ceph_stable_releases
 
 - name: set rados gateway key permissions (before infernalis release)
   file:
@@ -90,13 +62,7 @@
     mode: 0600
     owner: root
     group: root
-  when:
-    cephx and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+  when: cephx and ceph_stable_release in ceph_stable_releases
 
 - name: activate rados gateway with upstart (for or after infernalis release)
   file:
@@ -110,12 +76,7 @@
     - upstart
   changed_when: false
   when:
-    ansible_distribution == "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ansible_distribution == "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
 - name: activate rados gateway with upstart (before infernalis release)
   file:
@@ -129,12 +90,7 @@
     - upstart
   changed_when: false
   when:
-    ansible_distribution == "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ansible_distribution == "Ubuntu" and ceph_stable_release in ceph_stable_releases
 
 - name: activate rados gateway with sysvinit (for or after infernalis release)
   file:
@@ -148,12 +104,7 @@
     - sysvinit
   changed_when: false
   when:
-    ansible_distribution != "Ubuntu" and
-    (ceph_stable_release != 'dumpling' and
-    ceph_stable_release != 'emperor' and
-    ceph_stable_release != 'firefly' and
-    ceph_stable_release != 'giant' and
-    ceph_stable_release != 'hammer')
+    ansible_distribution != "Ubuntu" and ceph_stable_release not in ceph_stable_releases
 
 - name: activate rados gateway with sysvinit (before infernalis release)
   file:
@@ -167,12 +118,7 @@
     - sysvinit
   changed_when: false
   when:
-    ansible_distribution != "Ubuntu" and
-    (ceph_stable_release == 'dumpling' or
-    ceph_stable_release == 'emperor' or
-    ceph_stable_release == 'firefly' or
-    ceph_stable_release == 'giant' or
-    ceph_stable_release == 'hammer')
+    ansible_distribution != "Ubuntu" and ceph_stable_release in ceph_stable_releases
 
 - name: generate rados gateway sudoers file
   template: