cli: make startstop_node wait 20 seconds instead of 5 for a process to go away after...
authorZooko O'Whielacronx <zooko@zooko.com>
Tue, 30 Dec 2008 08:20:22 +0000 (01:20 -0700)
committerZooko O'Whielacronx <zooko@zooko.com>
Tue, 30 Dec 2008 08:20:22 +0000 (01:20 -0700)
Because the unit tests on the VirtualZooko buildslave failed when it took 16 seconds for a process to go away.
Perhaps getting notification after only 5 seconds instead of 20 seconds is desirable, and we should change the unit tests and set this back to 5, but I don't know exactly how to change the unit tests.  Perhaps match this particular warning message about the shutdown taking a while and allow the code under test to pass if the only stderr that it emits is this warning.

src/allmydata/scripts/startstop_node.py

index 056f76f7dddc8ca8ce49c40110a84251cea8df3c..3e366cf7c122e98b3c941ed4720c42a80eba08d8 100644 (file)
@@ -105,7 +105,7 @@ def do_stop(basedir, out=sys.stdout, err=sys.stderr):
     pid = int(pid)
 
     # kill it hard (SIGKILL), delete the twistd.pid file, then wait for the
-    # process itself to go away. If it hasn't gone away after 5 seconds, warn
+    # process itself to go away. If it hasn't gone away after 20 seconds, warn
     # the user but keep waiting until they give up.
     try:
         os.kill(pid, signal.SIGKILL)
@@ -123,7 +123,7 @@ def do_stop(basedir, out=sys.stdout, err=sys.stderr):
         pass
     start = time.time()
     time.sleep(0.1)
-    wait = 5
+    wait = 20
     first_time = True
     while True:
         # poll once per second until we see the process is no longer running