Bug #8849
closed
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.
- 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.
- Status changed from In Progress to Fix Under Review
- Status changed from Fix Under Review to Pending Backport
- Status changed from Pending Backport to Resolved
already backported, commit:e75dd2e4b7adb65c2de84e633efcd6c19a6e457b and ^
Also available in: Atom
PDF