openstack
/
instack-undercloud
Code Issues Proposed changes

Don't double load config 

This isn't necessary and will likely have some bad side effects
because re-calling the conf object clears all previously registered
opts, which defeats the purpose of the second call.

Change-Id: I59bd78e85b3d55aea23b46809f4de678d394f66d
Related-Bug: 1751815
This commit is contained in:
Ben Nemec 2018-02-26 16:17:53 +00:00 committed by Ben Nemec
parent 529796bed1
commit 0497316fae
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
instack_undercloud/undercloud.py
View File

@ -2294,9 +2294,6 @@ def install(instack_root, upgrade=False):
LOG.info('Logging to %s', PATHS.LOG_FILE)
_load_config()
_load_subnets_config_groups()
# Since 'subnets' parameter in opts is used to dynamically
# add per network groups, re-load config.
_load_config()
_clean_os_refresh_config()
_clean_os_collect_config()
_validate_configuration()