Skip to content

Commit a9a3a27

Browse files
author
Matthew Sackman
committed
Be consistent about attempting to remove the directory whenever we can; remove purge deletion of var/run which has existed since root of hg repo.
1 parent 6e22a7a commit a9a3a27

File tree

3 files changed

+14
-7
lines changed

3 files changed

+14
-7
lines changed

packaging/common/rabbitmq-server.init

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,11 @@ ensure_pid_dir () {
4343
fi
4444
}
4545

46+
remove_pid () {
47+
rm -f ${PID_FILE}
48+
rmdir `dirname ${PID_FILE}` || :
49+
}
50+
4651
start_rabbitmq () {
4752
status_rabbitmq quiet
4853
if [ $RETVAL = 0 ] ; then
@@ -64,7 +69,7 @@ start_rabbitmq () {
6469
fi
6570
;;
6671
*)
67-
rm -f $PID_FILE
72+
remove_pid
6873
echo FAILED - check ${INIT_LOG_DIR}/startup_\{log, _err\}
6974
RETVAL=1
7075
;;
@@ -80,7 +85,7 @@ stop_rabbitmq () {
8085
RETVAL=$?
8186
set -e
8287
if [ $RETVAL = 0 ] ; then
83-
rm -rf `dirname $PID_FILE`
88+
remove_pid
8489
if [ -n "$LOCK_FILE" ] ; then
8590
rm -f $LOCK_FILE
8691
fi

packaging/common/rabbitmq-server.ocf

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,11 @@ ensure_pid_dir () {
189189
return $OCF_SUCCESS
190190
}
191191

192+
remove_pid () {
193+
rm -f ${RABBITMQ_PID_FILE}
194+
rmdir `dirname ${RABBITMQ_PID_FILE}` || :
195+
}
196+
192197
export_vars() {
193198
[ ! -z $RABBITMQ_NODE_IP_ADDRESS ] && export RABBITMQ_NODE_IP_ADDRESS
194199
[ ! -z $RABBITMQ_NODE_PORT ] && export RABBITMQ_NODE_PORT
@@ -278,7 +283,7 @@ rabbit_start() {
278283
rabbit_wait $RABBITMQ_PID_FILE
279284
rc=$?
280285
if [ "$rc" != $OCF_SUCCESS ]; then
281-
rm -f $RABBITMQ_PID_FILE
286+
remove_pid
282287
ocf_log info "rabbitmq-server start failed: $rc"
283288
exit $OCF_ERR_GENERIC
284289
fi
@@ -309,7 +314,7 @@ rabbit_stop() {
309314
rabbit_status
310315
rc=$?
311316
if [ "$rc" = $OCF_NOT_RUNNING ]; then
312-
rm -rf `dirname ${RABBITMQ_PID_FILE}`
317+
remove_pid
313318
stop_wait=0
314319
break
315320
elif [ "$rc" != $OCF_SUCCESS ]; then

packaging/debs/Debian/debian/postrm.in

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,6 @@ case "$1" in
3232
if [ -d /var/log/rabbitmq ]; then
3333
rm -r /var/log/rabbitmq
3434
fi
35-
if [ -d /var/run/rabbitmq ]; then
36-
rm -r /var/run/rabbitmq
37-
fi
3835
if [ -d /etc/rabbitmq ]; then
3936
rm -r /etc/rabbitmq
4037
fi

0 commit comments

Comments
 (0)