Yes that's the reason we decided to keep the existing flow:1 as the default. Jozef's change already supports a config knob to be able to change it to openflow:1 (or any other string). This decision makes the weather report as informational - projects can change the string if they want to but are not forced to.
On Thu, Nov 10, 2016 at 9:44 AM, Colin Dixon <[email protected]> wrote: > 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_topolo >> gy_id_in_openflow-plugin >> > <https://wiki.opendaylight.org/view/Weather#Change_of_topol >> ogy_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/openflowplu >> gin-dev >> > <https://lists.opendaylight.org/mailman/listinfo/openflowpl >> ugin-dev> >> > >> > >> > >> > _______________________________________________ >> > release mailing list >> > [email protected] <mailto:[email protected] >> ight.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
