Merge pull request #901 from galthaus/dns-tweak

DHCP Hook protections

roles/kubernetes/preinstall/tasks/dhclient-hooks-undo.yml

@@ -3,10 +3,6 @@
 # These tasks will undo changes done by kargo in the past if needed (e.g. when upgrading from kargo 2.0.x
 # or when changing resolvconf_mode)
 
-- name: Check if dhclient conf file exists
-  stat: path={{dhclientconffile}}
-  register: dhclientconf_check
-
 - name: Remove kargo specific config from dhclient config
   blockinfile:
     dest: "{{dhclientconffile}}"
@@ -14,7 +10,7 @@
     backup: yes
     follow: yes
     marker: "# Ansible entries {mark}"
-  when: dhclientconf_check.stat.exists
+  when: dhclientconffile is defined
   notify: Preinstall | restart network
 
 - name: Remove kargo specific dhclient hook

roles/kubernetes/preinstall/tasks/dhclient-hooks.yml

@@ -13,6 +13,7 @@
     follow: yes
     marker: "# Ansible entries {mark}"
   notify: Preinstall | restart network
+  when: dhclientconffile is defined
 
 - name: Configue dhclient hooks for resolv.conf (non-RH)
   template:

roles/kubernetes/preinstall/tasks/set_resolv_facts.yml

@@ -50,10 +50,14 @@
     dhclientconffile: /etc/dhclient.conf
   when: dhclient_stat.stat.exists
 
+- name: check if /etc/dhcp/dhclient.conf exists
+  stat: path=/etc/dhcp/dhclient.conf
+  register: dhcp_dhclient_stat
+
 - name: target dhclient conf file for /etc/dhcp/dhclient.conf
   set_fact:
     dhclientconffile: /etc/dhcp/dhclient.conf
-  when: not dhclient_stat.stat.exists
+  when: dhcp_dhclient_stat.stat.exists
 
 - name: target dhclient hook file for Red Hat family
   set_fact: