Bug 533247 - clvmd initscript [LSB|FedoraGuidelines] compliance
Summary: clvmd initscript [LSB|FedoraGuidelines] compliance
Alias: None
Product: Fedora
Classification: Fedora
Component: lvm2-cluster
Version: 12
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Fabio Massimo Di Nitto
QA Contact: Corey Marthaler
Depends On:
Blocks: InitScriptsProject 553381
TreeView+ depends on / blocked
Reported: 2009-11-05 19:12 UTC by Nate Straz
Modified: 2010-03-11 17:37 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 553381 (view as bug list)
Last Closed: 2010-02-26 13:09:17 UTC
Type: ---

Attachments (Terms of Use)
Patch from Fabio (4.72 KB, patch)
2010-02-04 15:14 UTC, Christine Caulfield
no flags Details | Diff

Description Nate Straz 2009-11-05 19:12:51 UTC
Description of problem:

This is a list of issues for the clvmd init script when compared with the criteria in 

Version-Release number of selected component (if applicable):

LSB Header is missing Description: section

Missing required action force-reload

Missing required action try-restart and condrestart

`service clvmd` should have exit status 2, currently 1.

Exit status for an invalid action should be 2, currently 1.

 Should clvmd start up if the locking method is set to "1" for local locking?

Comment 1 Nate Straz 2009-11-11 22:41:42 UTC
Should the clvmd init script require cman is already started?  If so, it should be included in the LSB header.

Comment 2 Bug Zapper 2009-11-16 15:09:43 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 12 development cycle.
Changing version to '12'.

More information and reason for this action is here:

Comment 3 Christine Caulfield 2010-02-04 15:14:22 UTC
Created attachment 388811 [details]
Patch from Fabio

- fix whitespaces all over (tabs/spaces)
- use secure path
- remove hardcoding of paths for tools
- increase timeout to 30 secs (on Chrissie request)
- source both cluster and clvmd for options (like all the other cluster
init scripts)
- add clustered_vgs and _lvs commodity fns
- move rh_status* fns at the top, so they can be reused
- heavily cleanup start and stop fns from redundant code and unnecessary
- drop wait_for_finish. According to docs killproc will send first
specified signals and then SIGKILL and do the way for us.
- improve output from status
- make the init script lsb compliant

Please review and let me know if we need to add more changes.

Both Chrissie and I have tested this new version.


Comment 4 Fabio Massimo Di Nitto 2010-02-26 13:09:17 UTC
A new version of the init script has been committed in upstream CVS.

Note You need to log in before you can comment on or make changes to this bug.