server: remove peer's getBests method

Use TableManager's getBestPathList

Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>

2 files changed, 2 insertions, 12 deletions

diff --git a/server/peer.go b/server/peer.go
index ea1d8098..55240cf6 100644
--- a/server/peer.go
+++ b/server/peer.go
@@ -124,7 +124,7 @@ func (peer *Peer) getAccepted(rfList []bgp.RouteFamily) []*table.Path {
 }
 
 func (peer *Peer) getBestFromLocal() ([]*table.Path, []*table.Path) {
-	pathList, filtered := peer.ApplyPolicy(table.POLICY_DIRECTION_EXPORT, filterpath(peer, peer.getBests(peer.localRib)))
+	pathList, filtered := peer.ApplyPolicy(table.POLICY_DIRECTION_EXPORT, filterpath(peer, peer.localRib.GetBestPathList(peer.localRib.GetRFlist())))
 	if peer.isRouteServerClient() == false {
 		for _, path := range pathList {
 			path.UpdatePathAttrs(&peer.gConf, &peer.conf)
@@ -249,16 +249,6 @@ func (peer *Peer) handleBGPmessage(m *bgp.BGPMessage) ([]*table.Path, bool, []*b
 	return pathList, update, bgpMsgList
 }
 
-func (peer *Peer) getBests(rib *table.TableManager) []*table.Path {
-	pathList := []*table.Path{}
-	for _, rf := range peer.configuredRFlist() {
-		for _, paths := range rib.GetBestPathList(rf) {
-			pathList = append(pathList, paths)
-		}
-	}
-	return pathList
-}
-
 func (peer *Peer) startFSMHandler(incoming chan *fsmMsg) {
 	peer.fsm.h = NewFSMHandler(peer.fsm, incoming, peer.outgoing)
 }
diff --git a/server/server.go b/server/server.go
index 417d09d7..5f6ec58a 100644
--- a/server/server.go
+++ b/server/server.go
@@ -1124,7 +1124,7 @@ func (server *BgpServer) handleModPathRequest(grpcReq *GrpcRequest) []*table.Pat
 				macIpAdv := evpnNlri.RouteTypeData.(*bgp.EVPNMacIPAdvertisementRoute)
 				etag := macIpAdv.ETag
 				mac := macIpAdv.MacAddress
-				paths := server.globalRib.GetBestPathList(bgp.RF_EVPN)
+				paths := server.globalRib.GetBestPathList([]bgp.RouteFamily{bgp.RF_EVPN})
 				if m := getMacMobilityExtendedCommunity(etag, mac, paths); m != nil {
 					extcomms = append(extcomms, m)
 				}