diff options
| author | Stephen Worley <sworley@cumulusnetworks.com> | 2020-01-03 12:35:15 -0500 |
|---|---|---|
| committer | Stephen Worley <sworley@cumulusnetworks.com> | 2020-01-15 13:35:29 -0500 |
| commit | 1d049aba721bb05ecf2f8b5978d96863330093b6 (patch) | |
| tree | 376bee224d310c313f404d61fe5406492719ab22 /tools/generate_support_bundle.py | |
| parent | 47e0611d1698fe9a23c01a967ed57e987c726918 (diff) | |
zebra: just set nexthop member in handle_recursive_depend()
With recent changes to the lib nexthop_group
APIs (e1f3a8eb193267da195088cc515b598ae5a92a12), we are making
new assumptions that this should be adding a single nexthop
to a group, not a list of nexthops.
This broke the case of a recursive nexthop resolving to a group:
```
D> 2.2.2.1/32 [150/0] via 1.1.1.1 (recursive), 00:00:09
* via 1.1.1.1, dummy1 onlink, 00:00:09
via 1.1.1.2 (recursive), 00:00:09
* via 1.1.1.2, dummy2 onlink, 00:00:09
D> 3.3.3.1/32 [150/0] via 2.2.2.1 (recursive), 00:00:04
* via 1.1.1.1, dummy1 onlink, 00:00:04
K * 10.0.0.0/8 [0/1] via 172.27.227.148, tun0, 00:00:21
```
This group can instead just directly point to the nh that was passed.
Its only being used for a lookup (the memory gets copied and used
elsewhere if the nexthop is not found).
Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>
Diffstat (limited to 'tools/generate_support_bundle.py')
0 files changed, 0 insertions, 0 deletions
