Use allovercloud ansible group

With certmonger management moving to ansible, we lose the
"certmonger_user" service, hence the ansible inventory group.

We can therefore use the "allovercloud" group instead - it will clean
the OC nodes while preserving the undercloud node.

This patch also drops the "loop" setting the "hosts_list" fact in favor
of a more efficient |map() filter.

Resolves: rhbz#2017849
Change-Id: Icd662f74fc40bb6c1b5e4e01a236e1994ca3cc83

tripleo_ipa/playbooks/cli-cleanup-ipa.yml

@@ -57,8 +57,7 @@
 
     - name: create list of hosts to clean up in IPA
       set_fact:
-        hosts_list: "{{ hosts_list + [ hostvars[item]['canonical_hostname'] ] }}"
-      loop: "{{ groups.certmonger_user | default([]) }}"
+        hosts_list: "{{ groups.allovercloud | map('extract', hostvars, 'canonical_hostname') | list }}"
 
     - name: import cleanup tasks from the tripleo-ipa role
       include_role: