Merge pull request #70 from cloudnativelabs/route-cleanup

murali-reddy · web-flow · commit e8ce4a96ce64 · 2017-07-17T09:26:13.000+05:30
on BGP peer down, advertised route from peer is not cleaned-up in local routing table
diff --git a/app/controllers/network_routes_controller.go b/app/controllers/network_routes_controller.go
@@ -223,6 +223,10 @@ func (nrc *NetworkRoutingController) injectRoute(path *table.Path) error {
 		Protocol: 0x11,
 	}
 
+	if path.IsWithdraw {
+		glog.Infof("Removing route: '%s via %s' from peer in the routing table", dst, nexthop)
+		return netlink.RouteDel(route)
+	}
 	glog.Infof("Inject route: '%s via %s' from peer to routing table", dst, nexthop)
 	return netlink.RouteReplace(route)
 }

Original file line number	Diff line number	Diff line change
`@@ -223,6 +223,10 @@ func (nrc NetworkRoutingController) injectRoute(path table.Path) error {`
`223`	`223`	`Protocol: 0x11,`
`224`	`224`	`}`
`225`	`225`
	`226`	`+ if path.IsWithdraw {`
	`227`	`+ glog.Infof("Removing route: '%s via %s' from peer in the routing table", dst, nexthop)`
	`228`	`+ return netlink.RouteDel(route)`
	`229`	`+ }`
`226`	`230`	`glog.Infof("Inject route: '%s via %s' from peer to routing table", dst, nexthop)`
`227`	`231`	`return netlink.RouteReplace(route)`
`228`	`232`	`}`