Skip to content

Comments

systemd/network: Disable RoutesToDNS#109

Merged
jepio merged 1 commit intoflatcar-masterfrom
azure-multinic-fix
Oct 6, 2023
Merged

systemd/network: Disable RoutesToDNS#109
jepio merged 1 commit intoflatcar-masterfrom
azure-multinic-fix

Conversation

@jepio
Copy link
Member

@jepio jepio commented Oct 4, 2023

systemd/network: Disable RoutesToDNS

Flatcar VMs on Azure with multiple nics fail to provision correctly because of multiple routes to wireserver (168.63.129.16) with the same metric, and wireserver only responds to 'ready' signals on the primary nic. The primary nic gets an explicit route to wireserver through dhcp. It turns out systemd is adding the extra routes because wireserver is also the dhcp and dns server, and the default RoutesToDNS=true setting is the reason.

This behavior is non-standard and I don't think it's expected on any platform. Disable this behavior by default.

How to use

az vm create --nics ....

Testing done

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

Flatcar VMs on Azure with multiple nics fail to provision correctly because of
multiple routes to wireserver (168.63.129.16) with the same metric, and
wireserver only responds to 'ready' signals on the primary nic. The primary nic
gets an explicit route to wireserver through dhcp. It turns out systemd is
adding the extra routes because wireserver is also the dhcp and dns server, and
the default RoutesToDNS=true setting is the reason.

This behavior is non-standard and I don't think it's expected on any platform.
Disable this behavior by default.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
@jepio jepio requested a review from a team October 4, 2023 09:22
jepio added a commit to flatcar/scripts that referenced this pull request Oct 4, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
@jepio jepio merged commit 58b5c3c into flatcar-master Oct 6, 2023
jepio added a commit to flatcar/scripts that referenced this pull request Oct 9, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
@krnowak krnowak deleted the azure-multinic-fix branch October 10, 2023 05:52
jepio added a commit to flatcar/scripts that referenced this pull request Oct 10, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
jepio added a commit to flatcar/scripts that referenced this pull request Oct 11, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
jepio added a commit to flatcar/scripts that referenced this pull request Oct 12, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
jepio added a commit to flatcar/scripts that referenced this pull request Oct 12, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
jepio added a commit to flatcar/scripts that referenced this pull request Oct 12, 2023
Pull in this PR flatcar/init#109 to disable a
systemd-networkd setting causing unexpected routes to appear.

Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants