Project

General

Profile

Bug #8849

rpm restarts daemons on upgrade

Added by Sage Weil over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Community (user)
Tags:
Backport:
firefly
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Related issues

Related to devops - Bug #9061: dumpling to firefly upgrade on RH6 restarts the daemons Resolved 08/10/2014

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

#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 ^

Also available in: Atom PDF