stackforge/cookbook-pacemaker
Code Issues Proposed changes

UCLOUDNG-296 modify cinder-volume plugin so that it greps child pid.

Browse Source
This commit is contained in:
Robert 2013-06-04 12:03:43 +09:00
parent 50d6f5daa3
commit f855722a12
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
files/default/cinder-volume
View File

@ -230,9 +230,9 @@ cinder_volume_monitor() {
# We are sure to hit the scheduler process and not other nova process with the same connection behavior (for example nova-cert)
# check the connections according to the PID
# [Robert] This isn't working because it greps wrong pid. Need to look into it later. #
# volume_amqp_check=`netstat -punt | grep -s "$OCF_RESKEY_amqp_server_port" | grep -s "$pid" | grep -qs "ESTABLISHED"`
volume_amqp_check=`netstat -punt | grep -s "$OCF_RESKEY_amqp_server_port" | grep -qs "ESTABLISHED"`
# [Robert] Fixed: Need to grep child_pid instead of pid.
child_pid=`ps -o pid --no-headers --ppid $pid`
volume_amqp_check=`netstat -punt | grep -s "$OCF_RESKEY_amqp_server_port" | grep -s "$child_pid" | grep -qs "ESTABLISHED"`
rc=$?
if [ $rc -ne 0 ]; then
ocf_log err "Cinder Volume is not connected to the AMQP server: $rc"