From: David Galloway <dgallowa@redhat.com>
Date: Thu, 11 Jun 2020 19:14:53 +0000 (-0400)
Subject: generate-fog-csv: Only print a line if the MAC is defined
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=182f99eeccf57d829fefb7c0534485cf15ef64b6;p=ceph-cm-ansible.git

generate-fog-csv: Only print a line if the MAC is defined

There's some hosts that are "cobbler_managed" but don't have a MAC and don't need a MAC.  Ansible doesn't help us out any here and print WHICH host doesn't have a MAC.  Just throws an error.

Also, there's no need to gather facts on localhost so I set that to `false`

Signed-off-by: David Galloway <dgallowa@redhat.com>
---

diff --git a/tools/generate-fog-csv.yml b/tools/generate-fog-csv.yml
index c859448..d866261 100644
--- a/tools/generate-fog-csv.yml
+++ b/tools/generate-fog-csv.yml
@@ -7,3 +7,4 @@
   roles:
     - generate-fog-csv
   become: false
+  gather_facts: false
diff --git a/tools/roles/generate-fog-csv/templates/csv.j2 b/tools/roles/generate-fog-csv/templates/csv.j2
index 67947b5..61ee4f4 100644
--- a/tools/roles/generate-fog-csv/templates/csv.j2
+++ b/tools/roles/generate-fog-csv/templates/csv.j2
@@ -1,3 +1,5 @@
 {% for host in groups['cobbler_managed'] %}
+{% if hostvars[host]['mac'] is defined %}
 "{{ hostvars[host]['mac'] }}","{{ hostvars[host]['inventory_hostname_short'] }}","","","1","0","","","fog","","","","","","","","","{{ hostvars[host]['kernel_options'] }}","","{{ hostvars[host]['fog_install_drive']|default('/dev/sda') }}","","","","","0000-00-00 00:00:00","110","","",""
+{% endif %}
 {% endfor %}