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

(PUP-11433) Use systemd by default in CentOS/RHEL #9207

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

mhashizume
Copy link
Contributor

Since CentOS7/RHEL7, systemd is used to manage services. This change changes the default provider in CentOS/RHEL from the old redhat provider based on init, to the systemd provider, so that we no longer need to update the provider code when adding support for a new version of CentOS/RHEL.

The old redhat provider is still used for CentOS/RHEL<7 which are using init to manage services.

Since CentOS7/RHEL7, systemd is used to manage services. This change
changes the default provider in CentOS/RHEL from the old redhat
provider based on init, to the systemd provider, so that we no longer
need to update the provider code when adding support for a new version
of CentOS/RHEL.

The old redhat provider is still used for CentOS/RHEL<7 which are using
init to manage services.
@mhashizume mhashizume added the bug Something isn't working label Jan 16, 2024
@mhashizume mhashizume requested a review from a team as a code owner January 16, 2024 19:47
@mhashizume
Copy link
Contributor Author

mhashizume commented Jan 16, 2024

This PR comprises work from @kajinamit (originally in #8862) with a few changes:

  • Uses structured facts instead of legacy facts
  • Excludes Amazon Linux changes, as that work was done in 98e2254
  • More narrowly describes RHEL using os.name instead of os.family so as to not make more sweeping changes to other RHEL-derivative distros (Rocky, Amazon, etc.)

@AriaXLi AriaXLi merged commit 838a496 into puppetlabs:main Jan 17, 2024
10 checks passed
@mhashizume mhashizume deleted the rhel-systemd branch January 17, 2024 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants