]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Downstream Fix for Genconfig
authorKendall Nelson <kjnelson@us.ibm.com>
Wed, 7 Oct 2015 13:53:38 +0000 (08:53 -0500)
committerKendall Nelson <kjnelson@us.ibm.com>
Fri, 13 Nov 2015 18:36:21 +0000 (12:36 -0600)
commit2f9e4163f42ae5246fd997b9f35e16d3d97be54f
treea960d46d172372108f57f8dd5d745548bf93053f
parent72984f6ec86e1892a10e69ba52f777be4dbca4bb
Downstream Fix for Genconfig

This patch adds the opts.py file to the tree as a fix for deployers
that package cinder. The opts.py file is no longer being deleted right
away by generate_sample.sh after the cinder.sample.conf is being
generated.

This patch also introduces a pep8 check to make sure that the opts.py
file is up to date, so that it will catch when new opts get added to
Cinder without the opts.py being updated.

To support the ability to keep and check the opts file a number
of changes were needed in the check_uptodate.sh script as well as
the generate_sample.sh script:
    - check_uptodate now takes --checkopts instead of --checkonly
      When checkopts is used the opts.py file is generated using the
      current code and the generated file is compared to the existing
      file.  The check fails if there are differences.
    - generate_sample now has the --nosamplefile option.
      When this option is used, only the opts.py file is generated.
      The oslo-config-generator code is skipped so no sample file
      is created.
    - generate_sample also has some coding style consistency changes.
    - Added the 'genopts' option to tox so users can generate
      a fresh opts.py without a sample file when necessary.

Closes-Bug: 1501820
Co-Author: Jay Bryant <jsbryant@us.ibm.com>

Change-Id: I1f5494ebb19d5f4e8c651cbeef0acad07ad96829
cinder/config/generate_cinder_opts.py
cinder/opts.py [new file with mode: 0644]
tools/config/check_uptodate.sh
tools/config/generate_sample.sh
tox.ini