From: Guillaume Abrioux <gabrioux@redhat.com>
Date: Mon, 11 Jan 2021 13:57:00 +0000 (+0100)
Subject: osd: support removing ec_profiles
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=3a57908b4d6b8a4c0c1a510866d604b44e8ba944;p=ceph-ansible.git

osd: support removing ec_profiles

Take `state` as a parameter to be passed to the module call so it is
possible to remove an ec profile.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
---

diff --git a/group_vars/all.yml.sample b/group_vars/all.yml.sample
index 18e562746..8995dc704 100644
--- a/group_vars/all.yml.sample
+++ b/group_vars/all.yml.sample
@@ -370,6 +370,7 @@ dummy:
 # ec_profiles:
 #   - {'name': 'my_ec_profile1', 'k': 2, 'm': 4, 'stripe_unit': 8192}
 #   - {'name': 'my_ec_profile2', 'k': 4, 'm': 8, 'crush_root': 'HDD', 'crush_device_class': 'ssd'}
+#   - {'name': 'foobar', 'state': 'absent'}
 
 ## MDS options
 #
diff --git a/group_vars/rhcs.yml.sample b/group_vars/rhcs.yml.sample
index 61208f844..5686ae36b 100644
--- a/group_vars/rhcs.yml.sample
+++ b/group_vars/rhcs.yml.sample
@@ -370,6 +370,7 @@ ceph_iscsi_config_dev: false
 # ec_profiles:
 #   - {'name': 'my_ec_profile1', 'k': 2, 'm': 4, 'stripe_unit': 8192}
 #   - {'name': 'my_ec_profile2', 'k': 4, 'm': 8, 'crush_root': 'HDD', 'crush_device_class': 'ssd'}
+#   - {'name': 'foobar', 'state': 'absent'}
 
 ## MDS options
 #
diff --git a/roles/ceph-defaults/defaults/main.yml b/roles/ceph-defaults/defaults/main.yml
index 1d2b3f913..d9c0bbb37 100644
--- a/roles/ceph-defaults/defaults/main.yml
+++ b/roles/ceph-defaults/defaults/main.yml
@@ -362,6 +362,7 @@ filestore_xattr_use_omap: null
 # ec_profiles:
 #   - {'name': 'my_ec_profile1', 'k': 2, 'm': 4, 'stripe_unit': 8192}
 #   - {'name': 'my_ec_profile2', 'k': 4, 'm': 8, 'crush_root': 'HDD', 'crush_device_class': 'ssd'}
+#   - {'name': 'foobar', 'state': 'absent'}
 
 ## MDS options
 #
diff --git a/roles/ceph-osd/tasks/main.yml b/roles/ceph-osd/tasks/main.yml
index 67bd449d4..bab0d94b0 100644
--- a/roles/ceph-osd/tasks/main.yml
+++ b/roles/ceph-osd/tasks/main.yml
@@ -108,7 +108,7 @@
   ceph_ec_profile:
     cluster: "{{ cluster }}"
     name: "{{ item.name }}"
-    state: present
+    state: "{{ item.state | default(omit) }}"
     stripe_unit: "{{ item.stripe_unit | default(omit) }}"
     k: "{{ item.k }}"
     m: "{{ item.m }}"