Project

General

Profile

Bug #829

when upgrading hdactl, service is not restarted

Added by cpg over 9 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
-
Target version:
-
Start date:
05/15/2011
Due date:
% Done:

0%


Description

when hdactl is upgraded, hdactl should be restarted as part of the upgrade

this is to release new versions of software seamlessly and properly over time

History

#1 Updated by cpg over 9 years ago

technically it may be better to do what in redhat's world is called a condrestart

conditional restart - restart, but only if it was running already. i.e. if stopped, it remains stopped

but this is not a big deal with this service, since it has to be on all the time for amahi to work

other services and debs we make will have to respect that, however.

#2 Updated by bigfoot65 over 8 years ago

  • Status changed from Assigned to Closed

Closing as this is not an issue with Ubuntu 12.04.

#3 Updated by cpg over 8 years ago

  • Status changed from Closed to Assigned
  • Assignee changed from eFfeM to cpg

are you sure this is fixed?
i do not think we have code to take care of this restarting on updates.
i think the updates occur, but not te restarting.

i have been learning how this is done in the deb world (pre* scripts) and i do not see code in the deb to do this. eFfeM please correct me if i am wrong.

#4 Updated by eFfeM over 8 years ago

as far as I know this is indeed not fixed yet. not sure about all the internals either

#5 Updated by eFfeM over 8 years ago

checked and it is fixed although probably not in the best way (guess ideally the pre* scripts should be used).

near the end of the postinst script is a line:
(service hdactl restart || true) > /dev/null 2>&1

that'll take care of restarting
(and yes, I guess there is too much in postinst and some things could/should be done differently; I'm not really a deb packaging wizard).

Propose to close this one.

#6 Updated by bigfoot65 about 8 years ago

  • Status changed from Assigned to Feedback
  • Priority changed from Medium to High

Think this one is resolved.

#7 Updated by bigfoot65 about 8 years ago

  • Status changed from Feedback to Closed

Closing based on recommendation from EFfeM.

Also available in: Atom