saltstack-formulas / logrotate-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
23 stars 71 forks source link

fix(suse): skip service #71

Open tacerus opened 1 month ago

tacerus commented 1 month ago

SUSE based distributions do not use Cron, but instead a systemd timer for running logrotate on a schedule. As this timer is enabled through a systemd preset, skip the state completely.

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

Skip service management on SUSE based distributions.

Pillar / config required to test the proposed changes

None.

Debug log showing how the proposed changes work

Too long.

Documentation checklist

Testing checklist

Additional context

tacerus commented 1 month ago

pre-commit.ci failure does not seem related to my patch.