diff options
| author | Jenkins <jenkins@review.openstack.org> | 2015-09-28 11:22:25 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2015-09-28 11:22:25 +0000 |
| commit | edca14421e4f1e3c4c77d2af83beef960aa47578 (patch) | |
| tree | 345d8fa94de2182b0878d341076dfb6804dc1832 /docker/base | |
| parent | a0ac8764948f2dba28f648d053cd0a68d8c3e78d (diff) | |
| parent | 7622e53cc10b4871ebd3506bf590a58c1af76a9f (diff) | |
| download | kolla-edca14421e4f1e3c4c77d2af83beef960aa47578.tar.gz kolla-edca14421e4f1e3c4c77d2af83beef960aa47578.zip | |
Merge "Allow conf files to be optional"
Diffstat (limited to 'docker/base')
| -rw-r--r-- | docker/base/set_configs.py | 23 |
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') |
