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

[TECH-DEBT][BUG] RPM packages are missing init.d scripts #67888

Open
1 of 9 tasks
dmurphy18 opened this issue Mar 18, 2025 · 3 comments
Open
1 of 9 tasks

[TECH-DEBT][BUG] RPM packages are missing init.d scripts #67888

dmurphy18 opened this issue Mar 18, 2025 · 3 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior tech-debt

Comments

@dmurphy18
Copy link
Contributor

Description
RPM packages are missing their init.d scripts. Since 3006.0 rpm packages are missing their /etc/init.d/salt-xxxx scripts, see associated issue for Debian #67765

Setup
(Please provide relevant configs and/or SLS files (be sure to remove sensitive info. There is no general set-up of Salt.)

Please be as specific as possible and give set-up details.

  • on-prem machine
  • VM (Virtualbox, KVM, etc. please specify)
  • VM running on a cloud service, please be explicit and add details
  • container (Kubernetes, Docker, containerd, etc. please specify)
  • or a combination, please be explicit
  • jails if it is FreeBSD
  • classic packaging
  • onedir packaging
  • used bootstrap to install

Steps to Reproduce the behavior
(Include debug logs if possible and relevant)

Expected behavior
When a rpm package for salt-minion, salt-master, salt-syndic or salt-api, then then should also install /etc/init.d/ associated service scripts.

Screenshots
If applicable, add screenshots to help explain your problem.

Versions Report

salt --versions-report (Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)
PASTE HERE

Additional context
Add any other context about the problem here.

@dmurphy18 dmurphy18 added Bug broken, incorrect, or confusing behavior needs-triage labels Mar 18, 2025
@dmurphy18 dmurphy18 self-assigned this Mar 18, 2025
@dmurphy18 dmurphy18 added this to the Sulfur v3006.10 milestone Mar 18, 2025
@dmurphy18 dmurphy18 changed the title [BUG] RPM packages are missing init.d scripts [TECH-DEBT][BUG] RPM packages are missing init.d scripts Mar 18, 2025
@andreasdijkman
Copy link

Is this still relevant on modern RPM-systems? As EL7-support is removed, I don't think non-systemd-systems exist as EL8+ has switched to systemd-only.

@dmurphy18
Copy link
Contributor Author

@andreasdijkman Makes it easier to use on Docker containers where you don't have to jump through hoops to get systemd, and yes you can use podman, but docker is still used a lot.

Agree, real Redhat family OS or VM, doesn't matter, but having it is a convenience, tried of jumping through the hoops on docker containers from Docker Hub.

@andreasdijkman
Copy link

andreasdijkman commented Mar 20, 2025

@andreasdijkman Makes it easier to use on Docker containers where you don't have to jump through hoops to get systemd, and yes you can use podman, but docker is still used a lot.

Agree, real Redhat family OS or VM, doesn't matter, but having it is a convenience, tried of jumping through the hoops on docker containers from Docker Hub.

Ah, docker...
For 3008-roadmap: desupport docker and force everyone to Podman rootless 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior tech-debt
Projects
None yet
Development

No branches or pull requests

2 participants