Don't withdraw the intents during SDN-IP shutdown, because this creates
flux in the dataplane during switchover to another SDN-IP instance being the new SDN-IP leader. Change-Id: I93e271770001678c3438637d5a804d934cd83848
Showing
1 changed file
with
7 additions
and
0 deletions
| ... | @@ -105,6 +105,12 @@ public class IntentSynchronizer { | ... | @@ -105,6 +105,12 @@ public class IntentSynchronizer { |
| 105 | } | 105 | } |
| 106 | 106 | ||
| 107 | // | 107 | // |
| 108 | + // NOTE: We don't withdraw the intents during shutdown, because | ||
| 109 | + // it creates flux in the data plane during switchover. | ||
| 110 | + // | ||
| 111 | + | ||
| 112 | + /* | ||
| 113 | + // | ||
| 108 | // Build a batch operation to withdraw all intents from this | 114 | // Build a batch operation to withdraw all intents from this |
| 109 | // application. | 115 | // application. |
| 110 | // | 116 | // |
| ... | @@ -136,6 +142,7 @@ public class IntentSynchronizer { | ... | @@ -136,6 +142,7 @@ public class IntentSynchronizer { |
| 136 | peerIntents.clear(); | 142 | peerIntents.clear(); |
| 137 | routeIntents.clear(); | 143 | routeIntents.clear(); |
| 138 | log.debug("SDN-IP Intent Synchronizer shutdown completed"); | 144 | log.debug("SDN-IP Intent Synchronizer shutdown completed"); |
| 145 | + */ | ||
| 139 | } | 146 | } |
| 140 | } | 147 | } |
| 141 | 148 | ... | ... |
-
Please register or login to post a comment