Closed BrennanPaciorek closed 1 year ago
Patch and project coverage have no change.
Comparison is base (
e4042f1
) 61.05% compared to head (130d6d4
) 61.05%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Added some extra tests to verify that firewalld.conf won't change after running previous: replaced
with no changes.
[citest]
No additional changes planned. This is good to merge.
on previous replaced, revert firewalld.conf to fallback configuration instead of deletion
Enhancement: Replaces firewalld.conf on previous replaced instead of deleting when changed
Reason: EL7+ firewalld releases appeared to support a fallback configuration in firewalld, which could be effectively used to override the previous configuration.
Result: On reset, firewalld.conf is not deleted, and comments in the file are retained.
Addresses GitHub issue #164