diff options
| author | IWASE Yusuke <iwase.yusuke0@gmail.com> | 2018-04-19 11:53:18 +0900 |
|---|---|---|
| committer | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2018-05-24 08:41:35 +0900 |
| commit | eb568b68d2522a21ad87c95e427024113aea07dd (patch) | |
| tree | adf4c68621456a6e160f82584fff63f7b5c48266 /test | |
| parent | a7ecbaac46608455338367ea91f55c307fb589ed (diff) | |
server: Withdraw routes when received RTM withdrawal
Because the RTM is removed from adj-RIB-in before collecting the
candidate routes to be withdrawn, the candidates are unexpectedly
filtered before sending withdraw messages. Then on the peers, the VPN
routes are left on.
This patch fixes to selects candidates from the filtered paths and send
withdraw messages as expected. Also resolves the TODO in
"test/scenario_test/rtc_test.py".
Signed-off-by: IWASE Yusuke <iwase.yusuke0@gmail.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/scenario_test/rtc_test.py | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/test/scenario_test/rtc_test.py b/test/scenario_test/rtc_test.py index 3dfdef3f..d7d7b69c 100644 --- a/test/scenario_test/rtc_test.py +++ b/test/scenario_test/rtc_test.py @@ -94,9 +94,7 @@ class GoBGPTestBase(unittest.TestCase): self.g1.local("gobgp vrf del vrf1") time.sleep(2) self.assertEqual(2, len(self.g1.get_adj_rib_out(self.g2, rf='rtc'))) - # TODO: - # g2 should withdraw VPN routes when received a RTM withdrawal. - # self.assertEqual(1, len(self.g1.get_adj_rib_in(self.g2, rf='ipv4-l3vpn'))) + self.assertEqual(1, len(self.g1.get_adj_rib_in(self.g2, rf='ipv4-l3vpn'))) def test_06_rr_setup(self): # +------+ |