I think flow:1 is hard code a lot of places even inside OpenDaylight... --Colin
On Thu, Nov 10, 2016 at 11:42 AM, Daniel Malachovsky -X (dmalacho - PANTHEON TECHNOLOGIES at Cisco) <[email protected]> wrote: > For DLUX we’ll update topology app a bit and put there a select box. > > It will be populated with all available topologies and user can pick the > one he is interested in. > > > > I just started this with a simple question if “flow:1” is hardcoded > somewhere in openflow as a constant. Because in DLUX is J > > > > dano > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *Abhijit Kumbhare > *Sent:* 10. novembra 2016 17:25 > *To:* Robert Varga > *Cc:* openflowplugin-dev; Release ([email protected]) > *Subject:* Re: [release] [openflowplugin-dev] [WEATHER] - topology id in > openflowplugin > > > > We have decided in the OpenFlow Plugin meeting to have a config knob to > change the name from flow/1 to openflow/1 (for Dlux project) , but keep the > default as flow/1 to require no changes on the dependent projects. > > > > On Thu, Nov 10, 2016 at 8:04 AM, Robert Varga <[email protected]> wrote: > > Can we instantiate a proxy via topoprocessing to keep compatibility? > > Bye, > Robert > > On 11/10/2016 04:22 PM, Abhijit Kumbhare wrote: > > Yes - I agree with Colin. > > > > On Thu, Nov 10, 2016 at 7:16 AM, Colin Dixon <[email protected] > > <mailto:[email protected]>> wrote: > > > > I've commented on the relevant patch, but I'll also say it here. > > It's really not clear to me that this will reduce confusion. My > > guess is that changing the name we've used for 5 releases will cause > > at least as much confusion as renaming it fixes. > > > > --Colin > > > > > > On Tue, Nov 8, 2016 at 7:02 AM, Jozef Bacigál > > <[email protected] <mailto:[email protected]>> > > wrote: > > > > Hello, ____ > > > > __ __ > > > > the “*topology_id*” in openflow-plugin is defined as *“flow:1”* > > which can lead to confusion if you look at the topology tree. > ____ > > > > __ __ > > > > I’ve changed to topology id setting into yang configuration file > > and set the default value as *“openflow”* which is more > > describing. ____ > > > > __ __ > > > > If someone project using in *rest calls hard written with > > “/flow:1/”* as topology id *those links will be broken*.____ > > > > __ __ > > > > This change is mentioned to be merged *only into master > > (Carbon)* branch.____ > > > > __ __ > > > > Patch: https://git.opendaylight.org/gerrit/#/c/48108/ > > <https://git.opendaylight.org/gerrit/#/c/48108/>____ > > > > Weather: > > https://wiki.opendaylight.org/view/Weather#Change_of_ > topology_id_in_openflow-plugin > > <https://wiki.opendaylight.org/view/Weather#Change_of_ > topology_id_in_openflow-plugin>____ > > > > __ __ > > > > Jozef____ > > > > __ __ > > > > JozefBacigál____ > > > > Software Engineer > > > > ________ > > Sídlo / Mlynské Nivy 56 / 821 05 Bratislava / Slovakia > > R&D centrum / Janka Kráľa 9 / 974 01 Banská Bystrica / Slovakia > > +421 908 766 972 <tel:%2B421%20908%20766%20972 > <%2B421%20908%20766%20972>>/ > > [email protected] > > reception: +421 2 206 65 114 / www.pantheon.sk > > <http://www.pantheon.sk> > > > > logo > > > > > > > > > > _______________________________________________ > > openflowplugin-dev mailing list > > [email protected] > > <mailto:[email protected]> > > https://lists.opendaylight.org/mailman/listinfo/ > openflowplugin-dev > > <https://lists.opendaylight.org/mailman/listinfo/ > openflowplugin-dev> > > > > > > > > _______________________________________________ > > release mailing list > > [email protected] <mailto:release@lists. > opendaylight.org> > > https://lists.opendaylight.org/mailman/listinfo/release > > <https://lists.opendaylight.org/mailman/listinfo/release> > > > > > > > > > > > _______________________________________________ > > release mailing list > > [email protected] > > https://lists.opendaylight.org/mailman/listinfo/release > > > > > > _______________________________________________ > release mailing list > [email protected] > https://lists.opendaylight.org/mailman/listinfo/release > >
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
