ovsdb usage of custom.properties


Edward Warnicke <hagbard@...>
 

Guys,
   I was reviewing:


(which is a totally fine patch) and it occurred to me to make the suggestion that you guys
either use the config subsystem to configure these things or ConfigAdmin so that you can
get your config parameters out of custom.properties and into a place you manage yourselves
in ovsdb.

Please note: I'm not demanding you do this... I'm just politely suggesting that you might 
be happier if you did :)

Ed


Anil Vishnoi
 

point noted, and we were planning to do it in this release :)

Anil

On Wed, Aug 5, 2015 at 11:21 PM, Edward Warnicke <hagbard@...> wrote:
Guys,
   I was reviewing:


(which is a totally fine patch) and it occurred to me to make the suggestion that you guys
either use the config subsystem to configure these things or ConfigAdmin so that you can
get your config parameters out of custom.properties and into a place you manage yourselves
in ovsdb.

Please note: I'm not demanding you do this... I'm just politely suggesting that you might 
be happier if you did :)

Ed

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev




--
Thanks
Anil


Sam Hague
 

Anil, Ed,

what does this look like to move it to ovsdb? Is it something like we move the properties into the config subsystem and the user manipulates the values from the xml files?

Thanks, Sam

On Wed, Aug 12, 2015 at 4:21 AM, Anil Vishnoi <vishnoianil@...> wrote:
point noted, and we were planning to do it in this release :)

Anil

On Wed, Aug 5, 2015 at 11:21 PM, Edward Warnicke <hagbard@...> wrote:
Guys,
   I was reviewing:


(which is a totally fine patch) and it occurred to me to make the suggestion that you guys
either use the config subsystem to configure these things or ConfigAdmin so that you can
get your config parameters out of custom.properties and into a place you manage yourselves
in ovsdb.

Please note: I'm not demanding you do this... I'm just politely suggesting that you might 
be happier if you did :)

Ed

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev




--
Thanks
Anil

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev



Anil Vishnoi
 

yes, Manipulate the values from xml files or through restconf. User can change these value at run time as well.

We need to make sure that we keep this user param config subsystem module separate from the dependency injection module, so that will create less issues incase user want to upgrade the installation.

Anil 

On Wed, Aug 12, 2015 at 5:49 PM, Sam Hague <shague@...> wrote:
Anil, Ed,

what does this look like to move it to ovsdb? Is it something like we move the properties into the config subsystem and the user manipulates the values from the xml files?

Thanks, Sam

On Wed, Aug 12, 2015 at 4:21 AM, Anil Vishnoi <vishnoianil@...> wrote:
point noted, and we were planning to do it in this release :)

Anil

On Wed, Aug 5, 2015 at 11:21 PM, Edward Warnicke <hagbard@...> wrote:
Guys,
   I was reviewing:


(which is a totally fine patch) and it occurred to me to make the suggestion that you guys
either use the config subsystem to configure these things or ConfigAdmin so that you can
get your config parameters out of custom.properties and into a place you manage yourselves
in ovsdb.

Please note: I'm not demanding you do this... I'm just politely suggesting that you might 
be happier if you did :)

Ed

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev




--
Thanks
Anil

_______________________________________________
ovsdb-dev mailing list
ovsdb-dev@...
https://lists.opendaylight.org/mailman/listinfo/ovsdb-dev





--
Thanks
Anil