Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The requested handler 'Change_requires_reboot' was not found in either the main handlers list nor in the listening handlers list #306

Closed
bbaassssiiee opened this issue Aug 1, 2023 · 2 comments
Closed

The requested handler 'Change_requires_reboot' was not found in either the main handlers list nor in the listening handlers list #306

bbaassssiiee opened this issue Aug 1, 2023 · 2 comments
Assignees
@uk-bolly
Labels
bug Something isn't working

Comments

@bbaassssiiee
Copy link
Member

bbaassssiiee commented Aug 1, 2023 •
edited
Loading

Describe the Issue
My build started failing with this error after task 1.10 | PATCH | Ensure system-wide crypto policy is not legacy...

Expected Behavior
The handler is in lowercase change_requires_reboot

RHEL8-CIS/handlers/main.yml

Line 131 in a6f37cc

- name: change_requires_reboot

Actual Behavior
notify: Change_requires_reboot

RHEL8-CIS/tasks/section_1/cis_1.10.yml

Line 13 in a6f37cc

notify: Change_requires_reboot

A clear and concise description of what's happening.

Possible Solutions

  • Quick: change cis_1.10.yml#L13 to notify: change_requires_reboot to match the handler.
  • Thourough: enable 'name[casing]' in ansible-lint and fix all to comply
@bbaassssiiee bbaassssiiee added the bug Something isn't working label Aug 1, 2023
@bbaassssiiee bbaassssiiee mentioned this issue Aug 1, 2023
Merged
@bbaassssiiee
Copy link
Member Author

bbaassssiiee commented Aug 1, 2023 •
edited
Loading

@MarcoV-git #303

bbaassssiiee added a commit to TeamSalvador/RHEL8-CIS that referenced this issue Aug 3, 2023
@bbaassssiiee
Fix for ansible-lockdown#306
27cac4d 
Signed-off-by: Bas Meijer <bas.meijer@me.com>
@bbaassssiiee bbaassssiiee mentioned this issue Aug 3, 2023
Merged
uk-bolly added a commit that referenced this issue Aug 4, 2023
@uk-bolly
Merge pull request #307 from TeamSalvador/bugfix/306
6401b1b 
Fix for #306 - pre-commit not required as yet merging request
@uk-bolly uk-bolly self-assigned this Aug 4, 2023
@uk-bolly uk-bolly mentioned this issue Aug 4, 2023
Merged
@bbaassssiiee
Copy link
Member Author

Thanks

@uk-bolly uk-bolly mentioned this issue Aug 9, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@uk-bolly uk-bolly

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bbaassssiiee @uk-bolly