← Back to team overview

openstack-doc-core team mailing list archive

Re: Boolean fields (=True or not?)

 

I'll add it to the doc convention page. 
Take care,

Lorin
--
Lorin Hochstein
Lead Architect - Cloud Services
Nimbis Services, Inc.
www.nimbisservices.com





On Jun 1, 2012, at 1:01 PM, Anne Gentle wrote:

> Thanks for the clarification - the latest review copy of the install
> guide has force_dhcpelease=True in the nova.conf file. When I tested
> the nova.conf the old way, it didn't work anyway, so great catch!
> 
> Anne
> 
> On Fri, Jun 1, 2012 at 12:48 AM, Tom Fifield <fifieldt@xxxxxxxxxxxxxx> wrote:
>> Speaking as someone who's deployed OpenStack and had a lot of fun with the
>> negation, I agree with you on Format #1, especially as the 'default value'
>> can be made clearer.
>> 
>> Regards,
>> 
>> Tom
>> 
>> 
>> On 06/01/2012 02:18 AM, Lorin Hochstein wrote:
>>> 
>>> The config files (helpfully?) support two different formats for
>>> specifying boolean flags:
>>> 
>>> 
>>> Format 1:
>>> 
>>> force_dhcpelease=True
>>> force_dhcpelease=False
>>> 
>>> 
>>> Format 2:
>>> 
>>> force_dhcpelease
>>> noforce_dhcpelease
>>> 
>>> 
>>> I think we should standardize on one of these conventions for the docs
>>> (with a note in the docs about the alternate construction).
>>> 
>>> I vote for the =True/False because it's more explicit and consistent
>>> with the other flags.
>>> 
>>> Take care,
>>> 
>>> Lorin
>>> --
>>> Lorin Hochstein
>>> Lead Architect - Cloud Services
>>> Nimbis Services, Inc.
>>> www.nimbisservices.com <https://www.nimbisservices.com/>
>>> 
>>> 
>>> 
>>> 
>>> 
>>> 
>>> 
>> 
>> 
>> --
>> Mailing list: https://launchpad.net/~openstack-doc-core
>> Post to     : openstack-doc-core@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~openstack-doc-core
>> More help   : https://help.launchpad.net/ListHelp
> 
> -- 
> Mailing list: https://launchpad.net/~openstack-doc-core
> Post to     : openstack-doc-core@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openstack-doc-core
> More help   : https://help.launchpad.net/ListHelp

Attachment: smime.p7s
Description: S/MIME cryptographic signature


References