[Openstack-docs] [openstack/nova] DocImpact review request

gerrit2 at review.openstack.org gerrit2 at review.openstack.org
Thu Aug 23 11:09:33 UTC 2012


Hi, I'd like you to take a look at this patch for potential
documentation impact.
https://review.openstack.org/11863

Log:
commit 63cc191c2deee369b4076847204d846cf577d78b
Author: Mark McLoughlin <markmc at redhat.com>
Date:   Thu Aug 23 10:35:17 2012 +0100

    Include CommonConfigOpts options in sample config
    
    Options defined by CommonConfigOpts are declared within the class rather
    than at module level, so they weren't being included in the sample conf.
    
    Also, in essex (and up until commit 991614add8), the sample conf file
    didn't contain these options but they included the hyphen in the name.
    The hyphen is only used on the command line, it is converted to an
    underscore automatically in the config file. Use opt.dest rather than
    opt.name as the config file key. Fixes bug #1034970.
    
    DocImpact: update nova.conf docs
    Change-Id: Ia7f3dded9148deedeb198c19a8d343db6dd93f99




More information about the Openstack-docs mailing list