Re: [openflowplugin-dev] [yangtools-dev] XML-YANG

2016-01-08 Thread Anil Vishnoi
Hi Walid, Comments inline.. On Fri, Jan 8, 2016 at 12:21 PM, Robert Varga wrote: > On 2016-01-08 11:41, Walid Aljoby wrote: > > I was wondering if you help to solve an issue related to pushing flows > using RestConf. > > Actually I followed examples Here: > https://wiki.opendaylight.org/view/Ed

Re: [openflowplugin-dev] [yangtools-dev] XML-YANG

2016-01-08 Thread Anil Vishnoi
imes with different way but could not find among the > interfaces or plugin. > > Many Thanks > --- > Walid > > ------ > *From:* Anil Vishnoi > *To:* Robert Varga > *Cc:* Walid Aljoby ; " > openflowplugin-dev@lists.opendaylight.org"

Re: [openflowplugin-dev] Lithium Plugin : RPC Registration delay

2016-01-08 Thread Anil Vishnoi
On Fri, Jan 8, 2016 at 11:25 AM, Muthukumaran K wrote: > Hi Kamal, Anil, Abhijit, > > > > We were testing the reconciliation with back-to-back switch reboots across > nodes – ie. switch disconnects from Node A and reconnects to Node B in > rapid successions. > > OVS does NOT use OF-HA capability

Re: [openflowplugin-dev] [yangtools-dev] XML-YANG

2016-01-09 Thread Anil Vishnoi
t; Thanks in advance > > --- > Walid > > > > -- > *From:* Anil Vishnoi > *To:* Walid Aljoby > *Cc:* "openflowplugin-dev@lists.opendaylight.org" < > openflowplugin-dev@lists.opendaylight.org> > *Sent:* Saturday, January 9, 2016 3:28 PM > *Subject:* Re: [yan

Re: [openflowplugin-dev] He vs Li Plugins tests

2016-01-19 Thread Anil Vishnoi
On Mon, Jan 18, 2016 at 10:50 PM, Luis Gomez wrote: > Hi all, > > I spent most of the day fixing the cluster test so we can measure both > plugins with the same test. I have also taken a look at the longevity > (stability) test we have this release. Here are the observations: > > 1) I started wit

Re: [openflowplugin-dev] [controller-dev] FlowCapableInventoryProvider - Thread Dies

2016-01-19 Thread Anil Vishnoi
Hi Guy, Can you please open a bug here for the issue https://bugs.opendaylight.org/enter_bug.cgi?product=openflowplugin Thanks Anil On Tue, Jan 19, 2016 at 9:18 AM, Sela, Guy wrote: > Yeah I know the reason behind the exception. > > I don’t see a reason for this exception to kill the entire >

Re: [openflowplugin-dev] He vs Li Plugins tests

2016-01-19 Thread Anil Vishnoi
y=ERROR, > errorType=APPLICATION, tag=resource-denied, applicationTag=null, info=null, > cause=org.opendaylight.yangtools.yang.data.api.schema.tree.ConflictingModificationAppliedException: > Node was deleted by other transaction.]]} > > > Thanks, > JamO > > > >> On Jan 19, 2016, at 12:31 AM, Anil Vishnoi <mailto:vishnoia..

Re: [openflowplugin-dev] Additional node removal/updated notifications on ovs switch role transition to Master in Openflowplugin

2016-01-27 Thread Anil Vishnoi
Nope that should not happen, i am working on the patch that will fix this issue. Planning to push this patch sometime tomorrow, will update you once patch is ready for the test. Thanks Anil On Wed, Jan 27, 2016 at 12:46 AM, wrote: > Hi, > > > > With the Openflowplugin HE design clustering, when

Re: [openflowplugin-dev] Arbitrary bit mask matching implementation

2016-01-28 Thread Anil Vishnoi
Hi John, You can push a patch to the openflowplugin repo and add OFP comitter to the review list. Assuming you using stable/beryllium branch, use following command to push the patch git push origin HEAD:refs/for/stable/beryllium or if it's master branch git push origin HEAD:refs/for/master Bu

Re: [openflowplugin-dev] Need solution when changing lithium to helium migration

2016-01-28 Thread Anil Vishnoi
On Thu, Jan 28, 2016 at 7:01 AM, M Vinoth wrote: > Hi Anil and openflowplugin-dev’s, > > > > Currently we facing 2 problem when we changing the lithium feature to > helium feature in our task to NIC. > > > > Problem 1: Logging the flow statistics data > > > > The following code is working in lith

Re: [openflowplugin-dev] Additional node removal/updated notifications on ovs switch role transition to Master in Openflowplugin

2016-01-31 Thread Anil Vishnoi
> notifications on failover in the Openflowplugin He clustering is available > in the beryllium branch > > > > *Regards,* > > *K.Kalaiselvi* > > > > *From:* Anil Vishnoi [mailto:vishnoia...@gmail.com] > *Sent:* Wednesday, January 27, 2016 2:

Re: [openflowplugin-dev] Additional node removal/updated notifications on ovs switch role transition to Master in Openflowplugin

2016-02-01 Thread Anil Vishnoi
HI Kalaiselvi, Can you please checkout following patch of openflowplugin and verify if it fixes your issue. https://git.opendaylight.org/gerrit/#/c/33843/1 Thanks Anil On Sun, Jan 31, 2016 at 11:23 PM, Anil Vishnoi wrote: > Hi Kalaiselvi, > > I opened a bug for this issue to

Re: [openflowplugin-dev] Request for a vote from the OFP committers on recommending Li design for Be SR 1

2016-02-02 Thread Anil Vishnoi
+1 given that we should discuss this at a community level on how to carry out this migration, to make sure that we don't break any backward compatibility between the SRs and also don't create any road block for upgrade between SRs. Anil On Mon, Feb 1, 2016 at 10:22 PM, Abhijit Kumbhare wrote: >

Re: [openflowplugin-dev] Additional node removal/updated notifications on ovs switch role transition to Master in Openflowplugin

2016-02-02 Thread Anil Vishnoi
the cluster. Terminating the openflow connection to one of the controllers > should not trigger > > Node removal and node updation notifications in the remaining controllers > in the cluster as the switch is still available with the cluster. > ​Yes, looking at it.​ > > > >

Re: [openflowplugin-dev] Additional node removal/updated notifications on ovs switch role transition to Master in Openflowplugin

2016-02-02 Thread Anil Vishnoi
Hi Kalaiselvi, Can you please test with the following patch : https://git.opendaylight.org/gerrit/#/c/33938/2 Thanks Anil On Tue, Feb 2, 2016 at 11:42 AM, Anil Vishnoi wrote: > > > On Tue, Feb 2, 2016 at 1:35 AM, wrote: > >> Hi Anil, >> >> >> >> Th

Re: [openflowplugin-dev] RPC calls of Helium plugin

2016-02-05 Thread Anil Vishnoi
Hi Shirley, response inline On Thu, Feb 4, 2016 at 9:54 AM, Shirley Anend wrote: > Hi, > > I have following queries in connection with RPC semantics > > 1. Do(es) future(s) of RPC call(s) for flows/groups/meters depend upon > statistics - in other words, if statistics are turned-off, would RPC

Re: [openflowplugin-dev] Beryllium Blocker in OPENFLOWPLUGIN

2016-02-10 Thread Anil Vishnoi
updated the bug. On Wed, Feb 10, 2016 at 10:52 AM, Abhijit Kumbhare wrote: > Anil is looking at it. > > On Wed, Feb 10, 2016 at 8:09 AM, An Ho wrote: > >> Hi OPENFLOWPLUGIN and VTN Team, >> >> >> >> There may be a potential blocker related to openflowplugin and vtn. >> Further investigated is n

Re: [openflowplugin-dev] openflowplugin cluster test aborted with RC

2016-02-16 Thread Anil Vishnoi
jamo looking at the above message, it looks like cluster didn't come up properly. 381 is the build you see these messages ? On Tue, Feb 16, 2016 at 10:32 AM, Jamo Luhrsen wrote: > The re-run did not abort: > > > https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-3node-clustering-on

Re: [openflowplugin-dev] openflowplugin cluster test aborted with RC

2016-02-16 Thread Anil Vishnoi
i am trying to get the karaf logs of all the 3 nodes for 381 build, but i can't find it. We just keep the logs for latest run ? On Tue, Feb 16, 2016 at 12:04 PM, Anil Vishnoi wrote: > jamo looking at the above message, it looks like cluster didn't come up > properly. > >

Re: [openflowplugin-dev] openflowplugin cluster test aborted with RC

2016-02-16 Thread Anil Vishnoi
what you can discern from the console output: > > https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-3node-clustering-only-beryllium/381/consoleFull > > when the jobs abort, we don't get the artifacts which are karaf.log for > csit jobs. > > JamO > > On 02/1

Re: [openflowplugin-dev] [ovsdb-dev] feature tree graphs

2016-02-16 Thread Anil Vishnoi
these are scary :) On Tue, Feb 16, 2016 at 2:09 PM, Jamo Luhrsen wrote: > ovsdb, ofp, > > these are always fun to look at: > > https://wiki.opendaylight.org/view/OpenDaylight_OpenFlow_Plugin/FeatureTrees/Beryllium > https://wiki.opendaylight.org/view/OVSDB_Integration/FeatureTrees/Beryllium > >

Re: [openflowplugin-dev] openflowplugin cluster test aborted with RC

2016-02-16 Thread Anil Vishnoi
so looks like some intermittent issue. On Tue, Feb 16, 2016 at 3:54 PM, Jamo Luhrsen wrote: > fyi, I retried 6 more times and this problem did not show up. > > > On 02/16/2016 01:52 PM, Anil Vishnoi wrote: > > so this exception says that data store is not available > &

Re: [openflowplugin-dev] OFPT_ROLE_REQ/RESPONSE even in a steady state

2016-02-17 Thread Anil Vishnoi
Hi Shuva, Are you using helium plugin or lithium plugin? In Helium plugin, we just send role request only on the time of connection, so if there is no connection flapping happening, controller should not send the role request. Can you recreate this scenario with 1 switch, but installing 1K flows

Re: [openflowplugin-dev] OFPT_ROLE_REQ/RESPONSE even in a steady state

2016-02-17 Thread Anil Vishnoi
hanks > > Shuva > > > > *From:* Kamal Rameshan (kramesha) [mailto:krame...@cisco.com] > *Sent:* Thursday, February 18, 2016 1:14 AM > *To:* Anil Vishnoi; Shuva Jyoti Kar > *Cc:* openflowplugin-dev@lists.opendaylight.org; Abhijit Kumbhare > > *Subject:* Re: OFPT_ROLE_RE

Re: [openflowplugin-dev] OFP-Li and OFP-He in Beryllium SRs

2016-02-19 Thread Anil Vishnoi
; impediments to move to the Lithium >> design in the SRs (target SR 1) and how the OpenFlow plugin project can >> address them? Example of the issues - your request regarding the stats >> RPCs. >> >> 3. Develop a timeframe for completing this migration based on the &

Re: [openflowplugin-dev] OF HA sequence of Events

2016-02-22 Thread Anil Vishnoi
In Helium plugin, this is how it goes 1) Openflowplugin receives the switch connection 2) plugin register that controller as a candidate for device ownership with EOS 3) Plugin fetch the current ownership state of the device, if it does not get the ownership it doesn't do anything, if it gets the

Re: [openflowplugin-dev] OF HA sequence of Events

2016-02-23 Thread Anil Vishnoi
on step followed by Oper DS Update step happen > after this ? > > > > *From:* openflowplugin-dev-boun...@lists.opendaylight.org [mailto: > openflowplugin-dev-boun...@lists.opendaylight.org] *On Behalf Of *Anil > Vishnoi > *Sent:* Tuesday, February 23, 2016 5:15 AM > *T

Re: [openflowplugin-dev] OF HA sequence of Events

2016-02-23 Thread Anil Vishnoi
? If it happens any earlier that > 6.1 and applications use rpc between 3.1 and 6.1 what is the behavior, as > there is a level of uncertainty as to the who owns it. > > > > Thanks > > Shuva > > > > *From:* Anil Vishnoi [mailto:vishnoia...@gmail.com] > *Sent:*

Re: [openflowplugin-dev] OF HA sequence of Events

2016-02-23 Thread Anil Vishnoi
d push flows through the rpc? > > > > Thanks > > Shuva > > > > *From:* Shuva Jyoti Kar > *Sent:* Tuesday, February 23, 2016 5:33 PM > *To:* 'Anil Vishnoi'; Muthukumaran K > *Cc:* openflowplugin-dev@lists.opendaylight.org > *Subject:* RE: [openflowplug

Re: [openflowplugin-dev] OF HA sequence of Events

2016-02-23 Thread Anil Vishnoi
PC gossip propagation and perhaps RPC-call > serialization aspects improve, local-RPC can remain as norm for FRM and > Recon code. > > > > *From:* Anil Vishnoi [mailto:vishnoia...@gmail.com] > *Sent:* Wednesday, February 24, 2016 12:36 AM > *To:* Shuva Jyoti Kar > *Cc:

Re: [openflowplugin-dev] OF HA sequence of Events

2016-02-23 Thread Anil Vishnoi
re stable under HA scenarios like switch reboots > > > > Regards > > Muthu > > > > > > *From:* Anil Vishnoi [mailto:vishnoia...@gmail.com] > *Sent:* Wednesday, February 24, 2016 11:27 AM > > *To:* Muthukumaran K > *Cc:* Shuva Jyoti Kar; openflowplug

Re: [openflowplugin-dev] [release] OF-Plugin dependent projects migration plan to Li design in the service releases

2016-02-25 Thread Anil Vishnoi
On Wed, Feb 24, 2016 at 4:11 PM, Colin Dixon wrote: > The issue with not not providing the same API (including notifications) in > the Beryllium-SR2 release is that there are projects, users and downstream > consumers that are not projects in OpenDaylight and thus we won't be able > to tell how t

Re: [openflowplugin-dev] [OpenDaylight TSC] OF-Plugin dependent projects migration plan to Li design in the service releases

2016-03-04 Thread Anil Vishnoi
I think one possible solution is, if user use barrier flag, disregard the timeout & queue size threshold and flush the queue. On Fri, Mar 4, 2016 at 2:45 PM, Tai, Hideyuki wrote: > Hi Robert, > > > > I'm sorry for the delayed response. > > I was not aware of your mail until now. > > > > Now, I t

Re: [openflowplugin-dev] OF Extensions sub project

2016-03-04 Thread Anil Vishnoi
Gal/Kody, Here are the pointers to start from https://wiki.opendaylight.org/view/OpenDaylight_OFextensions_Circuitsw https://git.opendaylight.org/gerrit/#/admin/projects/ofextensions/circuitsw Anil On Thu, Mar 3, 2016 at 7:43 AM, Mainzer, Gal wrote: > Thanks Abhijit. > > Adding Koby as well.

Re: [openflowplugin-dev] Patch to swap plugins in master

2016-03-07 Thread Anil Vishnoi
patch looks good to me, added minor comment on it. I think project should start using this patch for their migration. On Mon, Mar 7, 2016 at 5:03 PM, Luis Gomez wrote: > Hi all, > > After our talk this morning I created this patch to swap features in > master: > > https://git.opendaylight.org/ge

Re: [openflowplugin-dev] Patch to swap plugins in master

2016-03-07 Thread Anil Vishnoi
confusing as well.​ > > Sorry I missed the later part of the discussion. > > Thx > Kamal > > > From: on behalf of > Anil Vishnoi > Date: Monday, March 7, 2016 at 10:19 PM > To: Luis Gomez > Cc: openflowplugin-dev > Subject: Re: [openflowplugin-dev] P

Re: [openflowplugin-dev] Patch to swap plugins in master

2016-03-07 Thread Anil Vishnoi
yup, this is not final patch yet :) we might have to make many more changes in it. On Mon, Mar 7, 2016 at 10:33 PM, Kamal Rameshan (kramesha) < krame...@cisco.com> wrote: > IMO , that would be a correct way to transition. The default and –He. > > Thx > Kamal > > Fr

Re: [openflowplugin-dev] Exception/Error report from user

2016-03-08 Thread Anil Vishnoi
Another reason can be that, two instances of the controller is running on the same machine. On Mon, Mar 7, 2016 at 10:46 PM, Jamo Luhrsen wrote: > The BindException makes me think they have installed both versions of > the plugin. They'll both try to bind to port 6633 and the 2nd one will > hit

Re: [openflowplugin-dev] [controller-dev] question on connection setup

2016-03-19 Thread Anil Vishnoi
If you want to connect to openvswitch through ovsdb, yes, ovsdb southbound plugin allows that. If your question is about the bridges that you can create using openvswitch, and talk to those bridges through openflow protocol, then No, it's only switch initiated connection. On Thu, Mar 17, 2016 at 1

Re: [openflowplugin-dev] Vote for Jozef Bacigal as OpenFlow Plugin committer

2016-03-22 Thread Anil Vishnoi
+1 Sent from my iPhone > On Mar 22, 2016, at 5:41 PM, Abhijit Kumbhare wrote: > > Hi OpenFlow Plugin committers, > > Jozef has taken an increasingly bigger role in the OpenFlow Plugin project > via code commits as well as participation in the OpenFlow Plugin meetings. He > has been involved

Re: [openflowplugin-dev] [L2switch-dev] Flow ID in operational

2016-04-15 Thread Anil Vishnoi
Abhijit, for RPC,s flow id doesn't matter, because they just ignore it, because OF switch don't have any flow id construct. Flow id contention that luis mentioned above comes when you use a specific flow-id to dump stats in operational data store for rpc installed flows, but user uses the same flow

Re: [openflowplugin-dev] [L2switch-dev] Flow ID in operational

2016-04-15 Thread Anil Vishnoi
By same mechanism you mean, using the same format for the flow-id creation? On Fri, Apr 15, 2016 at 3:27 PM, Abhijit Kumbhare wrote: > In-line. > > On Fri, Apr 15, 2016 at 2:27 PM, Anil Vishnoi > wrote: > >> Abhijit, for RPC,s flow id doesn't matter, because they jus

Re: [openflowplugin-dev] Li plugin cluster broken

2016-05-03 Thread Anil Vishnoi
In my opinion this is not a blocker, given that this plugin is not a default plugin for Beryllium release. On Tue, May 3, 2016 at 2:45 PM, Abhijit Kumbhare wrote: > I would like to know Jozef's thoughts on this. > > On Tue, May 3, 2016 at 1:15 PM, An Ho wrote: > >> Have we been able to determin

Re: [openflowplugin-dev] Sending REST command with tcp port range to OpenDaylight Lithium controller

2016-05-06 Thread Anil Vishnoi
Hi Daniel, OpenFlow protocol (spec 1.3) don't have support for L4 port range/mask, so that's the reason ODL openflowplugin yang model don't have support for that. When you directly installing it on the OVS through CLI, it's working because it's using Nicira extension port mask feature to enable t

Re: [openflowplugin-dev] packet forwarding test using Opendaylight Lithium controller

2016-05-08 Thread Anil Vishnoi
On Sun, May 8, 2016 at 9:08 AM, 한솔 wrote: > Hi, all. > > I'm trying to show how to forwarding table is installed. > > I'm using 3 Lithium controllers and each controller(c1, c2, c3) have 1 > switch(s1, s2, s3) > > c1 c2 c3 > lll > s1-s2-s3 > > and ping s1 to s3 > ​Are these controller ru

Re: [openflowplugin-dev] Sending REST command with tcp port range to OpenDaylight Lithium controller

2016-05-08 Thread Anil Vishnoi
On Fri, May 6, 2016 at 3:20 PM, Jamo Luhrsen wrote: > > > On 05/06/2016 03:09 PM, Anil Vishnoi wrote: > > Hi Daniel, > > > > OpenFlow protocol (spec 1.3) don't have support for L4 port range/mask, > so that's the reason ODL openflowplugin yang model don

Re: [openflowplugin-dev] Sending REST command with tcp port range to OpenDaylight Lithium controller

2016-05-08 Thread Anil Vishnoi
gt; On 7 May 2016 at 00:20, Jamo Luhrsen wrote: > >> >> >> On 05/06/2016 03:09 PM, Anil Vishnoi wrote: >> > Hi Daniel, >> > >> > OpenFlow protocol (spec 1.3) don't have support for L4 port range/mask, >> so that's the reason ODL o

Re: [openflowplugin-dev] [opnfv-tech-discuss] [SFC] [sfc-dev] ovs nsh patches are available for public use

2016-05-08 Thread Anil Vishnoi
Hi Yi, I can see three patches related to NSH extensions https://git.opendaylight.org/gerrit/#/c/30481/ https://git.opendaylight.org/gerrit/#/c/37937/ https://git.opendaylight.org/gerrit/#/c/27643/ Can you please point me to correct patch, because 30481 and 37937 has similar code in the patch. I

Re: [openflowplugin-dev] [opnfv-tech-discuss] [SFC] [sfc-dev] ovs nsh patches are available for public use

2016-05-08 Thread Anil Vishnoi
ort, not any other thing is needed any > more.I have verified it in our sfc103 demo and my Eth+NSH demo, everything > is ok. > > > > > > *From:* Anil Vishnoi [mailto:vishnoia...@gmail.com] > *Sent:* Monday, May 09, 2016 11:50 AM > *To:* Yang, Yi Y > *Cc:* Dave Neary

Re: [openflowplugin-dev] [opnfv-tech-discuss] [SFC] [sfc-dev] ovs nsh patches are available for public use

2016-05-08 Thread Anil Vishnoi
://github.com/yyang13/ovs_nsh_patches> > ​​ <https://github.com/yyang13/ovs_nsh_patches> > https://github.com/yyang13/ovs_nsh_patches and pushed > https://git.opendaylight.org/gerrit/#/c/37937/. Can this answer your last > question? > > > > *From:* Anil Vishnoi [m

Re: [openflowplugin-dev] [opnfv-tech-discuss] [SFC] [sfc-dev] ovs nsh patches are available for public use

2016-05-08 Thread Anil Vishnoi
t maintained any more, so > I published one new version based on the latest ovs, it is just to make > sure we can have a new version available for ODL integration, all the > people are anticipating it. > > > > *From:* Anil Vishnoi [mailto:vishnoia...@gmail.com] > *Sent:*

Re: [openflowplugin-dev] Fwd: openflow plugin - statistics collecting time

2016-05-09 Thread Anil Vishnoi
you can install odl-openflowplugin-flow-services-rest and then configure the stats collection value in the mentioned xml file, that should solve your problem. Anil On Fri, May 6, 2016 at 8:08 AM, Shuva Jyoti Kar < shuva.jyoti@ericsson.com> wrote: > For the difference: > > > https://wiki.open

Re: [openflowplugin-dev] table features

2016-05-09 Thread Anil Vishnoi
was this patch merged to stable/beryllium as well? On Tue, Apr 26, 2016 at 8:21 AM, Abhijit Kumbhare wrote: > OK. > > On Tue, Apr 26, 2016 at 12:18 AM, Jozef Bacigal -X (jbacigal - PANTHEON > TECHNOLOGIES at Cisco) wrote: > >> Hi Abhijit, >> >> >> I thought I get an answer from NIC and DIDM guy

Re: [openflowplugin-dev] table features

2016-05-09 Thread Anil Vishnoi
default > setting on OFF table features. > > > https://git.opendaylight.org/gerrit/#/c/36506/3 > > > Jozef > ------ > *From:* Anil Vishnoi > *Sent:* Monday, May 9, 2016 10:37 AM > *To:* Abhijit Kumbhare > *Cc:* Jozef Bacigal -

Re: [openflowplugin-dev] [OpenDaylight Discuss] Important: Inventory model migration proposal (instead of inventory to topology model migration)

2016-05-09 Thread Anil Vishnoi
On Mon, May 9, 2016 at 2:42 PM, Robert Varga wrote: > On 05/02/2016 11:52 PM, Abhijit Kumbhare wrote: > > Hi folks, > > > > [snip] > > > This will require some change by the dependent projects (some > > modifications in the dependency declaration in the pom files) - however > > it will be less ch

Re: [openflowplugin-dev] table features

2016-05-10 Thread Anil Vishnoi
be better. On Mon, May 9, 2016 at 11:32 PM, Jozef Bacigal -X (jbacigal - PANTHEON TECHNOLOGIES at Cisco) wrote: > You propose to leave the flag (skip table features) false ? Just not to > change default behavior? > > > Jozef > ------ > *From:*

Re: [openflowplugin-dev] table features

2016-05-10 Thread Anil Vishnoi
e it has > been baked in (may be in the next SR). > > So it would be better to release note it I think. > ​Yes, so lets do it.​ > > Do you have a short 3-4 line description Jozef for release note > (explaining why it was changed)? > > On Tue, May 10, 2016 at 12:22 AM

Re: [openflowplugin-dev] [controller-users] SAL addflow()

2016-05-16 Thread Anil Vishnoi
here is the implementation https://github.com/opendaylight/openflowplugin/blob/master/openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/ModelDrivenSwitchImpl.java#L140 and this is the place where openflowplugin register the rpc provider https://github.com/opendayl

[openflowplugin-dev] [OFP Committers] NSH Extension patch

2016-05-16 Thread Anil Vishnoi
Hi Committers, Please review the following patch and the discussion happened in the comment section. Please let me know your thought and if you have any objection in merging this patch. I will wait for your response till the end of this week and if i won't get any response, i will assume that you

Re: [openflowplugin-dev] [OFP Committers] NSH Extension patch

2016-05-16 Thread Anil Vishnoi
here is the patch, thanks shuva for pointing it out https://git.opendaylight.org/gerrit/#/c/37937/1 On Mon, May 16, 2016 at 10:31 PM, Anil Vishnoi wrote: > Hi Committers, > > Please review the following patch and the discussion happened in the > comment section. Please let me know

Re: [openflowplugin-dev] [controller-users] SAL addflow()

2016-05-17 Thread Anil Vishnoi
you know how is SalFlowServiceImpl different from > ModelDrivenSwitchImpl.java?? > And is there a way to test if Sal-flow:add-flow really uses > ModelDrivenSwitchImpl.java ?? > > Thanks again. > Best, > BK > > On Tue, May 17, 2016 at 1:48 PM, Anil Vishnoi > wrote: > >

Re: [openflowplugin-dev] New meeting time - Thursdays 8 am Pacific?

2016-05-23 Thread Anil Vishnoi
Abhijit, please clearly mention for the record that i agreed to that slot "for a case of beer at the start of every month" :-P On Mon, May 23, 2016 at 3:38 PM, Abhijit Kumbhare wrote: > Hi folks, > > Next Monday is a holiday in the US and the following Monday I am > traveling. Also the Monday sl

Re: [openflowplugin-dev] https://git.opendaylight.org/gerrit/#/c/39097/ breaks netvirt

2016-05-23 Thread Anil Vishnoi
We faced this problem before and that's the reason we decided to log the warning, but do not throw the exception. Robert patch again start throwing the exception and that's the reason we see all these exception. I think Robert patch is correct and it requires fix in the openflowplugin project itsel

Re: [openflowplugin-dev] Beryllium SR1 issue with OVS 2.4 - bug 5464

2016-05-24 Thread Anil Vishnoi
tore and RPC.​ > > > So I guess we should discuss table-features with downstreamers as soon as > possible in order to have this fixed by SR2. > > > > > Regards, > > Michal > > > > -- > *From:* Abhijit Kumbhare > *

Re: [openflowplugin-dev] [nic-dev] [release] [didm-dev] OFP getTableFeature() breakage in nic and didm

2016-05-24 Thread Anil Vishnoi
So looks like these yang model changes happened and only lithium plugin was adapted to these changes. Helium plugin is still not adapted to these changes and that's why NIC/DIDM projects are probably not able to find the table features and failing. I am looking into it, and push the patch as soon a

Re: [openflowplugin-dev] [release] [didm-dev] [nic-dev] OFP getTableFeature() breakage in nic and didm

2016-05-25 Thread Anil Vishnoi
day, May 24, 2016, Anil Vishnoi wrote: > >> So looks like these yang model changes happened and only lithium plugin >> was adapted to these changes. Helium plugin is still not adapted to these >> changes and that's why NIC/DIDM projects are probably not able to find the >&

Re: [openflowplugin-dev] [nic-dev] [release] [didm-dev] OFP getTableFeature() breakage in nic and didm

2016-05-25 Thread Anil Vishnoi
note of SR3 should have this. > > On Wed, May 25, 2016 at 1:09 AM, Robert Varga wrote: > >> On 05/25/2016 07:43 AM, Anil Vishnoi wrote: >> > So looks like these yang model changes >> >> Hello, >> >> This will need to be release-noted, as it is a change in

Re: [openflowplugin-dev] [nic-dev] [release] [didm-dev] OFP getTableFeature() breakage in nic and didm

2016-05-25 Thread Anil Vishnoi
was pushed. We had discussed at the OF Plugin meeting - > and that was the least of the evils that was decided. > > On Wed, May 25, 2016 at 3:21 PM, Anil Vishnoi > wrote: > >> I don't think so this patch ( >> https://git.opendaylight.org/gerrit/#/c/36559/) was push

Re: [openflowplugin-dev] [nic-dev] [release] [didm-dev] OFP getTableFeature() breakage in nic and didm

2016-05-25 Thread Anil Vishnoi
still generates huge data even after > the model change on the master branch? > > https://git.opendaylight.org/gerrit/#/c/36559/ > > > > Regards, > > Hideyuki Tai > > > > *From:* nic-dev-boun...@lists.opendaylight.org [mailto: > nic-dev-boun...@lists.opendayli

Re: [openflowplugin-dev] [openflowjava-dev] OpenFlow broken in master

2016-06-02 Thread Anil Vishnoi
Hi Luis, Following patch should not be merge, It needs to be merge with few openflowplugin patches. Can you locally revert this patch and test it distribution builds fine. https://git.opendaylight.org/gerrit/#/c/38658/2 Thanks Anil On Thu, Jun 2, 2016 at 2:26 PM, Luis Gomez wrote: > The distr

Re: [openflowplugin-dev] Get Flows RPC: Design choice?

2016-06-06 Thread Anil Vishnoi
Hi Icaro, [1] actually defines the grouping, rpc and notifications, but [2] provides the high level tree structure where these grouping are augmented. There are two ways you can do Flow related CRUD operations (a) Through Data Store : In this case you use [2] to store the flow in config data stor

Re: [openflowplugin-dev] DIDM functionality breakage

2016-06-08 Thread Anil Vishnoi
Hi Subhash, Sorry for the delayed response. Did anybody in your team tried to debug the issue with the patch? OFP change is really very simple. Before this change, table features for individual table was stored under the table (node/node-id/table/0/(table-features-of-table-0). With this change, w

Re: [openflowplugin-dev] DIDM functionality breakage

2016-06-08 Thread Anil Vishnoi
e in ovs switch. > > Anandhi, > > Could you please let us know for the results. Let me know if any help is > required from my side. > > -- > Regards, > Subhash Kumar Singh > > On Wed, Jun 8, 2016 at 1:51 PM, Anil Vishnoi > wrote: > >> Hi Subhash, >> >&

Re: [openflowplugin-dev] DIDM functionality breakage

2016-06-08 Thread Anil Vishnoi
r the table features for ovs in data store > using rest. > > Please let me know your suggestions. > > -- > Regards, > Subhash Kumar Singh > > On Wed, Jun 8, 2016 at 2:14 PM, Anil Vishnoi > wrote: > >> Hi Subhash, >> >> I don't think you need HP switch t

Re: [openflowplugin-dev] DIDM functionality breakage

2016-06-08 Thread Anil Vishnoi
platform so that we can debug > the code, if it has any issue. > > -- > Thanks and Regards, > Subhash Kumar Singh > > On Wed, Jun 8, 2016 at 2:43 PM, Anil Vishnoi > wrote: > >> I believe the breakage is because of table feature movement, so with HP >> sw

Re: [openflowplugin-dev] BUG 6064

2016-06-15 Thread Anil Vishnoi
Shuva, please address the comments from Bertrand on your patch. Otherwise patch looks good. Thanks Anil On Tue, Jun 14, 2016 at 9:28 PM, Shuva Jyoti Kar < shuva.jyoti@ericsson.com> wrote: > Thanks a ton Luis. Sorry Miro and all for the unexpected failures. > > > > @ Luis can you do a +1 on t

Re: [openflowplugin-dev] Flow stats

2016-06-17 Thread Anil Vishnoi
Hi, It fetches every 3 second (configurable parameter). Thanks Anil On Fri, Jun 17, 2016 at 12:16 AM, Borasy Kong wrote: > Hi, > > Does ODL gather flow stats periodically (says every 5 seconds) from all > connected OpenFlow enabled switches or ODL will only request for those > stats when an RP

Re: [openflowplugin-dev] NetIDE patch comments

2016-06-23 Thread Anil Vishnoi
Yes Luis, this is what i was trying to explain. We should have generic interfaces for external application that allows them to subscribe for the OF messages from specific switch and a notification mechanism to deliver the OF messages. I think Alex also agrees to that but he need some pointers on ho

Re: [openflowplugin-dev] More automated testing, please! And make it gating.

2016-07-06 Thread Anil Vishnoi
I echo your thoughts. I was talking to Jamo & Luis about this during the hackfest and we are planning to use CSIT to gate the commits at least for OVSDB project to start with. We also discuss about when and how to trigger CSIT jobs on patches so that we can use the existing jenkins resources optima

Re: [openflowplugin-dev] Another CONFIG/OP flow mismatch in openflowplugin

2016-07-06 Thread Anil Vishnoi
josh, can you point me to the patch you pushed for this issue? On Mon, Jul 4, 2016 at 12:32 AM, Josh Hershberg wrote: > adding lists > > On Sun, Jul 3, 2016 at 2:59 PM, Josh Hershberg > wrote: > >> Here's what I'm seeing: >> >> 1) Create the flow. It gets written to the DeviceFlowRegistry >> 2)

Re: [openflowplugin-dev] OpenflowPlugin and Netvirt, and Netvrit IT problem

2016-07-06 Thread Anil Vishnoi
I will have a look at it sometime tomorrow. On Sun, Jul 3, 2016 at 8:53 AM, Josh Hershberg wrote: > Guys, > > I pushed a patch for this. > > https://git.opendaylight.org/gerrit/#/c/41247/ > > -J > > On Thu, Jun 30, 2016 at 1:41 PM, Josh Hershberg > wrote: > >> All, this is a separate email disc

Re: [openflowplugin-dev] OpenflowPlugin and Netvirt, and Netvrit IT problem

2016-07-08 Thread Anil Vishnoi
Hi Josh, Can you please address comment from Jozef on this patch, so we can merge it asap. On Wed, Jul 6, 2016 at 1:07 AM, Anil Vishnoi wrote: > I will have a look at it sometime tomorrow. > > On Sun, Jul 3, 2016 at 8:53 AM, Josh Hershberg > wrote: > >> Guys, >> &

Re: [openflowplugin-dev] OpenflowPlugin and Netvirt, and Netvrit IT problem

2016-07-08 Thread Anil Vishnoi
Josh, can you cherry-pick it to master as well. Once you cherry-pick i will merge the master patch as jozef did +1 on that. On Fri, Jul 8, 2016 at 12:48 AM, Josh Hershberg wrote: > done > > On Fri, Jul 8, 2016 at 9:31 AM, Anil Vishnoi > wrote: > >> Hi Josh, >> >&g

Re: [openflowplugin-dev] OpenflowPlugin and Netvirt, and Netvrit IT problem

2016-07-11 Thread Anil Vishnoi
Thanks Josh, Both the patches are now merged. On Sun, Jul 10, 2016 at 1:42 AM, Josh Hershberg wrote: > The cherry is picked. > > On Fri, Jul 8, 2016 at 10:49 PM, Anil Vishnoi > wrote: > >> Josh, can you cherry-pick it to master as well. Once you cherry-pick i >> wil

Re: [openflowplugin-dev] Bug 5539 - Remove deprecated model

2016-07-18 Thread Anil Vishnoi
Hi All, In my personal opinion, notification-flow-removed-off should be false by default. Because commonly most of the application look for the functional aspect and then performance aspect. Disabling flow removal event is like disabling the functionality and some of the application might not even

Re: [openflowplugin-dev] openflowplugin issue observed in CI

2016-07-18 Thread Anil Vishnoi
Hi Abhijit/Luis, Let's track these bugs in our weekly meetings. Thanks Anil On Mon, Jul 18, 2016 at 9:17 PM, Luis Gomez wrote: > The status of the plugin in CI this week: > > GENERAL (both Be and Li plugins): > > - Cluster: > > 1) Reconciliation not triggering after a single node reboot in a n

Re: [openflowplugin-dev] Requests for reviewing and merging patches for Be SR3

2016-07-21 Thread Anil Vishnoi
Hi Hideyuki, This patch looks good to me, but looks like CSIT test is failing on this patch. Did you run the CSIT-only-boron /CSIT-only-Beryllium test on these patches ? https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-flow-services-only-beryllium/ https:/

Re: [openflowplugin-dev] Requests for reviewing and merging patches for Be SR3

2016-07-23 Thread Anil Vishnoi
On Thu, Jul 21, 2016 at 9:32 PM, Luis Gomez wrote: > > On Jul 21, 2016, at 8:04 PM, Anil Vishnoi wrote: > > Hi Hideyuki, > > This patch looks good to me, but looks like CSIT test is failing on this > patch. Did you run the CSIT-only-boron /CSIT-only-Beryllium te

Re: [openflowplugin-dev] Requests for reviewing and merging patches for Be SR3

2016-07-23 Thread Anil Vishnoi
-dev-boun...@lists.opendaylight.org] *On Behalf Of *Tai, > Hideyuki > *Sent:* Friday, July 22, 2016 17:45 > *To:* Anil Vishnoi ; > openflowplugin-dev@lists.opendaylight.org > *Cc:* Shigeru Yasuda > > *Subject:* Re: [openflowplugin-dev] Requests for reviewing and merging > pa

Re: [openflowplugin-dev] Flow comparison

2016-07-24 Thread Anil Vishnoi
4:19 PM, Josh Hershberg >> wrote: >> >>> Which I think my patch is! The differences between the Match objects in >>> question are syntactic and not semantic. I added a compare method that >>> ignores the structural differences. >>> >>> On Tue

Re: [openflowplugin-dev] Flow comparison

2016-07-24 Thread Anil Vishnoi
Abhijit, can we please put this on agenda for next weeks meeting, we need to resolve this issue because it probably will surface many regressions with Li plugin. On Sun, Jul 24, 2016 at 12:54 AM, Anil Vishnoi wrote: > > > On Wed, Jul 20, 2016 at 8:28 PM, Josh Hershberg > wrot

Re: [openflowplugin-dev] Flow comparison

2016-07-24 Thread Anil Vishnoi
issue as well. On Sun, Jul 24, 2016 at 12:55 AM, Anil Vishnoi wrote: > Abhijit, can we please put this on agenda for next weeks meeting, we need > to resolve this issue because it probably will surface many regressions > with Li plugin. > > On Sun, Jul 24, 2016 at 12:54 AM, Anil V

Re: [openflowplugin-dev] Flow comparison

2016-07-25 Thread Anil Vishnoi
10 AM, Abhijit Kumbhare wrote: > Sounds good (discussing over email till the meeting on Thursday). > > On Sun, Jul 24, 2016 at 1:14 AM, Anil Vishnoi > wrote: > >> I just realised that next meeting is on Thursday (still i feel >> Openflowplugin meeting is on Monday),

Re: [openflowplugin-dev] Flow comparison

2016-07-25 Thread Anil Vishnoi
a little later. However, our current code does not set cookies and > if we do not really think that is the correct long term solution, are we > sure we want it now? > > On Tue, Jul 26, 2016 at 1:15 AM, Anil Vishnoi > wrote: > >> So i hooked up the He plugin custom

Re: [openflowplugin-dev] Vote to add Shuva Jyoti Kar as OpenFlow Plugin committer

2016-07-26 Thread Anil Vishnoi
+1 On Mon, Jul 25, 2016 at 9:44 PM, Abhijit Kumbhare wrote: > Hi folks, > > Shuva has been an contributor to both the OpenFlow Plugin code as well as > in activities like community education via answering emails on the mailing > list and the OpenFlow Plugin meeting. I think it will be good for t

Re: [openflowplugin-dev] Adding TCP Flag Action and Match Support In Openflowjava

2016-07-26 Thread Anil Vishnoi
Hi Michal, There are two version of tcp_flags that is present (1) Nicira Extension (2) OpenFlow Approved Extension 1st should be implemented as you mention above, but (2) should be part of the openflow java models (as an experimenter match). Following is the link of approved extension document h

Re: [openflowplugin-dev] “select” type for group tables?

2016-07-26 Thread Anil Vishnoi
As per my knowledge,yes, and i believe luis tested this as well. On Mon, Jul 25, 2016 at 11:28 PM, Jan Medved (jmedved) wrote: > Hi, > > does anybody know if one of the OpenFlow plugins (Li or Bo design) > supports the “select” type for group tables? > > > > Thanks, > Jan > > > _

Re: [openflowplugin-dev] Adding TCP Flag Action and Match Support In Openflowjava

2016-07-27 Thread Anil Vishnoi
, Jul 27, 2016 at 4:25 AM, Anil Vishnoi wrote: > Hi Michal, > > Here is the TCP_FLAGS extension document. I will send you the remaining > sometime tomorrow. > > Thanks > Anil > > On Wed, Jul 27, 2016 at 4:17 AM, Michal Polkoráb < > michal.polko...@pantheon.tech

Re: [openflowplugin-dev] webex for todays hand over 7/27/16

2016-07-27 Thread Anil Vishnoi
Yeah, i had no idea that we had handover today :(. Can someone please send the invite for the meeting, if it's not in my calendar before 1 hour of the meeting, there is 200% chances that i will miss the meeting ;). On Wed, Jul 27, 2016 at 8:24 AM, Abhijit Kumbhare wrote: > Basically - instead o

Re: [openflowplugin-dev] OpenFlow tests for next ODL perf paper

2016-07-27 Thread Anil Vishnoi
On Wed, Jul 27, 2016 at 3:13 PM, Luis Gomez wrote: > Hi all, > > I got the action point from last S3P call to start some discussion around > OpenFlow tests we can do for next ODL perf paper (Boron). > > I am not sure we will have time for all the below but ideally I was > thinking in 4 tests: > >

  1   2   3   4   >