diff options
| author | Emanuele Giuseppe Esposito <eesposit@redhat.com> | 2021-12-06 18:34:26 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-12-06 11:34:26 -0600 |
| commit | 0e25076b34fa995161b83996e866c0974cee431f (patch) | |
| tree | c6f05f7ba56d61f42921b96f4a1714a612adbde2 /cloudinit/net | |
| parent | 6e39613da5cee66f2162f53c53358f2516b904d0 (diff) | |
| download | vyos-cloud-init-0e25076b34fa995161b83996e866c0974cee431f.tar.gz vyos-cloud-init-0e25076b34fa995161b83996e866c0974cee431f.zip | |
cloudinit/net: handle two different routes for the same ip (#1124)
If we set a dhcp server side like this:
$ cat /var/tmp/cloud-init/cloud-init-dhcp-f0rie5tm/dhcp.leases
lease {
...
option classless-static-routes 31.169.254.169.254 0.0.0.0,31.169.254.169.254
10.112.143.127,22.10.112.140 0.0.0.0,0 10.112.140.1;
...
}
cloud-init fails to configure the routes via 'ip route add' because to there are
two different routes for 169.254.169.254:
$ ip -4 route add 192.168.1.1/32 via 0.0.0.0 dev eth0
$ ip -4 route add 192.168.1.1/32 via 10.112.140.248 dev eth0
But NetworkManager can handle such scenario successfully as it uses "ip route append".
So change cloud-init to also use "ip route append" to fix the issue:
$ ip -4 route append 192.168.1.1/32 via 0.0.0.0 dev eth0
$ ip -4 route append 192.168.1.1/32 via 10.112.140.248 dev eth0
Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
RHBZ: #2003231
Diffstat (limited to 'cloudinit/net')
| -rw-r--r-- | cloudinit/net/__init__.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cloudinit/net/__init__.py b/cloudinit/net/__init__.py index 7558745f..f81f3a7b 100644 --- a/cloudinit/net/__init__.py +++ b/cloudinit/net/__init__.py @@ -1157,7 +1157,7 @@ class EphemeralIPv4Network(object): if gateway != "0.0.0.0": via_arg = ['via', gateway] subp.subp( - ['ip', '-4', 'route', 'add', net_address] + via_arg + + ['ip', '-4', 'route', 'append', net_address] + via_arg + ['dev', self.interface], capture=True) self.cleanup_cmds.insert( 0, ['ip', '-4', 'route', 'del', net_address] + via_arg + |