Why we have already weather process ? I thought if I made this everyone should stay alert and fix if an issue in his projects. Here in ODL is some unwritten rule if bigger change is coming in one project then we should/need/must to create patches for all touched projects ? Why the projects itself won’t fix any issue or react for weather or changes. And even then if you create an patch for some project the project won’t merge it because they don’t read mailing list.
Jozef From: Colin Dixon [mailto:[email protected]] Sent: Thursday, November 10, 2016 7:53 PM To: Robert Varga <[email protected]> Cc: openflowplugin-dev <[email protected]>; Release ([email protected]) <[email protected]>; Daniel Malachovsky -X (dmalacho - PANTHEON TECHNOLOGIES at Cisco) <[email protected]> Subject: Re: [openflowplugin-dev] [release] [WEATHER] - topology id in openflowplugin Something something. Reality vs. practice. Something something. More seriously, if we have a suggested best practice to avoid hard coding this in and then can search through all the projects and help them move away from it, I'm sure that would be appreciated by lots of people. --Colin On Thu, Nov 10, 2016 at 1:12 PM, Robert Varga <[email protected]<mailto:[email protected]>> wrote: On 11/10/2016 06:44 PM, Colin Dixon wrote: > I think flow:1 is hard code a lot of places even inside OpenDaylight... That is a *really* bad idea. This is equivalent of assuming a particular implementation of an interface. Whoever is using this hard-coded value should be getting a reference to a topology instead (and check whether the injected topology conforms to expectations by advertising a topology-type). Bye, Robert 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 / [email protected] reception: +421 2 206 65 114 / www.pantheon.sk [logo]
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
