-
Notifications
You must be signed in to change notification settings - Fork 70
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
selinux_module force install mode #118
Comments
Have you looked at the built-in resource in chef-client https://docs.chef.io/resources/selinux_module/ |
Otherwise what i've done in the past to help with idempotency for something like this is to write the config into a file resource. Then the file resource calls the custom resource with notifies and desired action. Thus if file's content changes due to module configuration changes then it triggers the update. |
We are not using the built in module resource, since we are on older
version of Chef.
The notification is a good idea. Would that also need the force_install
switch?
Thanks
…On Mon, Apr 15, 2024 at 2:41 PM Corey Hemminger ***@***.***> wrote:
Otherwise what i've done in the past to help with idempotency for
something like this is to write the config into a file resource. Then the
file resource calls the custom resource with notifies and desired action.
Thus if file's content changes due to module configuration changes then it
triggers the update.
—
Reply to this email directly, view it on GitHub
<#118 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHJK3CPWLSQP7RWIULIZNGLY5QUOFAVCNFSM6AAAAABGHZ6TYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXGY3TCNBUGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
@Stromweld Any further comments? |
🙍 Problem Statement
The "selinux_module" resource would skip the install, if the module is already installed. So there is no way to install a new version without first removing the module first. But removing the module can cause run time issue.
❔ Possible Solution
This is a fundamental issue since RHEL8 "semodule -l" no longer provides the module version information. So any solution would assume the absence of the information. I ended up with a workaround.
But for this to work (to install the updated module), there need to be a "force_install" mode for the "selinux_module" resource.
I am not aware of any, and would welcome ideas.
➕ Additional context
If it's acceptable, I submit a PR to add the "force_install" flag.
The text was updated successfully, but these errors were encountered: