Ubuntu
smartmontools package

smartmontools init scripts lacks the 'status' action

Bug #300535 reported by Steven Danna
4
Affects Status Importance Assigned to Milestone
smartmontools (Ubuntu)
Fix Released
Wishlist
Unassigned
Ubuntu jaunty-alpha-2

Bug Description

Binary package hint: smartmontools

smartmontools init script lacks the 'status' action. It would be useful to be able to determine if the daemon is running or not by querying:
 * /etc/init.d/smartmontools status

A concerted effort is being made to add status actions to many Ubuntu services' init scripts:
 * https://wiki.ubuntu.com/InitScriptStatusActions

Revision history for this message
Steven Danna (ssd7) wrote :

I created a patch using the directions at https://wiki.ubuntu.com/InitScriptStatusActions .

Changed in smartmontools:
status: New → Confirmed
Revision history for this message
Dustin Kirkland  (kirkland) wrote : Re: [Bug 300535] [NEW] smartmontools init scripts lacks the 'status' action

Thanks for the bug report!

Any chance you have a patch that adds the status action?

:-Dustin

Revision history for this message
Steven Danna (ssd7) wrote :

I believe that the patch I attached last night should add the status function. This was my first attempt to create a patch/use launchpad so please let me know if there are any problems with it.

Cheers,

Steven Danna

Revision history for this message
Luke Yelavich (themuso) wrote :

Thanks for your patch. Could you please re-create your patch against the latest version of smartmontools in jaunty? I'm going to unsubscribe ubuntu-main-sponsors, so once you are done and have uploaded the new patch, please re-subscribe ubuntu-main-sponsors for it to be attended to.

Also, the syntax for the bug closure in the changelog is not quite correct. You are missing a # before the bug number, so it should be (LP: #nnnnnn).

Thanks for your work.

Changed in smartmontools:
status: Confirmed → Incomplete
Revision history for this message
Steven Danna (ssd7) wrote : Re: [Bug 300535] Re: smartmontools init scripts lacks the 'status' action

Thanks for the feedback! I plan to re-submit this patch for jaunty in a
few weeks once I have some free time again.

Cheers,

Steve

Revision history for this message
Steven Danna (ssd7) wrote :

I think this patch should work. It is targeted at jaunty and the LP reference is fixed.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

+1, thanks for the patch, Steven. Hopefully, someone can get this sponsored for you soon.

:-Dustin

Changed in smartmontools:
importance: Undecided → Wishlist
milestone: none → jaunty-alpha-2
status: Incomplete → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package smartmontools - 5.38-2ubuntu2

---------------
smartmontools (5.38-2ubuntu2) jaunty; urgency=low

  * debian/control: Depend on lsb >= 3.2-14, which has the status_of_proc() function.
  * debian/smartmontools.init: Add the 'status' action (LP: #300535).

 -- Steven Danna <email address hidden> Mon, 01 Dec 2008 23:41:09 -0500

Changed in smartmontools:
status: In Progress → Fix Released
Revision history for this message
Jarl (jarl-dk) wrote :

For your information: 'status' action now exists (in Karmic), however it fails. I have created bug 491324 regarding that.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.