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

fix(scripts): fix rhel and alpine init scripts #919

Merged
merged 6 commits into from
Jun 18, 2024
Merged

fix(scripts): fix rhel and alpine init scripts #919

merged 6 commits into from
Jun 18, 2024

Conversation

ZNeumann
Copy link
Contributor

Systemd systems may not have access to the functions file provided by initscripts. The code from initscripts is provided under the GNU v2 license and will not be pulled in.

Additionally, a header comment block must be added, using https://refspecs.linuxbase.org/LSB_3.1.1/LSB-Core-generic/LSB-Core-generic/initscrcomconv.html for specs

@ZNeumann ZNeumann changed the title fix(scripts): allow rhel init script to be run by systemd fix(scripts): fix rhel and alpine init scripts Jun 18, 2024
agent/scripts/init.rhel Outdated Show resolved Hide resolved
agent/scripts/init.rhel Outdated Show resolved Hide resolved
ZNeumann and others added 2 commits June 18, 2024 11:33
Co-authored-by: Michal Nowacki <[email protected]>
Co-authored-by: Michal Nowacki <[email protected]>
Copy link
Member

@lavarou lavarou left a comment

Choose a reason for hiding this comment

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

LGTM

@ZNeumann ZNeumann merged commit 756c1aa into dev Jun 18, 2024
64 checks passed
@ZNeumann ZNeumann deleted the zjn/centos9 branch June 18, 2024 19:43
@lavarou lavarou added this to the next-release milestone Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants