- This commit aims to solve reported problem on issues #1611, #1648 and #1912
- Partial changes of this commit duplicate with changes on PR #1587 (not merged) and PR #1766 (not merged and already closed)
- This commit is tested with only FRRouting version 6.0.2 (which uses Zebra API 6)
- This commit fixes lack of LABEL_MANAGER_CONNECT_ASYNC for ZAPI6.
(This bug is introduced on commit 2bdb76f2dc "Supporting Zebra API version 6 which is used in FRRouting version 6")
$ gobgp n 10.0.255.1 p
invalid policy type: choose from (in|import|export)
The in policy was removed long ago.
Signed-off-by: FUJITA Tomonori <[email protected]>
showNeighborRib() executes net.ParseCIDR() for vrf prefix like
10.100:100:10.0.0.0/24 and then crashes.
Probably ListRib() API should support an option to return sorted
destinations.
Signed-off-by: FUJITA Tomonori <[email protected]>
Let's simply use 'validation' because there is no other validation
structure.
We could add validation results from other than rpki so drop rpki name.
Signed-off-by: FUJITA Tomonori <[email protected]>
Also rename PathAttrsAny to PathAttrs. Both are the first choice
rather than the binary format members.
Support SortType member to ListPathRequest to add an option to return
unordered paths.
Signed-off-by: FUJITA Tomonori <[email protected]>
timeout context is for only connecting. We should refactor the code to
avoid global context variant.
Signed-off-by: FUJITA Tomonori <[email protected]>
Now the gRPC API doesn't require the binary wire-format for path
attributes. No need for the library for C.
Signed-off-by: FUJITA Tomonori <[email protected]>
The config is an internal package so it should not be exported.
NewAPIPolicyAssignmentFromTableStruct() is approprate for config/.
Signed-off-by: FUJITA Tomonori <[email protected]>
The table is an internal package so it should not be exported.
NewAPIPolicyAssignmentFromTableStruct() is approprate for table/.
Signed-off-by: FUJITA Tomonori <[email protected]>
NewPeerFromConfigStruct()
NewPeerGroupFromConfigStruct()
NewGlobalFromConfigStruct()
The config is an internal package so they should not be exported.
We will remove the usage of the config in server/. This is a part of
the goal.
Signed-off-by: FUJITA Tomonori <[email protected]>
This patch enables support for Color Extended Community in the CLI.
Usage Example:
$ gobgp global rib add -a ipv4 10.0.0.0/24 color 10
$ gobgp global rib -a ipv4
Network Next Hop AS_PATH Age Attrs
*> 10.0.0.0/24 0.0.0.0 00:00:00 [{Origin: ?} {Extcomms: [10]}]
Signed-off-by: Mikael Magnusson <[email protected]>
fixes #1812
Avoid the cast and use strightforward data strcuture; two variables
for afi and safi instead of squashing two into one variable.
Signed-off-by: FUJITA Tomonori <[email protected]>
- remove ReplaceDefinedSet and ReplaceStatement APIs; not intutive and
should create a new one instead of modifying the existing.
- Rename ReplacePolicyAssignment to SetPolicyAssignment API; we use
internally SetPolicy() name from the beginning.
- Rename UpdatePolicy() to SetPolicies() API; It doesn't update
anything so the name is confusing. It discards the all policies and
create policies from the argument.
- Changes some member names in structures for policy.
Signed-off-by: FUJITA Tomonori <[email protected]>
- clean up RPC function names
- rewrite gobgp command to use the api instead of config package
- delete unused client package
Signed-off-by: FUJITA Tomonori <[email protected]>
Sometimes fails due to listen port conflict with tests in
internal/pkg/client.
Using unix domain socket migth be a proper fix.
Signed-off-by: FUJITA Tomonori <[email protected]>
FUJITA Tomonori
JieJhih JhangandFUJITA Tomonori
Nasato GotoandFUJITA Tomonori
Robin DouineandFUJITA Tomonori
Hitoshi IrinoandFUJITA Tomonori
FUJITA TomonoriandFUJITA Tomonori
Thomas Rosenstein
Artur MakutunowiczandFUJITA Tomonori
Sergey Elantsev
Carl Baldwin
Eiichiro WatanabeandFUJITA Tomonori
Yoichi HariguchiandFUJITA Tomonori
wanghonghao
Mikael MagnussonandFUJITA Tomonori
[email protected]andFUJITA Tomonori