Fix exporter code to handle delete-link followed by add-link.
We recently changed the code in IFMapLinkTable::AddLink() to fix 1426175 and
that fix is invalidating an assert in the exporter. In the case where we get a
delete-link followed by add-link, before the link is completely cleaned up, the
new code in IFMapLinkTable::AddLink() will find the older link and send a
change on it. This will result in the exporter asserting. Fix this by checking
if the link is being revived and calling appropriate asserts for the revival
case and the change case. Add test-case and fix test case that fails.
Reviewed: https:/ /review. opencontrail. org/10627 github. org/Juniper/ contrail- controller/ commit/ 81102c705d656c9 5bd7afa3edffb5e f3aef3f4cd
Committed: http://
Submitter: Zuul
Branch: R2.0
commit 81102c705d656c9 5bd7afa3edffb5e f3aef3f4cd
Author: Tapan Karwa <email address hidden>
Date: Tue May 19 11:05:34 2015 -0700
Fix exporter code to handle delete-link followed by add-link.
We recently changed the code in IFMapLinkTable: :AddLink( ) to fix 1426175 and :AddLink( ) will find the older link and send a
that fix is invalidating an assert in the exporter. In the case where we get a
delete-link followed by add-link, before the link is completely cleaned up, the
new code in IFMapLinkTable:
change on it. This will result in the exporter asserting. Fix this by checking
if the link is being revived and calling appropriate asserts for the revival
case and the change case. Add test-case and fix test case that fails.
Change-Id: Ib594d73847c80b 3e3d3f3650565b7 85d8aaf7f25
Closes-Bug: 1454380