Skip to content

Commit

Permalink
Bug 1928147: Respect DHCP option 119 for on-prem
Browse files Browse the repository at this point in the history
Signed-off-by: Ben Howard <ben.howard@redhat.com>
  • Loading branch information
Ben Howard authored and openshift-cherrypick-robot committed Jan 10, 2022
1 parent 51dc080 commit e609684
Showing 1 changed file with 3 additions and 3 deletions.
Expand Up @@ -50,15 +50,15 @@ contents:
show \
"{{ onPremPlatformAPIServerInternalIP . }}" \
"{{ onPremPlatformIngressIP . }}")
DOMAIN="{{.DNS.Spec.BaseDomain}}"
DOMAINS="${IP4_DOMAINS} ${IP6_DOMAINS} {{.DNS.Spec.BaseDomain}}"
if [[ -n "$NAMESERVER_IP" ]]; then
if systemctl -q is-enabled systemd-resolved; then
>&2 echo "NM resolv-prepender: Setting up systemd-resolved for OKD domain and local IP"
if [[ ! -f /etc/systemd/resolved.conf.d/60-kni.conf ]]; then
>&2 echo "NM resolv-prepender: Creating /etc/systemd/resolved.conf.d/60-kni.conf"
echo "[Resolve]" > /etc/systemd/resolved.conf.d/60-kni.conf
echo "DNS=$NAMESERVER_IP" >> /etc/systemd/resolved.conf.d/60-kni.conf
echo "Domains=$DOMAIN" >> /etc/systemd/resolved.conf.d/60-kni.conf
echo "Domains=$DOMAINS" >> /etc/systemd/resolved.conf.d/60-kni.conf
if systemctl -q is-active systemd-resolved; then
>&2 echo "NM resolv-prepender: restarting systemd-resolved"
systemctl restart systemd-resolved
Expand All @@ -67,7 +67,7 @@ contents:
else
>&2 echo "NM resolv-prepender: Prepending 'nameserver $NAMESERVER_IP' to /etc/resolv.conf (other nameservers from /var/run/NetworkManager/resolv.conf)"
sed -e "/^search/d" \
-e "/Generated by/c# Generated by KNI resolv prepender NM dispatcher script\nsearch $DOMAIN\nnameserver $NAMESERVER_IP" \
-e "/Generated by/c# Generated by KNI resolv prepender NM dispatcher script\nsearch $DOMAINS\nnameserver $NAMESERVER_IP" \
/var/run/NetworkManager/resolv.conf > /etc/resolv.tmp
# Only leave the first 3 nameservers in /etc/resolv.conf
sed -i ':a $!{N; ba}; s/\(^\|\n\)nameserver/\n# nameserver/4g' /etc/resolv.tmp
Expand Down

0 comments on commit e609684

Please sign in to comment.