]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Dynamically Pull Out Option Sections
authorKendall Nelson <kjnelson@us.ibm.com>
Fri, 4 Dec 2015 00:10:39 +0000 (18:10 -0600)
committerJay S Bryant <jsbryant@us.ibm.com>
Sat, 5 Dec 2015 11:23:13 +0000 (05:23 -0600)
commit63600a34fb3d771b7e57eff9bde33c8b1072b3c4
treec407d43a50349f95077e87b2540622a9e094c50f
parent03363ae6dd8adbf78229fc1b294c76f001adc9d8
Dynamically Pull Out Option Sections

Previously in the generation of the opts.py file, the option groups
were hard coded. Now, they are pulled out as the lines are parsed.
If a group doesn't already exist in the dictionary it gets added and
then the options are added to the section. If the group already exists,
the options are added to the proper section as you would expect, and if
the options being registered don't have a group, they are added to the
default section.

This patch also moves the check for a non-zero exit code from
oslo-config-generator to the right location in the file.  This
logic had been put too late in the file and was not catching
oslo-config-generator failures as it should have been.

Change-Id: I59c8067e8091d8bb212e3609f3a174ddc287707e
cinder/config/generate_cinder_opts.py
cinder/opts.py
tools/config/generate_sample.sh