Bug #8849
closedrpm restarts daemons on upgrade
0%
Updated by Greg Farnum almost 10 years ago
I'm all for changing this, but we want to be careful when doing so. It sounds familiar to me and I think maybe we set up packaging to restart daemons because it's part of some package requirements for one of the distros...or maybe I just remember when we switched it off in Debian (and forgot to for rpm)?
Anyway, whoever handles the packaging changes should figure out the history on both RPM and the debs to understand the full context.
Updated by Alfredo Deza almost 10 years ago
- Status changed from New to In Progress
- Assignee set to Alfredo Deza
Greg: that might be Suse, as we are specifically looking for it for certain restart-related things in the Spec file. But we do have a restart everything if ceph was installed for any other RPM
The offending part:
%postun /sbin/ldconfig if [ "$1" -ge "1" ] ; then /sbin/service ceph condrestart >/dev/null 2>&1 || : fi %if %{defined suse_version} %restart_on_update ceph %insserv_cleanup %endif
So the "restart on update" line for suse is redundant because we are actually already restarting for everyone else.
Updated by Alfredo Deza almost 10 years ago
- Status changed from In Progress to Fix Under Review
Pull request opened https://github.com/ceph/ceph/pull/2109
Updated by Sage Weil almost 10 years ago
- Status changed from Fix Under Review to Pending Backport
Updated by Sage Weil over 9 years ago
- Status changed from Pending Backport to Resolved
already backported, commit:e75dd2e4b7adb65c2de84e633efcd6c19a6e457b and ^