Project

General

Profile

Actions

Bug #8849

closed

rpm restarts daemons on upgrade

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

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

0%

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

Related issues 1 (0 open1 closed)

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

Actions
Actions #1

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.

Actions #2

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.

Actions #3

Updated by Alfredo Deza almost 10 years ago

  • Status changed from In Progress to Fix Under Review
Actions #4

Updated by Sage Weil almost 10 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #5

Updated by Sage Weil almost 10 years ago

  • Backport set to firefly
Actions #6

Updated by Sage Weil over 9 years ago

  • Status changed from Pending Backport to Resolved

already backported, commit:e75dd2e4b7adb65c2de84e633efcd6c19a6e457b and ^

Actions

Also available in: Atom PDF