openstack
/
openstack-virtual-baremetal
Code Issues Proposed changes

Fix environment-generator 

When the pep8 check that environments are up to date was added it
added a mandatory output_path parameter. This wasn't intentional
as it should have still defaulted to 'environments' and it broke the
genconfig tox target.

This change correctly makes the parameter optional so both the pep8
check and genconfig will work as expected.

Change-Id: I2e80956f3c433d2f09966364a7d53cf56114ea21
This commit is contained in:
Ben Nemec 2019-02-12 16:28:00 -06:00
parent 277a36dd26
commit a77c4c2fb2
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
bin/environment-generator.py
View File

@ -238,7 +238,9 @@ def _parse_args():
help='Filename or directory containing the sample '
'environment definitions.')
parser.add_argument('output_path',
help='Location to write generated files.')
help='Location to write generated files.',
default='environments',
nargs='?')
parser.add_argument('--index',
help='Specify the output path for an index file '
'listing all the generated environments. '