Skip to content

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

Description

@TheBigBear

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

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions