diff --git a/releasenotes/notes/role-specific-warning-63ce21643d51236c.yaml b/releasenotes/notes/role-specific-warning-63ce21643d51236c.yaml new file mode 100644 index 000000000..2fea19dcb --- /dev/null +++ b/releasenotes/notes/role-specific-warning-63ce21643d51236c.yaml @@ -0,0 +1,5 @@ +--- +features: + - | + Added a warning message if user has provided an invalid role-specific + parameter in the environment file. diff --git a/tripleoclient/workflows/parameters.py b/tripleoclient/workflows/parameters.py index 7d724da67..d7cfa600a 100644 --- a/tripleoclient/workflows/parameters.py +++ b/tripleoclient/workflows/parameters.py @@ -110,22 +110,31 @@ def check_deprecated_parameters(clients, container): deprecated_params = [] unused_params = [] + invalid_role_specific_params = [] for message in messages: if message['status'] == 'SUCCESS': for param in message.get('deprecated', []): if param.get('user_defined'): deprecated_params.append(param['parameter']) unused_params = message.get('unused', []) + invalid_role_specific_params = message.get( + 'invalid_role_specific', []) if deprecated_params: - print('WARNING: Following parameters are deprecated and still ' + print('WARNING: Following parameter(s) are deprecated and still ' 'defined. Deprecated parameters will be removed soon!') print('\n'.join([' {}'.format(param) for param in deprecated_params])) if unused_params: - print('WARNING: Following parameters are defined but not used in ' - 'plan. Could be possible that parameter is valid but ' + print('WARNING: Following parameter(s) are defined but not used ' + 'in plan. Could be possible that parameter is valid but ' 'currently not used.') print('\n'.join([' {}'.format(param) for param in unused_params])) + + if invalid_role_specific_params: + print('WARNING: Following parameter(s) are not supported as ' + 'role-specific inputs.') + print('\n'.join([' {}'.format(param) + for param in invalid_role_specific_params]))