Rework DOCKER_PUPPET_CONFIG validate tool
* Split it to REQUIRED/OPTIONAL * Move puppet_tags to OPTIONAL as it already has a default set of tags that need not to be repeated explicitly. Change-Id: Ib70176f1edf61228771c983b0c3231fb7939a316 Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
This commit is contained in:
parent
65f1516697
commit
25fe4bf746
|
@ -28,8 +28,9 @@ REQUIRED_DOCKER_SECTIONS = ['service_name', 'docker_config', 'puppet_config',
|
|||
OPTIONAL_DOCKER_SECTIONS = ['docker_puppet_tasks', 'upgrade_tasks',
|
||||
'service_config_settings', 'host_prep_tasks',
|
||||
'metadata_settings', 'kolla_config']
|
||||
DOCKER_PUPPET_CONFIG_SECTIONS = ['config_volume', 'puppet_tags', 'step_config',
|
||||
'config_image']
|
||||
REQUIRED_DOCKER_PUPPET_CONFIG_SECTIONS = ['config_volume', 'step_config',
|
||||
'config_image']
|
||||
OPTIONAL_DOCKER_PUPPET_CONFIG_SECTIONS = [ 'puppet_tags' ]
|
||||
|
||||
|
||||
def exit_usage():
|
||||
|
@ -146,13 +147,16 @@ def validate_docker_service(filename, tpl):
|
|||
if 'puppet_config' in role_data:
|
||||
puppet_config = role_data['puppet_config']
|
||||
for key in puppet_config:
|
||||
if key in DOCKER_PUPPET_CONFIG_SECTIONS:
|
||||
if key in REQUIRED_DOCKER_PUPPET_CONFIG_SECTIONS:
|
||||
continue
|
||||
else:
|
||||
print('ERROR: %s should not be in puppet_config section.'
|
||||
% key)
|
||||
return 1
|
||||
for key in DOCKER_PUPPET_CONFIG_SECTIONS:
|
||||
if key in OPTIONAL_DOCKER_PUPPET_CONFIG_SECTIONS:
|
||||
continue
|
||||
else:
|
||||
print('ERROR: %s should not be in puppet_config section.'
|
||||
% key)
|
||||
return 1
|
||||
for key in REQUIRED_DOCKER_PUPPET_CONFIG_SECTIONS:
|
||||
if key not in puppet_config:
|
||||
print('ERROR: %s is required in puppet_config for %s.'
|
||||
% (key, filename))
|
||||
|
|
Loading…
Reference in New Issue