Sergey Klyaus

@fujita I think I fixed the test. As I understood, the problem arises if BGP session is not yet established when we start waiting for hold timer expiry, so `g2.wait_for(expected_state=BGP_FSM_ACTIVE,...

@SkalaNetworks thanks for interest in this patch! We can use config-strict freely because we have testing environment where we can tolerate total outage due to invalid config before we deploy...

7e5eb10a8470826a14ec225cf6a7264785ea3603 contains only server-side changes. This PR contains CLI support for it.

Ugh, sorry for the confusion. The correct commit is bec61c4f38344febda7495a35afa02061bb2880b I guess - the one that adds a simple `EnableFiltered` to the server. This PR sets said flag in the...

``` goroutine 1483 [runnable]: github.com/osrg/gobgp/v4/pkg/server.drainChannel[...](...) /home/runner/work/gobgp/gobgp/pkg/server/util.go:27 github.com/osrg/gobgp/v4/pkg/server.cleanInfiniteChannel(...) /home/runner/work/gobgp/gobgp/pkg/server/util.go:38 github.com/osrg/gobgp/v4/pkg/server.(*fsmHandler).loop(0xc00066e4e0, {0x15b6610, 0xc0001bc280}, 0xc000702010) /home/runner/work/gobgp/gobgp/pkg/server/fsm.go:2091 +0x1085 created by github.com/osrg/gobgp/v4/pkg/server.newFSMHandler in goroutine 1610 /home/runner/work/gobgp/gobgp/pkg/server/fsm.go:392 +0x387 ``` This is kinda strange test failure since...

@fujita have you had the chance to look at it? I know PR is large, but conflicts are growing too I may try to split it into several, like make...

Seems logical to me, since one peer cannot produce more than one path per destination. In GRT, though, since it is combined view of all peers it's not true -...