[NETVIRT-869] neutronvpn : refacror addToNeutronRouterInterfacesMap() Created: 24/Aug/17 Updated: 06/Apr/18 Resolved: 06/Apr/18 |
|
| Status: | Resolved |
| Project: | netvirt |
| Component/s: | General |
| Affects Version/s: | Nitrogen |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Medium |
| Reporter: | Philippe Guibert | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 9041 |
| Description |
|
The ticket is created, because of an improvement in the code of the function addToNeutronRouterInterfacesMap(). |
| Comments |
| Comment by Valentina Krasnobaeva [ 24/Aug/17 ] |
|
if optRouterInterfaces entry exists, we need to recuperate its list of ports (interfaces) and update this list with a new interface, then update optRouterInterfaces with a new list and than it is better to rewrite updated entry to CONFIGURATIONAL DS. If optRouterInterfaces entry does not exist, we will create it with new list of ports, and write it to CONFIGURATIONAL DS. |
| Comment by Valentina Krasnobaeva [ 24/Aug/17 ] |
|
Please, find a followin patch to fix this: |
| Comment by Valentina Krasnobaeva [ 24/Aug/17 ] |
|
Update/rewrite in LogicalDatastoreType.CONFIGURATION, only new created Interface object seems strange, I think we need at first to update interfaces list for current router Id and than update router-interfaces-map with updated list itself. And then we can rewrite it in CONFIGURATION DS. |