diff --git a/releasenotes/notes/unused-param-warning-c5717c129adf543a.yaml b/releasenotes/notes/unused-param-warning-c5717c129adf543a.yaml new file mode 100644 index 000000000..e105e37dc --- /dev/null +++ b/releasenotes/notes/unused-param-warning-c5717c129adf543a.yaml @@ -0,0 +1,5 @@ +--- +features: + - | + Added a warning message to provide the list of parameters which are + not used in a deployment plan, but provided by the user via environments. diff --git a/tripleoclient/workflows/parameters.py b/tripleoclient/workflows/parameters.py index 54b29adbe..e54989fa1 100644 --- a/tripleoclient/workflows/parameters.py +++ b/tripleoclient/workflows/parameters.py @@ -117,14 +117,23 @@ def check_deprecated_parameters(clients, container): messages = base.wait_for_messages(workflow_client, ws, execution, 120) deprecated_params = [] + unused_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', []) if deprecated_params: print('WARNING: Following parameters 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 ' + 'currently not used.') + print('\n'.join([' {}'.format(param) + for param in unused_params]))