Docs: note that service.masked may require service.disabled first (#61322)#68836
Docs: note that service.masked may require service.disabled first (#61322)#68836kimimgo wants to merge 1 commit intosaltstack:masterfrom
Conversation
…ltstack#61322) Add a note to the service.masked docstring advising users to disable the service before masking it, as masking an enabled service can result in a systemd error.
|
Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here's some information that may help as you continue your Salt journey. There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar. |
|
Sounds like a code bug that should be fixed. I'm not sure papering over it with documentation is a good "fix" here |
|
@frebib Good point — you're right that Looking at it again, the proper fix would be in the Salt service module code rather than documentation. I'll update this PR to investigate the actual code path, or close it if a code fix is more appropriate. Thanks for the feedback! |
Fixes #61322
Add a note to the
service.maskeddocstring advising users todisable the service before masking it, as masking an enabled
service can result in a systemd error.