testserver: use SIGTERM instead of SIGKILL during upgrade

This makes it shutdown gracefully instead of abruptly.
cockroachdb · Mar 10, 2023 · d465a56 · d465a56
1 parent ad4962b
commit d465a56
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/testserver/testservernode.go b/testserver/testservernode.go
@@ -19,6 +19,7 @@ import (
 	"log"
 	"os/exec"
 	"strings"
+	"syscall"
 )
 
 func (ts *testServerImpl) StopNode(nodeNum int) error {
@@ -29,7 +30,7 @@ func (ts *testServerImpl) StopNode(nodeNum int) error {
 
 	// Kill the process.
 	if cmd.Process != nil {
-		if err := cmd.Process.Kill(); err != nil {
+		if err := cmd.Process.Signal(syscall.SIGTERM); err != nil {
 			return err
 		}
 		if _, err := cmd.Process.Wait(); err != nil {