use 'docker stop' instead of 'docker kill'

rhansen · rhansen · commit c56fc47501a5 · 2016-01-31T16:30:32.000-05:00
The 'stop' command first tries SIGTERM before resorting to SIGKILL,
which is a gentler way to stop processes.  (SIGTERM gives processes an
opportunity to clean up before exiting; SIGKILL can't be caught so it
is very abrupt.)
diff --git a/tools/build_test_env.sh b/tools/build_test_env.sh
@@ -55,8 +55,8 @@ CONFIG=/tmp/python-gitlab.cfg
 
 cleanup() {
     rm -f "${CONFIG}"
-    log "Killing gitlab-test docker container..."
-    docker kill gitlab-test >/dev/null 2>&1
+    log "Stopping gitlab-test docker container..."
+    docker stop gitlab-test >/dev/null 2>&1
     log "Removing gitlab-test docker container..."
     docker rm gitlab-test >/dev/null 2>&1
     log "Deactivating Python virtualenv..."
