I propose, llike we discuss on the last meeting, to make an openflow config switch to add the possibility to switch on and off the flow removal notification with default state OFF, that we don't have any performance issue.
Jozef ________________________________________ From: [email protected] <[email protected]> on behalf of Luis Gomez <[email protected]> Sent: Wednesday, June 15, 2016 7:43 PM To: Manohar SL Cc: [email protected] Subject: Re: [openflowplugin-dev] Bug 5602 - Li Migration: Problems to detect the removal of flow entries +1, it is already bad OF protocol does not support Flow Added message, at least if it supports Flow Removed lets use it. > On Jun 15, 2016, at 5:06 AM, Manohar SL <[email protected]> wrote: > > Hi All, > > It would be good to retain the handling of " OFPT_FLOW_REMOVED" handling > similar to the He PlugIn. > This also accounts to OpenFlow Compliance. > > Basing the detection of the Flow Removal on Stats based solution will be very > costly, in the context of time consumed for the detection of the flow removal. > Completely agree with the below very valid points mentioned by Hideyuki Tai: >>>>>>>>>>>>>>>>> > This is about problems which applications face when it needs to detect the > removal of flow entries. > 1. It takes so long time (several seconds) to detect the removal. > 2. It is possible that applications fail to detect the removal. > <<<<<<<<<<<<<<<< > > Also, there should always be the flexibility to disable Stats at any given > time. > If the Flow Removal detection is based on Stats functionality, then we will > land up in losing this basic OpenFlow functionality. > > So, request to retain the Flow Removed handling similar to OF He PlugIn. > > Regs, > Manohar. > > ---------------------------------------------------------------------- > > Message: 1 > Date: Thu, 9 Jun 2016 23:14:28 +0000 > From: "Tai, Hideyuki" <[email protected]> > To: openflowplugin-dev <[email protected]> > Cc: "[email protected]" <[email protected]> > Subject: [openflowplugin-dev] Bug 5602 > Message-ID: > <c71616c1c81143b5bb3c234f53612...@dlncacitexmbx03.na-idm.na-gad.nec.com> > > Content-Type: text/plain; charset="us-ascii" > > Hi OpenFlow Plugin project, > > VTN project would like the OpenFlow Plugin project to provide a way to solve > the bug 5602 in the OFP-Li (the new plugin). > This is about problems which applications face when it needs to detect the > removal of flow entries. > 1. It takes so long time (several seconds) to detect the removal. > 2. It is possible that applications fail to detect the removal. > > You can find more detailed explanation in the Bugzilla. > https://bugs.opendaylight.org/show_bug.cgi?id=5602 > > Please note that the OFP-He (the old plugin) doesn't have this issue. > > First, I would like you to decide by which way we solve the problems in Boron. > > I think there are several ways. > > One way is to support FlowRemoved notification like the OFP-He does. > Actually, there are patches for that way. > https://git.opendaylight.org/gerrit/#/c/38639 > https://git.opendaylight.org/gerrit/#/c/39906/ > https://git.opendaylight.org/gerrit/#/c/39552/ > > Since the patch (gerrit 38639) is merged, I thought OFP project decided to > take this approach. > > Another way is to update the operational DS (flow-node-inventory:table) > immediately after the OFP-Li receives FLOW_REMOVED from openflowjava. > Then, applications can detect the removal of flow entries correctly using > listeners for the DS. > > Which way do you take for Boron? > Or other way? > > Regards, > Hideyuki Tai > > > > ------------------------------ > > Message: 2 > Date: Thu, 9 Jun 2016 16:35:00 -0700 > From: Jamo Luhrsen <[email protected]> > To: "[email protected]" > <[email protected]>, > "[email protected]" > <[email protected]>, > OpenDayLight-L2switch-Dev > <[email protected]> > Subject: [openflowplugin-dev] CSIT troubles. > openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron > Message-ID: <[email protected]> > Content-Type: text/plain; charset=utf-8 > > Earlier today I pointed out [0] that l2switch had something broken happening, > but now I think I notice it in openflowplugin CSIT [1] as well. This job is > getting aborted as it's running for 6 hours (or timeout for these jobs) > normally it's a 20m test. > > something serious here. > > the exception in the bug [2] may give a clue. > > > JamO > > [0] > https://lists.opendaylight.org/pipermail/integration-dev/2016-June/007073.html > [1] > https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron > [2] https://bugs.opendaylight.org/show_bug.cgi?id=6042 > > > ------------------------------ > > Message: 3 > Date: Thu, 9 Jun 2016 23:37:48 +0000 > From: "Venkatrangan G - ERS, HCL Tech" <[email protected]> > To: Jamo Luhrsen <[email protected]>, > "[email protected]" > <[email protected]>, > "[email protected]" > <[email protected]>, OpenDayLight-L2switch-Dev > <[email protected]> > Cc: "[email protected]" <[email protected]> > Subject: Re: [openflowplugin-dev] CSIT troubles. > openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron > Message-ID: > > <sg2pr04mb1246119085c591d691f76117a3...@sg2pr04mb1246.apcprd04.prod.outlook.com> > > Content-Type: text/plain; charset="us-ascii" > > Hi, > > We are facing this with the VTN jobs as well > Reference: > https://jenkins.opendaylight.org/releng/view/vtn/job/vtn-csit-1node-openstack-mitak > a-neutron-beryllium/lastSuccessfulBuild/artifact/odl1_karaf.log.tar.xz > >> From our understanding, the PACKET_IN handling is causing this. > > Regards, > Venkat G > > > -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf Of Jamo > Luhrsen > Sent: Thursday, June 9, 2016 4:35 PM > To: [email protected]; > [email protected]; OpenDayLight-L2switch-Dev > <[email protected]> > Subject: [openflowplugin-dev] CSIT troubles. > openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron > > Earlier today I pointed out [0] that l2switch had something broken happening, > but now I think I notice it in openflowplugin CSIT [1] as well. This job is > getting aborted as it's running for 6 hours (or timeout for these jobs) > normally it's a 20m test. > > something serious here. > > the exception in the bug [2] may give a clue. > > > JamO > > [0] > https://lists.opendaylight.org/pipermail/integration-dev/2016-June/007073.html > [1] > https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron > [2] https://bugs.opendaylight.org/show_bug.cgi?id=6042 > _______________________________________________ > openflowplugin-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev > > > ::DISCLAIMER:: > ---------------------------------------------------------------------------------------------------------------------------------------------------- > > The contents of this e-mail and any attachment(s) are confidential and > intended for the named recipient(s) only. > E-mail transmission is not guaranteed to be secure or error-free as > information could be intercepted, corrupted, lost, destroyed, arrive late or > incomplete, or may contain viruses in transmission. The e mail and its > contents (with or without referred errors) shall therefore not attach any > liability on the originator or HCL or its affiliates. > Views or opinions, if any, presented in this email are solely those of the > author and may not necessarily reflect the views or opinions of HCL or its > affiliates. Any form of reproduction, dissemination, copying, disclosure, > modification, distribution and / or publication of this message without the > prior written consent of authorized representative of HCL is strictly > prohibited. If you have received this email in error please delete it and > notify the sender immediately. > Before opening any email and/or attachments, please check them for viruses > and other defects. > > ---------------------------------------------------------------------------------------------------------------------------------------------------- > > > > ------------------------------ > > Message: 4 > Date: Thu, 9 Jun 2016 16:54:22 -0700 > From: Luis Gomez <[email protected]> > To: controller-dev <[email protected]> > Cc: "[email protected]" > <[email protected]>, > "[email protected]" > <[email protected]>, > OpenDayLight-L2switch-Dev > <[email protected]>, > "[email protected]" <[email protected]> > Subject: Re: [openflowplugin-dev] CSIT troubles. > openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron > Message-ID: <[email protected]> > Content-Type: text/plain; charset="us-ascii" > > Hi controller dev, > > It seems this patch (according to CI times) broke OF he plugin topology + OF > Li plugin topology + inventory with all the consequences in the downstream > projects: > > https://git.opendaylight.org/gerrit/#/c/38962/ > <https://git.opendaylight.org/gerrit/#/c/38962/> > > I see this patch is part of a larger controller merge list, so is the > regression expected as part of some major change (e.g. Whether) or is this > unexpected regression? > > BR/Luis > > >> On Jun 9, 2016, at 4:37 PM, Venkatrangan G - ERS, HCL Tech >> <[email protected]> wrote: >> >> Hi, >> >> We are facing this with the VTN jobs as well >> Reference: >> https://jenkins.opendaylight.org/releng/view/vtn/job/vtn-csit-1node-op >> enstack-mitak >> a-neutron-beryllium/lastSuccessfulBuild/artifact/odl1_karaf.log.tar.xz >> >> From our understanding, the PACKET_IN handling is causing this. >> >> Regards, >> Venkat G >> >> >> -----Original Message----- >> From: [email protected] >> [mailto:[email protected]] On Behalf >> Of Jamo Luhrsen >> Sent: Thursday, June 9, 2016 4:35 PM >> To: [email protected]; >> [email protected]; OpenDayLight-L2switch-Dev >> <[email protected]> >> Subject: [openflowplugin-dev] CSIT troubles. >> openflowplugin-csit-1node-flow-services-lithium-redesign-only-boron >> >> Earlier today I pointed out [0] that l2switch had something broken >> happening, but now I think I notice it in openflowplugin CSIT [1] as well. >> This job is getting aborted as it's running for 6 hours (or timeout for >> these jobs) normally it's a 20m test. >> >> something serious here. >> >> the exception in the bug [2] may give a clue. >> >> >> JamO >> >> [0] >> https://lists.opendaylight.org/pipermail/integration-dev/2016-June/007 >> 073.html [1] >> https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node- >> flow-services-lithium-redesign-only-boron >> [2] https://bugs.opendaylight.org/show_bug.cgi?id=6042 >> _______________________________________________ >> openflowplugin-dev mailing list >> [email protected] >> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev >> >> >> ::DISCLAIMER:: >> ---------------------------------------------------------------------- >> ---------------------------------------------------------------------- >> -------- >> >> The contents of this e-mail and any attachment(s) are confidential and >> intended for the named recipient(s) only. >> E-mail transmission is not guaranteed to be secure or error-free as >> information could be intercepted, corrupted, lost, destroyed, arrive >> late or incomplete, or may contain viruses in transmission. The e mail and >> its contents (with or without referred errors) shall therefore not attach >> any liability on the originator or HCL or its affiliates. >> Views or opinions, if any, presented in this email are solely those of >> the author and may not necessarily reflect the views or opinions of >> HCL or its affiliates. Any form of reproduction, dissemination, >> copying, disclosure, modification, distribution and / or publication of this >> message without the prior written consent of authorized representative of >> HCL is strictly prohibited. If you have received this email in error please >> delete it and notify the sender immediately. >> Before opening any email and/or attachments, please check them for viruses >> and other defects. >> >> ---------------------------------------------------------------------- >> ---------------------------------------------------------------------- >> -------- >> >> _______________________________________________ >> openflowplugin-dev mailing list >> [email protected] >> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev > > -------------- next part -------------- > An HTML attachment was scrubbed... > URL: > <http://lists.opendaylight.org/pipermail/openflowplugin-dev/attachments/20160609/738058e4/attachment.html> > > ------------------------------ > > _______________________________________________ > openflowplugin-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev > > > End of openflowplugin-dev Digest, Vol 36, Issue 24 > ************************************************** > _______________________________________________ > openflowplugin-dev mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev _______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev _______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
