Skip to content

Switch Default for Restart-Command to systemctl - #457

Merged
d1nuc0m merged 3 commits into
voxpupuli:masterfrom
marcusdots:master
Oct 14, 2026
Merged

d1nuc0m merged 3 commits into
voxpupuli:masterfrom
marcusdots:master

Conversation

@marcusdots

Copy link
Copy Markdown
Contributor

This superseeds #443

Almost all Linux-Distributions now default to Systemd. So switch the Default for Restart-Command to systemctl.

  • use /usr/bin/systemctl instead of /bin/systemctl, because with usr-Merge the former Dir is dead
  • keep Alpine on /etc/init.d

@marcusdots marcusdots added the backwards-incompatible This change will lead to a major version bump for the next release label Sep 15, 2026
@d1nuc0m

d1nuc0m commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Now, Idk about Alpine, but with systemd shouldn't a notify to service resource be enough?

@cocker-cc

Copy link
Copy Markdown
Contributor

Now, Idk about Alpine, but with systemd shouldn't a notify to service resource be enough?

Yes. Move subscribe => Package['postfix'], from the exec to the service, and delete the exec-Block.
Puppet will handle this on Alpine appropriately.

@marcusdots

Copy link
Copy Markdown
Contributor Author

I incorporated your feedback, removed an exec and handed dependencies to service-ressource

Comment thread data/osfamily/RedHat/6.yaml Resolved
cocker-cc and others added 2 commits October 9, 2026 10:19
Almost all Linux-Distributions now default to Systemd.  So switch the
Default for Restart-Command to systemctl.

- use /usr/bin/systemctl instead of /bin/systemctl, because with
  usr-Merge the former Dir is dead
- keep Alpine and RedHat 6 on init.d
Signed-off-by: Marcus Poller <marcus.poller@betadots.de>
@marcusdots
marcusdots force-pushed the master branch 2 times, most recently from cdd1473 to 03a9048 Compare October 12, 2026 10:43
…before executing the test suite, we will reach idempotency one OpenVox run faster.
Comment thread data/osfamily/RedHat/6.yaml Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't EL6 EOL?

@d1nuc0m

d1nuc0m commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

@marcusdots can you squash the commits?

@d1nuc0m
d1nuc0m merged commit 7f4c4c3 into voxpupuli:master Oct 14, 2026
71 of 73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backwards-incompatible This change will lead to a major version bump for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants