l23network provider doesn't flush def routes if someone add def route with metric

[Stanislav Makar]

Added by hand the route
0.0.0.0 10.109.0.222 0.0.0.0 UG 33 0 0 br-fw-admin

Finally we have
[root@node-4 ~]# route -n
Kernel IP routing table
Destination Gateway Genmask Flags Metric Ref Use Iface
169.254.169.254 - 255.255.255.255 !H 0 - 0 -
192.168.1.0 0.0.0.0 255.255.255.0 U 0 0 0 br-storage
192.168.0.0 0.0.0.0 255.255.255.0 U 0 0 0 br-mgmt
10.109.0.0 0.0.0.0 255.255.255.0 U 0 0 0 br-fw-admin
192.168.122.0 0.0.0.0 255.255.255.0 U 0 0 0 virbr0
0.0.0.0 10.109.0.2 0.0.0.0 UG 0 0 0 br-fw-admin
0.0.0.0 10.109.0.222 0.0.0.0 UG 33 0 0 br-fw-adminпрогнали л23

Applied the manifest /etc/puppet/modules/osnailyfacter/modular/netconfig/netconfig.pp
nothing changed
[root@node-4 ~]# route -n
Kernel IP routing table
Destination Gateway Genmask Flags Metric Ref Use Iface
169.254.169.254 - 255.255.255.255 !H 0 - 0 -
192.168.1.0 0.0.0.0 255.255.255.0 U 0 0 0 br-storage
192.168.0.0 0.0.0.0 255.255.255.0 U 0 0 0 br-mgmt
10.109.0.0 0.0.0.0 255.255.255.0 U 0 0 0 br-fw-admin
192.168.122.0 0.0.0.0 255.255.255.0 U 0 0 0 virbr0
0.0.0.0 10.109.0.2 0.0.0.0 UG 0 0 0 br-fw-admin
0.0.0.0 10.109.0.222 0.0.0.0 UG 33 0 0 br-fw-admin

The problem is that
0.0.0.0 10.109.0.222 0.0.0.0 UG 33 0 0 br-fw-admin
is the same as in network scheme

l23 prefetch doesn't collect def routes with metric

[Sergey Vasilenko (xenolog)]

Should be re-tested. Because too much patches was been merged to the routing related resources.

[Sergey Vasilenko (xenolog)]

Even if this bug actual now, I doubt, this bug can be fixed without fixing #1452617.

i.e dependency is: bug/1452617 -> bug/1449527

[Bogdan Dobrelya (bogdando)]

cannot reproduce
# route -n
Kernel IP routing table
Destination Gateway Genmask Flags Metric Ref Use Iface
0.0.0.0 10.109.1.2 0.0.0.0 UG 33 0 0 br-ex

# puppet apply -vd /etc/puppet/modules/osnailyfacter/modular/netconfig/netconfig.pp | grep route
Debug: Puppet::Type::L3_clear_route::ProviderLnx: PREFETCHED properties for 'default,metric:33': {:ensure=>:present, :name=>"default,metric:33", :destination=>"default", :metric=>33, :gateway=>"10.109.1.2"}
Debug: Executing '/sbin/ip route del default dev br-ex'
Debug: Executing '/sbin/ip route add default via 10.109.1.1 dev br-ex'

# route -n
Kernel IP routing table
Destination Gateway Genmask Flags Metric Ref Use Iface
0.0.0.0 10.109.1.1 0.0.0.0 UG 0 0 0 br-ex