[NETVIRT-412] unimgr - Auto tunnel - tunnels sometimes not created Created: 08/Jan/17 Updated: 13/Mar/17 Resolved: 13/Mar/17 |
|
| Status: | Resolved |
| Project: | netvirt |
| Component/s: | General |
| Affects Version/s: | Boron |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Amit Cohen | Assignee: | David Goldberg |
| 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: | 7490 |
| Description |
|
when we create elan service and we have interfaces elan on any CPE should be created Tunnels between CPEs , sometimes not created tunnels. this info from yakir. |
| Comments |
| Comment by Yakir Dorani [ 15/Jan/17 ] |
|
This bug is happening when creating a transport zone for different DPNs (dpn by dpn) very quickly (in this case Amit used a script). The end result should be a mesh of tunnels between all DPNs, but due to some race condition in genius some tunnels aren't created. I wrote an email describing this bug to someone in the community, it contains more information: "We are experiencing a bug when creating a transport zone with a few DPNs (one after the other), the bug is that a tunnel mesh should be created between all DPNs, but some tunnels are not created (different tunnels every time). It is related to the new “auto transport zone creation” feature: https://git.opendaylight.org/gerrit/#/c/49093/ I have found the bug to happen due to a use of cache in the ItmUtils.getTunnelMeshInfo method. The problem is that the cache is updated by another listener and if the transport zone is updated very quickly then the ItmTepAddWorker doesn’t find the other DPNs in the cache when building tunnels for the other DPNs. The bug doesn’t happen if I comment out the cache code." |
| Comment by Yakir Dorani [ 18/Jan/17 ] |
|
Apparently this patch fixes the problem: https://git.opendaylight.org/gerrit/#/c/50067/ I just cherry-picked it over the auto-tunnels patch and tested. |