aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2015-09-28 11:22:25 +0000
committerGerrit Code Review <review@openstack.org>2015-09-28 11:22:25 +0000
commitedca14421e4f1e3c4c77d2af83beef960aa47578 (patch)
tree345d8fa94de2182b0878d341076dfb6804dc1832
parenta0ac8764948f2dba28f648d053cd0a68d8c3e78d (diff)
parent7622e53cc10b4871ebd3506bf590a58c1af76a9f (diff)
downloadkolla-edca14421e4f1e3c4c77d2af83beef960aa47578.tar.gz
kolla-edca14421e4f1e3c4c77d2af83beef960aa47578.zip
Merge "Allow conf files to be optional"
-rw-r--r--docker/base/set_configs.py23
1 files changed, 17 insertions, 6 deletions
diff --git a/docker/base/set_configs.py b/docker/base/set_configs.py
index 7b5773550..19415c6fc 100644
--- a/docker/base/set_configs.py
+++ b/docker/base/set_configs.py
@@ -43,13 +43,23 @@ def validate_config(config):
sys.exit(1)
-def copy_files(data):
- dest = data.get('dest')
+def validate_source(data):
source = data.get('source')
if not os.path.exists(source):
- LOG.error('The source to copy does not exist: {}'.format(source))
- sys.exit(1)
+ if data.get('optional'):
+ LOG.warn('{} does not exist, but is not required'.format(source))
+ return False
+ else:
+ LOG.error('The source to copy does not exist: {}'.format(source))
+ sys.exit(1)
+
+ return True
+
+
+def copy_files(data):
+ dest = data.get('dest')
+ source = data.get('source')
if os.path.exists(dest):
LOG.info('Removing existing destination: {}'.format(dest))
@@ -149,8 +159,9 @@ def load_config():
if 'config_files' in config:
LOG.info('Copying service configuration files')
for data in config['config_files']:
- copy_files(data)
- set_permissions(data)
+ if validate_source(data):
+ copy_files(data)
+ set_permissions(data)
else:
LOG.debug('No files to copy found in config')