From 23b1f43897db0a03ef94f51e83ed3c562c4584d0 Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Wed, 20 Nov 2019 11:02:49 +0100 Subject: [PATCH] facts: avoid duplicated element in devices list When using `osd_auto_discovery`, `devices` is built multiple times due to multiple runs of `ceph-facts` role. It end up with duplicate instances of a same device in the list. Using `unique` filter when building the list fixes this issue. Signed-off-by: Guillaume Abrioux --- roles/ceph-facts/tasks/facts.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/ceph-facts/tasks/facts.yml b/roles/ceph-facts/tasks/facts.yml index 6020ce1d5..8b5b0da71 100644 --- a/roles/ceph-facts/tasks/facts.yml +++ b/roles/ceph-facts/tasks/facts.yml @@ -181,7 +181,7 @@ - name: set_fact devices generate device list when osd_auto_discovery set_fact: - devices: "{{ devices | default([]) + [ item.key | regex_replace('^', '/dev/') ] }}" + devices: "{{ (devices | default([]) + [ item.key | regex_replace('^', '/dev/') ]) | unique }}" with_dict: "{{ ansible_devices }}" when: - osd_auto_discovery | default(False) | bool -- 2.39.5