[openstack-dev] [oslo][all] Make CONF.set_override with paramter enforce_type=True by default may break unit test

2017-04-03 Thread ChangBo Guo
Hi ALL, oslo_config provides method CONF.set_override[1] , developers usually use it to change config option's value in tests. That's convenient . By default parameter enforce_type=False, it doesn't check any type or value of override. If set enforce_type=True , will check parameter override's

Re: [openstack-dev] [oslo][All] Make CONF.set_override with paramter enforce_type=True by default

2015-11-18 Thread Doug Hellmann
> > <openstack-dev@lists.openstack.org> > > Date: 11/17/2015 01:37 PM > > Subject: [openstack-dev] [oslo][All] Make CONF.set_override with > > paramter enforce_type=True by default > > > > Hi ALL, > > > 1. Problems : > >oslo_config prov

[openstack-dev] [oslo][All] Make CONF.set_override with paramter enforce_type=True by default

2015-11-17 Thread ChangBo Guo
Hi ALL, 1. Problems : oslo_config provides method CONF.set_override[1] , developers usually use it to change config option's value in tests. That's convenient . By default parameter enforce_type=False, it doesn't check any type or value of override. If set enforce_type=True , will check

Re: [openstack-dev] [oslo][All] Make CONF.set_override with paramter enforce_type=True by default

2015-11-17 Thread Davanum Srinivas
+1 to what Doug said. Thanks ChangBo! -- dims On Tue, Nov 17, 2015 at 10:47 AM, Doug Hellmann wrote: > Excerpts from ChangBo Guo's message of 2015-11-17 20:29:53 +0800: >> Hi ALL, >> >> 1. Problems : >>oslo_config provides method CONF.set_override[1] , developers

Re: [openstack-dev] [oslo][All] Make CONF.set_override with paramter enforce_type=True by default

2015-11-17 Thread Doug Hellmann
Excerpts from ChangBo Guo's message of 2015-11-17 20:29:53 +0800: > Hi ALL, > > 1. Problems : >oslo_config provides method CONF.set_override[1] , developers usually > use it to change config option's value in tests. That's convenient . >By default parameter enforce_type=False, it

Re: [openstack-dev] [oslo][All] Make CONF.set_override with paramter enforce_type=True by default

2015-11-17 Thread Markus Zoeller
ChangBo Guo <glongw...@gmail.com> wrote on 11/17/2015 01:29:53 PM: > From: ChangBo Guo <glongw...@gmail.com> > To: "OpenStack Development Mailing List (not for usage questions)" > <openstack-dev@lists.openstack.org> > Date: 11/17/2015 01:37 PM >