I guess I did not capture the details in the minutes - here they are: Anil raised some concerns about the patch that this is dependent on work in Linux kernel and OVS - and that the upstream changes will incur changes to this patch in the future. His concern was that currently this patch does not keep backward compatibility for Srini's and Ed's patches earlier - but he thinks that may be OK as the earlier work is not used somewhere. On the other hand the concern he had was that the code may have to be changed in the future (depending on OVS & Linux kernel) - and in future it may be hard to keep backward compatibility to this current work that you have done. He was planning to send an email to the OpenFlow plugin mailing list about this for their concerns - and then merge.
On Mon, May 9, 2016 at 6:27 PM, Yang, Yi Y <[email protected]> wrote: > Hi, Abhijit > > > > So what is next step for https://git.opendaylight.org/gerrit/#/c/37937/ ? > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *Abhijit > Kumbhare > *Sent:* Tuesday, May 10, 2016 2:09 AM > *To:* [email protected] > *Subject:* [openflowplugin-dev] OpenFlow Plugin Meeting Minutes May 9, > 2016 > > > > --------------------------------------------------------- > > Browser Friendly format meeting minutes: > > --------------------------------------------------------- > > > http://meetings.opendaylight.org/opendaylight-openflowplugin/2016/openflow_plugin_may_9__2016/o > pendaylight-openflowplugin-openflow_plugin_may_9__2016.2016-05-09-16.04.html > > > > > ---------------------------------------------------------------------------------------------- > > Text format friendly for direct linking of lists.opendaylight.org > archives: > > > ---------------------------------------------------------------------------------------------- > > Meeting summary > > --------------- > > * Agenda (abhijitkumbhare, 16:05:14) > > * M3 items - bugs for the new design for projects (abhijitkumbhare, > > 16:05:44) > > * NSH patch from Yi Yang (abhijitkumbhare, 16:06:16) > > * BluePrint (abhijitkumbhare, 16:06:48) > > * TWS topic - merge of OpenFlow projects (abhijitkumbhare, 16:07:34) > > * Merges pending / code reviews (abhijitkumbhare, 16:08:05) > > > > * BluePrint (abhijitkumbhare, 16:08:47) > > * TomP has couple of draft patches for BluePrint (abhijitkumbhare, > > 16:10:49) > > > > * M3 items - bugs for the new design from the dependent projects > > (abhijitkumbhare, 16:22:24) > > * L2 switch blocker - goes to the switch but not the stats: > > https://bugs.opendaylight.org/show_bug.cgi?id=5822 > > (abhijitkumbhare, 16:26:40) > > * 5822 needed for migration (abhijitkumbhare, 16:28:27) > > * Regression of the functionality - target for M4 (abhijitkumbhare, > > 16:29:41) > > * 5659 - not related to Li Migration (abhijitkumbhare, 16:31:23) > > * Bug 5602: Blocker for Li migration for VTN project > > (abhijitkumbhare, 16:34:08) > > * LINK: https://bugs.opendaylight.org/show_bug.cgi?id=5692 (hideyuki, > > 16:39:23) > > * Blocker bugs for Li migration default change would be: 5692, 5602 & > > 4671 (abhijitkumbhare, 16:43:52) > > * ACTION: eshuvka_ to take a quick look at 5602; after that will > > likely need jbacigal to take over (abhijitkumbhare, 16:46:37) > > * ACTION: jbacigal looking into 4671 with michal_rehak > > (abhijitkumbhare, 16:47:07) > > * ACTION: hideyuki to take a look into 5692 if it can wait after the > > migration - since the estimate from Tomas Slusny for this M4 > > (abhijitkumbhare, 16:48:20) > > > > * NSH patch from Yi Yang (abhijitkumbhare, 16:49:22) > > * vishnoianil has some concerns - there was some previous work by > > Srini and Ed (abhijitkumbhare, 16:49:57) > > * Yi’s patch undoes some of the work by Srini & Ed (abhijitkumbhare, > > 16:50:26) > > * Yi’s patch modifies some of the existing extensions from Beryllium > > (abhijitkumbhare, 16:51:28) > > * LINK: https://git.opendaylight.org/gerrit/#/c/37937/ (vishnoianil, > > 16:56:04) > > Meeting ended at 17:00:25 UTC. > > > > Action items, by person > > ----------------------- > > * eshuvka_ > > * eshuvka_ to take a quick look at 5602; after that will likely need > > jbacigal to take over > > * hideyuki > > * hideyuki to take a look into 5692 if it can wait after the migration > > - since the estimate from Tomas Slusny for this M4 > > * jbacigal > > * eshuvka_ to take a quick look at 5602; after that will likely need > > jbacigal to take over > > * jbacigal looking into 4671 with michal_rehak > > >
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
