Merge pull request #11025 from anlancs/bgpd-remove-l3nhg-control

bgpd: remove unnecessary l3nhg knob for evpn-mh

Merge pull request #11010 from opensourcerouting/feature/reuse_bgp_attr_set_community_for_flags

bgpd: Reuse bgp_attr_set_[el]community() for setting attribute flags

Merge pull request #11009 from opensourcerouting/fix/med_for_aggregate-address

bgpd: aggregate-address fixes

Merge pull request #11013 from opensourcerouting/fix/convert_bgp_peer_sort_t_to_enum

bgpd: typedef enum to enum

Merge pull request #10935 from anlancs/zebra-mh-esi-warning

zebra: adjust the warnings for ESI of evpn-mh

Merge pull request #10931 from g-balaji1/pimv6-ssm-cmds

pim6d: Addition of ipv6 ssmpingd config commands

Merge pull request #10972 from iqras23/default_originate

bgpd: Metric not set with default route

bgpd: remove unnecessary l3nhg knob for evpn-mh

Remove unnecessary `install_l3nhg` knob because it has already
been controlled by the command: "[no$no] use-es-l3nhg".

Signed-off-by: anlan_cs <vic.lan@pica8.com>

Merge pull request #10969 from donaldsharp/test_multicast_remove_support_bundle_collection

tests: Fix test_multicast_pim_sm_topo3.py from generating a support b…

bgpd: Metric not set with default route.

Description:
- When default route is originated using the
  neighbor default-originate command, MED is
  not set as part of the update message
  attribute.

- Changes are done to set the MED value and MED
  flag for default route.

Co-authored-by: Abhinay Ramesh <rabhinay@vmware.com>
Signed-off-by: Iqra Siddiqui <imujeebsiddi@vmware.com>

bgpd: Apply frrbot styling

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Convert bpacket_attr_vec_type to enum

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Convert bgp_attr_parse_ret_t to enum

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Convert vpn_policy_direction_t to enum

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Convert bgp_policy_type_e to enum

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Convert bgp_peer_sort_t to enum

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Fix styling, drop braces for single statement block

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Reuse bgp_attr_set_ecommunity() for setting attribute flags

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Reuse bgp_attr_set_[l]community() for setting attribute flags

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #10994 from opensourcerouting/fix/autocomplete_for_advertise_map

bgpd: Use autocomplete for route-map under conditional advertisements CLI

Merge pull request #10999 from opensourcerouting/fix/bgp_aliases_with_community

bgpd: Allow setting BGP [large]community in route-maps

Merge pull request #11000 from opensourcerouting/feature/show_conditional_adv_timer_in_neighbor_cli

bgpd: Show conditional advertisement timers in neighbor CLI output

Merge pull request #10989 from opensourcerouting/pim-options-remove

pimd: remove pim_interface->options

pim6d: Addition of ipv6 ssmpingd config commands

Signed-off-by: Balaji Gurudoss <G_Balaji1@dell.com>

Merge pull request #11003 from anlancs/bgpd-mh-trival-remove

bgpd: correct one flag name on comment for evpn-mh

Merge pull request #11006 from opensourcerouting/fix/remove_&routemap_optimization_cmd

lib: Remove deprecated routemap_optimization_cmd

bgpd: Fix styling for aggregate_addressv4_cmd

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Add autocomplete for `aggregate-address ... route-map X`

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Send MED attribute when aggregate prefix is created

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #11007 from mobash-rasool/fixes

pimd: correct the show ip igmp sources output

pimd: correct the show ip igmp sources output

frr(config-if)# ip igmp join 232.1.1.1 10.10.10.10
frr(config-if)# do sh ip igmp sources
Interface        Address         Group           Source          Timer Fwd Uptime
ens192           232.1.1.1       10.10.10.10     04:10   N 00:00:10
frr(config-if)#

The above output is misaligned and is having Address field which is not
required here.

Fixing it.

Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

lib: Remove deprecated routemap_optimization_cmd

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Do not turn off multicast stream

The test is testing whether interface flaps are causing
the appropriate pim reactions.  Unfortunately the test
is turning off the multicast stream and the test also
has a keep alive timer of 15 seconds set on all routers.
Which of course means the test has 15 seconds(at most) to finish
testing.  This is not always possible given system loads.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

tests: Fix test_multicast_pim_sm_topo3.py from generating a support bundle

The test_multicast_pim_sm_topo3.py test is both spending extra time
looking for state that will never occurr but also generating a support
bundle when it doesn't find it.  Fix the test to come to the correct
solution faster.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

bgpd: correct one flag name on comment for evpn-mh

Correct flag name of `attr.es_flags` - ATTR_ES_L3_NHG_USE.

"bgp_path_info"s (Per "es-vrf") with this flag can use l3nhg.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

bgpd: Show conditional advertisement timers in neighbor CLI output

```
spine1-debian-11# sh ip bgp neighbors 192.168.0.1
BGP neighbor is 192.168.0.1, remote AS 65001, local AS 65000, external link
Hostname: exit1-debian-11
  BGP version 4, remote router ID 192.168.10.123, local router ID 192.168.100.1
  BGP state = Established, up for 00:00:32
  Last read 00:00:30, Last write 00:00:30
  Hold time is 180, keepalive interval is 60 seconds
  Configured conditional advertisements interval is 5 seconds
  Time until conditional advertisements begin is 4 seconds
```

```
    "bgpTimerConfiguredConditionalAdvertisementsSec":5,
    "bgpTimerUntilConditionalAdvertisementsSec":1,
```

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #10919 from donaldsharp/lost_kernel_routes

zebra: Allow multiple connected routes to be choosen for kernel routes

bgpd: Allow setting BGP [large]community in route-maps

Before:
```
spine1-debian-11(config-route-map)# bgp community alias 65001:65001 test1
spine1-debian-11(config)# route-map rm permit 10
spine1-debian-11(config-route-map)# set community 65001:65001
% Malformed communities attribute
```

After:
```
spine1-debian-11(config)# bgp community alias 65001:65001 test1
spine1-debian-11(config)# route-map rm permit 10
spine1-debian-11(config-route-map)# set community 65001:65001
spine1-debian-11(config-route-map)#
```

Same for large-communities.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #10981 from opensourcerouting/vty-wrmem-newline

vtysh: remove extraneous newline

bgpd: Use autocomplete for route-map under conditional advertisements CLI

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Add a test to ensure that FRR does not loose kernel routes

Add a test case to ensure that Kernel routes are not lost
when there are multiple overlapping connected routes.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

zebra: Allow multiple connected routes to be choosen for kernel routes

This bug should only really affect kernel routes.  To reproduce:

a) Have multiple connected routes that point to the same prefix
swp8  up      default         169.254.0.250/30
swp9  up      default         169.254.0.250/30

b) Have a kernel route that uses one of those connected routes
7.6.2.8 via 169.254.0.249 dev swp8 proto static
(But have it choose a non-selected connected nexthop)

c) Introduce an event that causes the rib table to be reprocessed,
say a unrelated interface going up / down

  This causes the route to be lost with this message:
2022/03/28 21:21:53 ZEBRA: [YXCJP-0WZWV] netlink_nexthop_msg_encode: ID (3454): 169.254.0.249, via swp8(1383) vrf default(0)
2022/03/28 21:21:53 ZEBRA: [YF2E6-J60JH] nexthop_active: 169.254.0.249, via swp8 given ifindex does not match nexthops ifindex found found: directly connected, swp9

Effectively the nexthop that zebra is choosing would not be the one
that the kernel route has choosen and FRR removes the route:
022/03/28 21:21:53 ZEBRA: [NM15X-X83N9] rib_process: (0:254):7.6.2.8/32: rn 0x56042e632e90, removing re 0x56042e6316e0
2022/03/28 21:21:53 ZEBRA: [Y53JX-CBC5H] rib_unlink: (0:254):7.6.2.8/32: rn 0x56042e632e90, re 0x56042e6316e0
2022/03/28 21:21:53 ZEBRA: [KT8QQ-45WQ0] rib_gc_dest: (0:?):7.6.2.8/32: removing dest from table

What is happening?

Zebra is not looking at all connected routes and if any of them
would have the appropriate ifindex and just blindly rejecting
the route.

So when nexthop resolution happens and it matches a connected
route and the dest->selected nexthop ifindex does not match, let's sort
through the rest of them and see if any of them match and if so
let's keep the route.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Merge pull request #10982 from donaldsharp/enum_watchfrr

Enum watchfrr

pimd: remove pim_interface->options

I should've removed this in #10960.  It's a hazard in terms of
forgetting to adjust PRs/other changes that might accidentally still
reference the field.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #10979 from opensourcerouting/pim-ssm-clean

lib, pimd: small cleanup in multicast addr helpers (SSM & co.)

vtysh: remove extraneous newline

vtysh_client_execute() expects just a string without a newline; the
newline is passed through and ends up in logging output where newlines
are not quite wanted.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

watchfrr: Convert daemon_state_t to an enum instead of a typedef

Align watchfrr with our coding standard

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

watchfrr: Rename restart_phase_t to `enum restart_phase`

This will align with our coding standards.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

pimd: use SSM check helper

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib, pimd: add a bunch of address helpers

Just simple helpers to get a scope value, never-forward, and is-SSM for
a given address.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #10927 from bobuhiro11/refactor_srv6_locator_chunks

bgpd: refactor type of srv6_locator_chunks list

Merge pull request #10971 from opensourcerouting/fix/rpki_nits

bgpd: RPKI memory leak

bgpd: Delay returning RPKI instance as running

Start a separate timer which does the sync with the RPKI manager until
returns the synced status.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Return enum RPKI_NOT_BEING_USED instead of 0

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Reuse is_stopping() in RPKI code same as with is_running()

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Reuse is_running() function in RPKI code for rtr_is_running

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #10973 from mobash-rasool/fixes

pimd: Remove pim_channel_oil_change_iif from header file

pimd: Remove pim_channel_oil_change_iif from header file

pim_channel_oil_change_iif is sitting idly in .h file with no
definition.

Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

bgpd: refactor type of srv6_locator_chunks list

Since additional information such as block_bits_length is needed to
generate SIDs properly, the type of elements in srv6_locator_chunks
list is extended from "struct prefix_ipv6 *" to
"struct srv6_locator_chunk *". Even in terms of variable name,
"struct srv6_locator_chunk *" is appropriate.

Signed-off-by: Nobuhiro MIKI <nmiki@yahoo-corp.jp>

Merge pull request #10965 from opensourcerouting/fix/pim_igmp_group_delete

pimd: Replace ALL_LIST_ELEMENTS_RO to ALL_LIST_ELEMENTS group_list it…

Merge pull request #10966 from opensourcerouting/fix/memory_leak_rpki

bgpd: RPKI changes

Merge pull request #10365 from SaiGomathiN/ipv6-config-cmd

Merge pull request #10849 from anlancs/bgpd-cleanup-2

bgpd: remove unnecessary checkings for bgp_evpn_es_new()

Merge pull request #10968 from opensourcerouting/fix/gr_ordering

bgpd: Use proper debug msg for graceful restart commands

Merge pull request #10928 from anlancs/zebra-cleanup-1

zebra: use "assert" instead of unnecessary check

Merge pull request #10940 from opensourcerouting/pim-5549-clarify

pimd: clarify RFC5549 nexthop handling (& fix for pim6d)

Merge pull request #10959 from patrasar/pim_coverity

pimd: Fixing pim coverity issues

Merge pull request #10938 from anlancs/fix-zebra-vxlan-change-vrfid

zebra: fix missing vrf change of l2vni on vxlan interface

Merge pull request #10948 from anlancs/bgpd-cleanup-4

bgpd : use "assert" for one check on evpn-mh

bgpd: Use proper debug msg for graceful restart commands

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #10960 from opensourcerouting/pim-unmacro-1

pimd: remove useless PIM_IF_* macros

bgpd: No need to check for NULL before calling XFREE() in RPKI code

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Track RTRLib memory allocations separately

With fast `rpki reset`, seems that RTRLib related memory allocations leak
the memory. Need to figure this out later.

```
$ vtysh -c 'show memory | include RPKI'
BGP RPKI Cache server         :       12 variable       576       12       576
BGP RPKI Cache server group   :        0     48           0        1        56
BGP RPKI RTRLib               :   849657 variable  34453096   849662  41293504

$ for x in $(seq 1 60); do vtysh -c 'con' -c 'rpki' -c 'rpki reset'; sleep 1; done

$ vtysh -c 'show memory | include RPKI'
BGP RPKI Cache server         :       12 variable       576       12       576
BGP RPKI Cache server group   :        0     48           0        1        56
BGP RPKI RTRLib               :   849699 variable  69490616   849701  76323832
```

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

pimd: Replace ALL_LIST_ELEMENTS_RO to ALL_LIST_ELEMENTS group_list iteration

To reproduce:
```
ip pim ssm prefix-list pim-ssm-group
ip prefix-list pim-ssm-group seq 10 permit 238.4.0.0/16 ge 32 le 32
```

Assert:
```
pimd[645545]: pimd/pim_igmp.c:148: igmp_source_forward_reevaluate_all(): assertion ((srcnode)->data != NULL) failed
PIM[645545]: Received signal 6 at 1649140750 (si_addr 0x6e0009d9a9, PC 0x7fe5e3f95ce1); aborting...
PIM[645545]: /usr/local/lib/libfrr.so.0(zlog_backtrace_sigsafe+0x5e) [0x7fe5e41f2d7e]
PIM[645545]: /usr/local/lib/libfrr.so.0(zlog_signal+0xe6) [0x7fe5e41f2f56]
PIM[645545]: /usr/local/lib/libfrr.so.0(+0xc9412) [0x7fe5e421d412]
PIM[645545]: /lib/x86_64-linux-gnu/libpthread.so.0(+0x14140) [0x7fe5e4133140]
PIM[645545]: /lib/x86_64-linux-gnu/libc.so.6(gsignal+0x141) [0x7fe5e3f95ce1]
PIM[645545]: /lib/x86_64-linux-gnu/libc.so.6(abort+0x123) [0x7fe5e3f7f537]
PIM[645545]: /usr/local/lib/libfrr.so.0(_zlog_assert_failed+0xd7) [0x7fe5e4246c37]
PIM[645545]: /usr/lib/frr/pimd(igmp_source_forward_reevaluate_all+0x230) [0x5638c65e3d90]
PIM[645545]: /usr/lib/frr/pimd(pim_prefix_list_update+0x53) [0x5638c65cbbf3]
PIM[645545]: /usr/local/lib/libfrr.so.0(prefix_list_entry_update_start+0x82) [0x7fe5e420a0a2]
```

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

pim6d: Adding ipv6 mld watermark-warn CLI

Adding the config mode command

ipv6 mld watermark-warn <1-65535>
This command can be use to warn the user
when more than the desired limit of groups gets configured.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pimd: Moving the common function from pim_cmd.h file

Moving the PIM_DECLVAR_CONTEXT function to the
common file pim_cmd_common.h file

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "ipv6 mld last-member-query-interval" CLI

Adding the Interface level config command

ipv6 mld last-member-query-interval (1-65535)
This command can be use to tune the response time for group specific queries.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding ipv6 mld last-member-query-count CLI

Adding the Interface level config command

ipv6 mld last-member-query-count (1-255)
This command can be use to tune the number of Multicast-Address- Specific Queries
sent before the router assumes there are no remaining listeners for an address on a link.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "ipv6 mld query-max-response-time" CLI

Adding the Interface level config command

ipv6 mld query-max-response-time <1-65535>
This command can be use to tune the max response time for general queries.
The number of seconds represented by the [Query Response Interval] must be less than the [Query Interval]

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

Merge pull request #10953 from leonshaw/fix/bgp-rm-leak

bgpd: Fix memory leak in subgroup_announce_check

Merge pull request #10946 from donaldsharp/pim_fhr_lhr_same_pimreg

pimd: Send immediate join( with possible SG RPT prune bit set

pim6d: fix mis-printed nexthop

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

pimd, pim6d: clarify RFC5549 MRIB nexthop handling

The entire `case NEXTHOP_TYPE_IPV6_IFINDEX:` block here was a bit of a
tripwire to stumble over, since there was no indication at all that this
concerns RFC5549 nexthop handling.  So it got mis-adapted for PIM IPv6
support.

Clarify this a whole bunch that this is for v4-over-v6 nexthop mangling,
and nothing else.

This should really also use neighbor's secondary address lists for the
lookup, but that is probably going to break compatibility with other
boxes that don't include v6 addrs in v4 hellos and needs further
machinery to do properly, so for now just leave a breadcrumb.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #10710 from SaiGomathiN/pimv6-showcli

pimd: fix pim_instance NULL deference in zclient_lookup_read_pipe

Fixing the below coverity issue.
>>>     CID 1515546:  Null pointer dereferences  (NULL_RETURNS)
>>>     Dereferencing a pointer that might be "NULL" "pim" when calling "zclient_lookup_nexthop_once".
391             zclient_lookup_nexthop_once(pim, nexthop_tab, 10, l);

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: fix pim_instance NULL deference in pim_zebra_if_address_add

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: fix pim_instance NULL deference in pim_vxlan_mlag_update

Fixing the below coverity issue.
>>>     CID 1515545:    (NULL_RETURNS)
>>>     Dereferencing a pointer that might be "NULL" "pim" when calling "pim_vxlan_set_peerlink_rif".
902                     pim_vxlan_set_peerlink_rif(pim, NULL);

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: remove useless PIM_IF_* macros

The only function these macros have is to make the code confusing.
"PIM_IF_DO_PIM" sounds like it triggers some action, but it doesn't.

Replace with "bool" fields in struct pim_interface.

(Note: PIM_IF_*_IGMP_LISTEN_ALLROUTERS was always set, without any way
to unset it.  It is completely removed now and always enabled.)

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

pim6d: Adding show ipv6 pim nexthop and nexthop-lookup commands

Adding new show commands to display pim nexthops and
pim nexthop lookups received from zebra.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "show ipv6 [vrf|vrf ALL] pim neighbor" command

Adding new show CLI to display regarding pim neighbors.
Changing DEFUN to DEFPY for "show ip pim neighbor" command.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "show ipv6 pim local-membership"

Adding the new cli to display pim local membership information.
Changing DEFUN to DEPFY for "show ip pim local-membership"

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "show ipv6 pim jp-agg" command

Adding new show CLI to display pim join prune aggregate information.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "show ipv6 pim [vrf|vrf all] join" command

Adding show ipv6 pim join and show ipv6 pim vrf all join
CLIs to display pim join related information and
formatted the json output for "show ip pim join" and
"show ip pim vrf all join" commands.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "show ipv6 pim [vrf|vrf all] interface" command

Adding show ipv6 pim interface and show ipv6 pim vrf all interface
CLIs to display pim enabled interface informations and
formatted the json output for "show ip pim interface" and
"show ip pim vrf all interface" commands.

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pim6d: Adding "show ipv6 pim channel" command

Adding new CLI to display pim channel informations.
Changing DEFUN to DEPFY for ""show ip pim channel"

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

pimd: Moving the common functions from pim_cmd.c file

Moving the functions that are used by both IPV4 and IPV6 to a
common file pim_cmd_common.c file

Signed-off-by: Sai Gomathi N <nsaigomathi@vmware.com>

Merge pull request #10831 from anlancs/doc-mh-esi-type0

doc: add mh type-0 ESI support

Merge pull request #10954 from donaldsharp/speell

Speell

lib, yang: Fix spellinig of miliseconds

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

*: Fix spelling of Inteface

Signed-off-by: Donald Sharp <sharpd@nvidia.com>