Back to bug 2222433

Who When What Removed Added
Red Hat Bugzilla 2023-07-12 17:55:39 UTC Pool ID sst_system_roles_rhel_8
Red Hat One Jira (issues.redhat.com) 2023-07-12 17:57:20 UTC Link ID Red Hat Issue Tracker RHELPLAN-162180
Rich Megginson 2023-07-12 17:59:18 UTC Status NEW POST
Target Release --- 8.9
Doc Type If docs needed, set a value Bug Fix
Rich Megginson 2023-07-12 18:01:07 UTC Link ID Github linux-system-roles/firewall/pull/153
Rich Megginson 2023-07-12 18:37:00 UTC Flags needinfo?(djez) needinfo?(jharuda) needinfo?(vdanek)
CC djez, jharuda, vdanek
David Jež 2023-07-19 15:25:32 UTC QA Contact rhel-cs-system-management-subsystem-qe djez
Jakub Haruda 2023-07-21 15:56:49 UTC Flags needinfo?(jharuda)
Rich Megginson 2023-07-21 17:28:26 UTC Fixed In Version rhel-system-roles-1.22.0-0.16.el8
Status POST MODIFIED
errata-xmlrpc 2023-07-21 17:33:59 UTC Status MODIFIED ON_QA
Lucie Vařáková 2023-08-09 11:58:39 UTC Docs Contact fhanzelk
Filip Hanzelka 2023-08-09 13:58:23 UTC Docs Contact fhanzelk
Rich Megginson 2023-08-10 13:52:30 UTC Doc Text Enhancement:
- firewall_lib.py - check if service exists before running firewalld methods that would cause failure
- fails if service does not exist and in diff mode, warns if in check mode and service does not exist
- README.md - reflects changes and explains interaction with check mode
- tests/tests_service.yml - add integration test case for adding non-existent services in check mode
- tests/unit/test_firewall_lib.py - Mock necessary output from fw.config().getServiceNames()
Reason:
Better compliance with Ansible best practices for check mode (not failing in check mode, especially where they would not fail in diff mode)
Reason for this particular solution - We cannot track changes from previous check modes without overhauling how check mode is handled throughout the entire system role.
Result:
Undefined services being enabled or disabled will not result in failure while in check mode, but a warning will be displayed intended to prompt the user to confirm that the service is defined in a previous play, since the same action could result in failure when run in diff mode.
Issue Tracker Tickets (Jira or BZ if any):
- Addresses GitHub Issue #146
David Jež 2023-08-11 13:44:01 UTC Status ON_QA VERIFIED
Flags needinfo?(djez)

Back to bug 2222433