+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
