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.

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

New clobber_default_config paramater #247

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

traylenator
Copy link
Collaborator

@traylenator traylenator commented Jun 26, 2024

Pull Request (PR) description

Certain OSes namely Debian and Archlinux provide default rules with the OS.

This module has always respected those rules and appended all of its own rules to the end of the existing rules.

The new parameter clobber_default_config if set true (default false) will drop any existing OS provided rules.

Also related to acceptance tests only on Archlinux where the default OS provided configuration requires kernel >= 6.3 we purge the default rules if required.

Fixes

@traylenator traylenator added the enhancement New feature or request label Jun 26, 2024
@traylenator
Copy link
Collaborator Author

traylenator commented Jun 26, 2024

Along with #245 (centos8) and #246 (debian11) this should green up the tests.
Merge them first and I'll rebase this one to confirm.

Certain OSes namely Debian and Archlinux provide default rules
with the OS.

This module has always respected those rules and appended all of
its own rules to the end of the existing rules.

The new parameter `clobber_default_config` if set `true` (default `false`)
will drop any existing OS provided rules.

Also related to acceptance tests only on Archlinux where the default
OS provided configuration requires kernel >= 6.3 we purge the default rules
if required.
@traylenator traylenator changed the title New clobber_default_config paramter New clobber_default_config paramater Jun 27, 2024
@traylenator traylenator merged commit de945d0 into voxpupuli:master Jun 27, 2024
26 of 28 checks passed
@traylenator traylenator deleted the overarch branch June 27, 2024 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants