-
Notifications
You must be signed in to change notification settings - Fork 251
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
binding: Fix race condition when claiming vif.
When a vif is claimed, chassis is written in sb Port Binding. If some vif update was received by the controller before the sb notification of the chassis update (i.e. while pb->chassis = NULL), the port was considered as TRACKED_REMOVED and ct_zone was flushed. When, later, the pb->chassis update was received, ct_zone was flushed again. This issue seems to only have as side effect a few extra ct_zone flush while the port is getting added. The fix avoid the extra bump in the claim process and hence avoids the unnecessary additional ct_flush. This issue was causing some flaky failures of test "Migration of CT zone from UUID to name" as unexpected ct_zone flus happened. Signed-off-by: Xavier Simonart <[email protected]> Acked-by: Ales Musil <[email protected]> Signed-off-by: Numan Siddique <[email protected]>
- Loading branch information
1 parent
f74b38f
commit ab1cfb9
Showing
2 changed files
with
74 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters