You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The notify: directive for running handlers supports a syntax <role name> : <Task name> to help avoid name collisions between handlers from different roles.
As role names are usually all lower case, the changes in #4149, which check the casing of the first letter in the argument to a notify: directive, cause failures when this <role name> : <Task name> syntax is used.
Issue Type
Bug Report
OS / ENVIRONMENT
ansible-lint --versionansible-lint 24.5.0 using ansible-core:2.16.6 ansible-compat:24.5.1 ruamel-yaml:0.18.6 ruamel-yaml-clib:0.2.8
ansible installation method: pip
ansible-lint installation method: pip
STEPS TO REPRODUCE
Have a role role with a tasks/main.yml as follows:
The argument of the notify: directive fails the name[casing] lint, even though it is legal syntax and the handler and role names individually follow conventions and pass ansible-lint's checks.
name[casing]: Task notify 'role : Handler' should start with an uppercase letter.
roles/role/tasks/main.yml:1 Task/Handler: Task
The text was updated successfully, but these errors were encountered:
Summary
The
notify:
directive for running handlers supports a syntax<role name> : <Task name>
to help avoid name collisions between handlers from different roles.As role names are usually all lower case, the changes in #4149, which check the casing of the first letter in the argument to a
notify:
directive, cause failures when this<role name> : <Task name>
syntax is used.Issue Type
OS / ENVIRONMENT
STEPS TO REPRODUCE
Have a role
role
with atasks/main.yml
as follows:and a
handlers/main.yml
as follows:Then run
ansible-lint
onrole/tasks/main.yml
.Desired Behavior
No failures.
Actual Behavior
The argument of the
notify:
directive fails thename[casing]
lint, even though it is legal syntax and the handler and role names individually follow conventions and passansible-lint
's checks.The text was updated successfully, but these errors were encountered: