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

[BUG] ssl.conf should never be removed - should always be managed #381

Open
TheBigBear opened this issue Aug 25, 2022 · 0 comments
Open
Labels

Comments

@TheBigBear
Copy link

Your setup

Formula commit hash / release tag

v1.2.2 4a46e03

Versions reports (master & minion)

Pillar / config used


Bug details

Describe the bug

In the redhat family the file apache/config/modules/mod_ssl.sls on line 52 has a file.absent state for the ssl.conf file.

This is wrong. Any subsequent dnf or yum update will re-introduce the default distributors version of it.
Which can conflict and break things. The correct way would be to actually use file.managed at this point.
If there is a modified ( and managed ) file there rpm will not introduce one but simply create a ssl.conf.rpmnew which does not break anything.

Using file.absent on a config file maintained by a linux package manager breaks idempotency.

Steps to reproduce the bug

Expected behaviour

Attempts to fix the bug

Additional context

@TheBigBear TheBigBear added the bug label Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant