]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Hacking check for opt name registration
authorKendall Nelson <kjnelson@us.ibm.com>
Tue, 15 Sep 2015 01:37:00 +0000 (20:37 -0500)
committerKendall Nelson <kjnelson@us.ibm.com>
Tue, 6 Oct 2015 13:35:48 +0000 (08:35 -0500)
commit391d18dc640de24d65662444c1c022244b9a6ed5
tree5292cca6ff1503416793856be82d0699b52f0e26
parent10f5e92e1246dd33b244c4a7031bb81b17b240a3
Hacking check for opt name registration

Depending on how opts are registered (either with register_opt() or
register_opts()), the name needs to be singular or plural to match
the method.  This patch adds a hacking check to make sure the names
of the opts and opt lists (or tuples) are correct given how they are
being registered.  The check also verifies that a single option is
sent when register_opt() is used and a list is used when using
register_opts().

Includes fixes to files that don't meet the naming convention and a
addition to the generate_cinder_opts.py file in order to skip
checks.py in the generation of the opts.py file.

Closes-Bug: 1495752
Change-Id: Ia795915c6e3d46272acc30407961d5d876f783ce
HACKING.rst
cinder/config/generate_cinder_opts.py
cinder/hacking/checks.py
cinder/image/glance.py
cinder/tests/unit/test_hacking.py