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(a)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(a)redhat.com>