saltstack-formulas / apache-formula

Set up and configure the Apache HTTP server
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
50 stars 285 forks source link

fix(logs): don't change owners of logfiles with salt #291

Closed SuperTux88 closed 3 years ago

SuperTux88 commented 3 years ago

The logs are created by apache or logrotate, but salt shouldn't change permissions of existing logfiles everytime it runs.

PR progress checklist (to be filled in by reviewers)


What type of PR is this?

Primary type

Secondary type

Does this PR introduce a BREAKING CHANGE?

No.

Related issues and/or pull requests

Describe the changes you're proposing

Don't change owners of existing logfiles everytime salt runs. The logs are created by apache or logrotate, so salt shouldn't change them.

Pillar / config required to test the proposed changes

Debug log showing how the proposed changes work

Documentation checklist

Testing checklist

Additional context

saltstack-formulas-travis commented 3 years ago

:tada: This PR is included in version 1.0.4 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: