Skip to content

Commit

Permalink
init/service scripts: make environment filename consistent with servi…
Browse files Browse the repository at this point in the history
…ce name

Generally speaking if a service is called foo, users expects to pass
options to the script via /etc/sysconfig/foo and not foo_something.

This fix an oversight on my side in the previous round of init fixes.

Signed-off-by: Fabio M. Di Nitto <[email protected]>
  • Loading branch information
fabbione committed Feb 13, 2012
1 parent 792fdad commit 0f19d2e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion init/dlm.init
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ progdir="cluster"
lockfile="/var/run/$progdir/$prog.pid"
exec="/usr/sbin/$prog"

[ -f /etc/sysconfig/$prog ] && . /etc/sysconfig/$prog
[ -f /etc/sysconfig/dlm ] && . /etc/sysconfig/dlm

setup() {
modprobe dlm > /dev/null 2>&1
Expand Down
2 changes: 1 addition & 1 deletion init/dlm.service
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ After=syslog.target network.target corosync.service

[Service]
Type=forking
EnvironmentFile=/etc/sysconfig/dlm_controld
EnvironmentFile=/etc/sysconfig/dlm
ExecStartPre=/sbin/modprobe dlm
ExecStart=/usr/sbin/dlm_controld $DLM_CONTROLD_OPTS
#ExecStopPost=/sbin/modprobe -r dlm
Expand Down

0 comments on commit 0f19d2e

Please sign in to comment.