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/NETVIRT-1062 netvirt <p>Too Many exceptions observed while deleting interface from Router</p> NETVIRT-1062

Too Many exceptions observed while deleting interface from Router

Bug Highest Verified Done Philippe Guibert RajaRajan Manickam csit:exception Thu, 11 Jan 2018 11:37:52 +0000 Tue, 19 Jun 2018 17:50:50 +0000 Wed, 28 Mar 2018 19:50:38 +0000 Oxygen Oxygen-SR1 General 0 6 <p>At first, refactored vpn-manager/SubnetmapChangeListener.java,</p> <p>Take in account VPN type (BGPVPN External/InternetVPN) in onSubnetAddedToVpn() and onSubnetDeletedFromVpn() methods from VpnSubnetRouteHandler.java.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/68558/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/68558/</a></p> <p>Is 68558 the fix for this issue?  Please cherry pick to the oxygen branch and +2 when ready.</p> <p>Patch for Oxygen is already here:</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/68808/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/68808/</a></p> <p> </p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=valentina.krasnobaeva" class="user-hover" rel="valentina.krasnobaeva">valentina.krasnobaeva</a> - Looks like 68808 needs a rebase? Friendly reminder that we need to close this ASAP to unblock the imminent Oxygen release.</p> <p>Hi,</p> <p>68808 was rebased, will wait recheck job results, to prove that it compiles well</p> <p> </p> <p> </p> <p>Looks like rebase is done, the standard verify job is passing and some extra verify jobs are currently running.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=valentina.krasnobaeva" class="user-hover" rel="valentina.krasnobaeva">valentina.krasnobaeva</a> - Looks like the job finished. How do the results look? Issue fixed? Should we start bugging <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> et al for a +2?</p> <p>Yes, thank you.  Results look as expected. vpn_basic.robot and vpn_basic_ipv6.robot are passed successfully, so we can start bugging <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> et al for a +2 for this patch.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> - To verify 68808 fixes the issue and +2/merge.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> - CSIT results:</p> <p><a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-oxygen/87/robot-plugin/log_full.html.gz" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-oxygen/87/robot-plugin/log_full.html.gz</a></p> <p><a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-tdd-wip-oxygen/4/robot-plugin/log_full.html.gz" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-tdd-wip-oxygen/4/robot-plugin/log_full.html.gz</a></p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> - Your test-netvirt-current verify job finished but it doesn't seem like the logs were copied?</p> <p><a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-patch-test-current-oxygen/977" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-patch-test-current-oxygen/977</a></p> <p>I guess they are still in the console:</p> <p><a href="https://jenkins.opendaylight.org/releng/job/netvirt-patch-test-current-oxygen/977/console" class="external-link" target="_blank" rel="nofollow noopener">https://jenkins.opendaylight.org/releng/job/netvirt-patch-test-current-oxygen/977/console</a></p> <p>I kicked off another one, just to do something that might help.</p> <blockquote><p>Sam Hague - Your test-netvirt-current verify job finished but it doesn't seem like the logs were copied?</p> <p><a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-patch-test-current-oxygen/977" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-patch-test-current-oxygen/977</a></p></blockquote> <p>this is the <a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-oxygen/104/" class="external-link" target="_blank" rel="nofollow noopener">gate job</a> we want to look at. you have<br/> to click on the child job of the patch-test job.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> can verify, but the <a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/netvirt-csit-1node-openstack-queens-gate-stateful-oxygen/104/odl_1/odl1_exception.log.gz" class="external-link" target="_blank" rel="nofollow noopener">exception log</a> of that job looks totally clean.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> said on a mailing list thread that <a href="https://git.opendaylight.org/gerrit/#/c/69379/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/69379/</a> is related to this (and is merged). Just tracking it here too.</p> <p>Bumping that <a href="https://git.opendaylight.org/gerrit/#/c/68808/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/68808/</a> is the currently outstanding (but seemingly ready to merge) patch.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a> - What's going on with 68808? It seems to be ready to merge.</p> <p>68808 has a -1 - it was removed on the last rebase, but there are still comments to address there. working on getting those comments addressed.</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/68203/16" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/68203</a> is the patch that 68808 is parented on. That one is about ready. Running another csit job on it, which should be good and then we can merge 68203.</p> <p>Oh okay, yeah I see the ongoing feedback on the master version. I was only looking at the stable/oxygen cherry-pick, sorry. Thanks for clarifying/pushing <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a>!</p> <p>The parent of the change that fixes this bug seems ready to merge: <a href="https://git.opendaylight.org/gerrit/#/c/68807/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/68807/</a></p> <p>We're still waiting on: <a href="https://git.opendaylight.org/gerrit/#/c/68808/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/68808/</a></p> <p>We need to get it merged immediately if we want to have any chance of making the (already delayed, really last possible date) Oxygen release date. We then need to get lucky and get an autorelease with no "sporadic" failures almost immediately, and then get quick sign-offs on the remaining failures from all projects.</p> <p>We're at the point that imho, this fix has taken too long. If we don't get a fix in immediately I'm going to start recommending we release without this.</p> <p>Per discussion on <a href="https://lists.opendaylight.org/pipermail/release/2018-March/014328.html" class="external-link" target="_blank" rel="nofollow noopener">release list</a>, as <a href="https://git.opendaylight.org/gerrit/#/c/68808/" class="external-link" target="_blank" rel="nofollow noopener">68808</a> has not been merged and <a href="https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/autorelease-release-oxygen/234/" class="external-link" target="_blank" rel="nofollow noopener">autorelease 234</a> passed, we <a href="https://lists.opendaylight.org/pipermail/release/2018-March/014331.html" class="external-link" target="_blank" rel="nofollow noopener">have declared</a> Oxygen RC3 from autorelease 234. We can target 68808 for the Oxygen SR1 release, or from the <a href="https://nexus.opendaylight.org/content/repositories/opendaylight-oxygen-epel-7-x86_64-devel/org/opendaylight/integration-packaging/opendaylight/" class="external-link" target="_blank" rel="nofollow noopener">oxygen CD repos</a> as soon as it's merged into stable/oxygen.</p> <p>I'm not sure how we should handle the Jira flags here <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=shague" class="user-hover" rel="shague">shague</a>. Set the Fix Version to Oxygen-SR1 or change the Severity flag to Critical?</p> <p>Working with latest build.</p> Relates NETVIRT-1134 Development Rank 0|i039f3: