Fix value of ctlplane fqdn fact
authorJuan Antonio Osorio Robles <jaosorior@redhat.com>
Fri, 9 Sep 2016 05:40:06 +0000 (08:40 +0300)
committerJuan Antonio Osorio Robles <jaosorior@redhat.com>
Mon, 12 Sep 2016 11:04:32 +0000 (14:04 +0300)
This fact was being retrieving the value of the hostname for the management
network. We should instead be using a value set explicitly in t-h-t.

Depends-On: Idb3ca22ac136691b0bff6f94524d133a4fa10617
Change-Id: I6fcf7c7853071a9f3377aec475308bc8d10d5b33
Related-Bug: #1621742

lib/facter/alt_fqdns.rb

index 216beef..2d78b9a 100644 (file)
   'storage_mgmt',
   'tenant',
   'management',
+  'ctlplane',
 ].each do |network|
   Facter.add('fqdn_' + network) do
     setcode do
-      external_hostname_parts = [
+      hostname_parts = [
         Facter.value(:hostname),
         network.gsub('_', ''),
         Facter.value(:domain),
       ].reject { |part| part.nil? || part.empty? }
-      external_hostname_parts.join(".")
+      hostname_parts.join(".")
     end
   end
 end
-# map ctlplane network to management fqdn
-Facter.add('fqdn_ctlplane') do
-  setcode do
-    hostname_parts = [
-      Facter.value(:hostname),
-      'management',
-      Facter.value(:domain),
-    ].reject { |part| part.nil? || part.empty? }
-    hostname_parts.join(".")
-  end
-end