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/10574 github. org/Juniper/ contrail- controller/ commit/ be520461dec4d4d 0dc999d651bae18 9d73fc9cde
Committed: http://
Submitter: Zuul
Branch: R2.1
commit be520461dec4d4d 0dc999d651bae18 9d73fc9cde
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