Change in vdsm[ovirt-3.3]: sysvinit: Give vdsm enough time to write profile

nsoffer at redhat.com nsoffer at redhat.com
Thu Mar 27 14:03:50 UTC 2014


Nir Soffer has uploaded a new change for review.

Change subject: sysvinit: Give vdsm enough time to write profile
......................................................................

sysvinit: Give vdsm enough time to write profile

Previously vdsm was killed brutally if it did exit 2 seconds after
receiving a SIGTERM. Currently clientIF check if shutdown is started
only once every 3 seconds. This make it impossible to write profile data
when vdsm is terminated.

This patch enlarge the shutdown delay to 10 seconds. This works fine for
profiling of serveral minutes of idle vdsm.

Change-Id: I5434caff5bc95e2a76345191dec9ce4e33070a57
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M init/sysvinit/vdsmd.init.in
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/56/26156/1

diff --git a/init/sysvinit/vdsmd.init.in b/init/sysvinit/vdsmd.init.in
index dc7847e..32a0279 100755
--- a/init/sysvinit/vdsmd.init.in
+++ b/init/sysvinit/vdsmd.init.in
@@ -148,7 +148,8 @@
         log_failure_msg "$prog: not running"
         RETVAL=0
     else
-        killproc -p "$PIDFILE" -d 2
+        # Give process enough time for saving profiling data
+        killproc -p "$PIDFILE" -d 10
         RETVAL=$?
         [ "$RETVAL" = 0 ] && "@RM_PATH@" -f "$LOCK_FILE" > /dev/null 2>&1
     fi


-- 
To view, visit http://gerrit.ovirt.org/26156
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5434caff5bc95e2a76345191dec9ce4e33070a57
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list