-
Notifications
You must be signed in to change notification settings - Fork 34
Add special treatment of certain deb services #169
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
Conversation
| if service == "dbus-broker": | ||
| cmd = "systemctl restart dbus" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where does dbus-broker come from? I don't see that on my Debian 12 system.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got these special treatments from [1] and tried to apply the most useful of them. We got a system on which we had the systemd-manager issue.
[1] https://github.com/liske/needrestart/tree/master/ex/restart.d
For certain debian services a special treatment is necessary to resolve the traces of the system.
b64c4d3 to
76e9b87
Compare
|
Looking at other maintainers: do we still want to enforce Redmine issues here? |
|
Thoughs @ianballou ? |
We can look after Monday next week. I'd need to try this out to fully understand. |
|
I don't care too much about the redmine requirement here in katello-host-tools, mostly because so many commits have gone by without redmines already. |
ianballou
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks straight-forward, I don't think I need to do a test after all now that I've given it a closer read. Ack!
For certain debian services a special treatment is necessary to resolve the traces of the system.