From fd96bcd22faf3656f0f830203841bc07d0e22937 Mon Sep 17 00:00:00 2001 From: Martin Krizek Date: Thu, 29 Mar 2018 13:53:01 +0200 Subject: [PATCH] Use correct action in warning about missing name --- lib/ansible/playbook/role_include.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/ansible/playbook/role_include.py b/lib/ansible/playbook/role_include.py index 9d32f432138..423fcdd4e63 100644 --- a/lib/ansible/playbook/role_include.py +++ b/lib/ansible/playbook/role_include.py @@ -112,12 +112,12 @@ class IncludeRole(TaskInclude): # name is needed, or use role as alias ir._role_name = ir.args.get('name', ir.args.get('role')) if ir._role_name is None: - raise AnsibleParserError("'name' is a required field for include_role.") + raise AnsibleParserError("'name' is a required field for %s." % ir.action) # validate bad args, otherwise we silently ignore bad_opts = my_arg_names.difference(IncludeRole.VALID_ARGS) if bad_opts: - raise AnsibleParserError('Invalid options for include_role: %s' % ','.join(list(bad_opts))) + raise AnsibleParserError('Invalid options for %s: %s' % (ir.action, ','.join(list(bad_opts)))) # build options for role includes for key in my_arg_names.intersection(IncludeRole.FROM_ARGS):