https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/OPNFLWPLUG-850 OpenFlowPlugin <p>A controller that holds mastership to a switch and lose connectivity delete the switch from the topology even though there are other controllers which will valid connectivity and another controller takes ownership.</p> <p>Reproducing this error is very straightforward:</p> <ul class="alternate" type="square"> <li>Start a 3 controllers nodes</li> <li>connect a switch to all controllers</li> <li>delete the controller that holds mastership from the switch</li> </ul> <p>Then, switch disappears from the topology and appears again when another controllers takes ownership.</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-850

[Boron] Openflow plugin deletes switch from topology while changing mastership from one controller to another

Bug Resolved Done Luis Gomez Jon Castro Thu, 9 Feb 2017 03:00:51 +0000 Mon, 27 Sep 2021 09:01:59 +0000 Mon, 18 Sep 2017 15:49:50 +0000 General 0 4 <p>I think this issue will be adressed here:</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/51071/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/51071/</a></p> <p>BR/Luis</p> <p>To be more precise, we need all these patches merged:<br/> <a href="https://git.opendaylight.org/gerrit/#/q/topic:%22Bug+6802%22" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/q/topic:%22Bug+6802%22</a></p> <p>stable/boron : <a href="https://git.opendaylight.org/gerrit/#/c/52230/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/52230/</a></p> <p>In Carbon release I got promise from mdsal guys to change the API on singleton service, so we don't need to combine EOS and singleton. (Plan is RC0)</p> <p>RC0 probably is going to be too late for us. We can use this fix till we get the singleton clustering api changes. We can take a call at that time.</p> <p>This is needed fixed in master.</p> <p>Luis, is this still an issue ?</p> <p>Which test should I look at ?</p> <p>There is no system test for this, I think it is better to ask Jon directly.</p> <p>Is this bug solved on Master branch? I could tested if the fix is merched to Boron branch.</p> <p>Looking at the bug history, it seems Boron is the fixed branch and there is some promise to fix for Carbon too.</p> <p>Jon, I will let you know when I cherry-pick patches from carbon to boron.</p> <p>I have done tests with latest Boron (SR3) but switches does not move from one controller to another without topology/flows changes.</p> <p>I detected that master branch contains some commits that may be related to this bug but does not seem to be merged in carbon and Boron</p> <p><a href="https://git.opendaylight.org/gerrit/#/q/topic:%22Split+connection+layer+...%22" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/q/topic:%22Split+connection+layer+...%22</a></p> <p>Anil, is your patch: <a href="https://git.opendaylight.org/gerrit/#/c/52230" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/52230</a> doing something for this issue or is it really the split connection patches in master that help here?</p> <p>@luis, Yes <a href="https://git.opendaylight.org/gerrit/#/c/52230/3" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/52230/3</a> This patch should fix this behavior. I believe jozef is doing similar fix to the stable/carbon branch now.</p> <p>@luis, Yes <a href="https://git.opendaylight.org/gerrit/#/c/52230/3" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/52230/3</a> This patch should fix this behavior. I believe jozef is doing similar fix to the stable/carbon branch now.</p> <p>(In reply to Jon Castro from comment #12)<br/> &gt; I have done tests with latest Boron (SR3) but switches does not move from<br/> &gt; one controller to another without topology/flows changes.<br/> &gt; <br/> &gt; I detected that master branch contains some commits that may be related to<br/> &gt; this bug but does not seem to be merged in carbon and Boron<br/> &gt; <br/> &gt; <a href="https://git.opendaylight.org/gerrit/#/q/topic:%22Split+connection+layer+" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/q/topic:%22Split+connection+layer+</a>...<br/> &gt; %22</p> <p>These have been merged in master and in stable/carbon (but not yet in stable/boron). Can you please test it in stable/carbon so we can know whether the Split Connection Handler fixes this issue?</p> <p>The question is for Jon again as we do not have system test for this.</p> <p>@Anil, <a href="https://git.opendaylight.org/gerrit/#/c/52230/3" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/52230/3</a> was merged back in March well before SR3 but Jon said above Boron SR3 does not work, so there is some confusion here whether the issue is addressed in your patch or in the split connection patches.</p> <p>(In reply to Luis Gomez from comment #18)<br/> &gt; @Anil, <a href="https://git.opendaylight.org/gerrit/#/c/52230/3" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/52230/3</a> was merged back in<br/> &gt; March well before SR3 but Jon said above Boron SR3 does not work, so there<br/> &gt; is some confusion here whether the issue is addressed in your patch or in<br/> &gt; the split connection patches.</p> <p>In stable/boron, above patch supposed to fix this issue. In stable/carbon, split connection patches contains the fix for this issue.</p> <p>I will test with stable/boron and confirm if this issue is resolved or not.</p> <p>I think this should be fixed now, please double check.</p> <p>This has been fixed for a while</p> Development External issue ID 7763 External issue URL ODL SR Target Milestone Rank 0|i033p3: