Bug #8849
rpm restarts daemons on upgrade
0%
Related issues
History
#1 Updated by Greg Farnum over 9 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.
#2 Updated by Alfredo Deza over 9 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.
#3 Updated by Alfredo Deza over 9 years ago
- Status changed from In Progress to Fix Under Review
Pull request opened https://github.com/ceph/ceph/pull/2109
#4 Updated by Sage Weil over 9 years ago
- Status changed from Fix Under Review to Pending Backport
#5 Updated by Sage Weil over 9 years ago
- Backport set to firefly
#6 Updated by Sage Weil over 9 years ago
- Status changed from Pending Backport to Resolved
already backported, commit:e75dd2e4b7adb65c2de84e633efcd6c19a6e457b and ^