[openstack-dev] [nova] config options help text improvement: current status

Markus Zoeller mzoeller at de.ibm.com
Thu Mar 3 12:15:20 UTC 2016


Markus Zoeller/Germany/IBM at IBMDE wrote on 03/03/2016 11:45:52 AM:

> From: Markus Zoeller/Germany/IBM at IBMDE
> To: "OpenStack Development Mailing List \(not for usage questions\)" 
> <openstack-dev at lists.openstack.org>
> Date: 03/03/2016 11:47 AM
> Subject: Re: [openstack-dev] [nova] config options help text 
> improvement: current status
> 
> Matt Riedemann <mriedem at linux.vnet.ibm.com> wrote on 03/03/2016 03:50:18 

> AM:
> 
> > From: Matt Riedemann <mriedem at linux.vnet.ibm.com>
> > To: openstack-dev at lists.openstack.org
> > Date: 03/03/2016 03:51 AM
> > Subject: Re: [openstack-dev] [nova] config options help text 
> > improvement: current status
> > 
> > 
> > 
> > On 3/2/2016 11:45 AM, Markus Zoeller wrote:
> > > TL;DR: From ~600 nova specific config options are:
> > >              ~140 at a central location with an improved help text
> > >              ~220 options in open reviews (currently on hold)
> > >              ~240 options todo
> > >
> > > [...]
> > >
> > >
> > > Regards, Markus Zoeller (markus_z)
> > >
> > >
> > 
> > Thanks, this looks a lot better.
> > 
> > On a side note, can we get someone to herd the cats around this bug?
> > 
> > https://bugs.launchpad.net/nova/+bug/1532228
> > 
> > There are a few duplicates and at least 3 patches up for the same 
thing, 
> 
> > but all three are incomplete. Basically all of the 
> > nova.virt.libvirt.volume.* options are missing from the generated 
sample 
> 
> > config. It would be nice if someone drove those 3 patches into one and 

> > resolved all of the missing libvirt volume options rather than a 
couple 
> > here and there.
> > 
> > -- 
> > 
> > Thanks,
> > 
> > Matt Riedemann
> 
> I found 3 open patches for that and combined it into [1]. My gerrit 
> query was: 
>     project:openstack/nova 
>     status:open 
>     file:opts.py 
>     NOT bp/centralize-config-options 
> Hope this helps.
> 
> References:
> [1] https://review.openstack.org/#/c/287685
> [2] https://youtu.be/Pk7yqlTMvp8
> 
> Regards, Markus Zoeller (markus_z)

Matt, I added a 4th one in the same area to the same patch. 
That patch replaces these 4 reviews:
https://review.openstack.org/#/c/266268/
https://review.openstack.org/#/c/265339/
https://review.openstack.org/#/c/276649/
https://review.openstack.org/#/c/265299/

Regards, Markus Zoeller (markus_z)




More information about the OpenStack-dev mailing list