Add back Debian support for Sys V init.d service scripts#69578
Open
dwoz wants to merge 1 commit into
Open
Conversation
Restore init.d scripts for salt-minion, salt-master, salt-api, and salt-syndic that were removed from Debian packages. Systems without systemd (e.g. Devuan) need these scripts to manage Salt services. Also add test_salt_sysv_service_files to verify init.d scripts are present in Debian and RedHat packages, and remove the ubuntu/debian exclusion from the Linux PID change assertion in salt_test_upgrade so Debian/Ubuntu upgrades are also validated. Fixes saltstack#67765
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Rebase of #67978 (twangboy's rebase of #67909) onto current 3006.x.
Restores init.d scripts for salt-minion, salt-master, salt-api, and salt-syndic in
Debian packages. Systems without systemd (e.g. Devuan, bare Debian) need these
scripts to manage Salt services via
serviceor/etc/init.d/directly.What issues does this PR fix or reference?
Fixes #67765
Changes
pkg/debian/salt-{minion,master,api,syndic}.initSys V init scriptstest_salt_sysv_service_filesto verify init.d scripts are present in packagessalt_test_upgradeso Debian/Ubuntu upgrades are also validated post-upgrade