From 5072530a802e543c14e844cf9cbdb535fde63c7d Mon Sep 17 00:00:00 2001
From: Anthony Romano <anthony.romano@coreos.com>
Date: Thu, 3 Aug 2017 11:18:36 -0700
Subject: [PATCH] e2e: remove SIGQUIT debugging for elect and lock

Causes etcdctl to hang with pending SIGQUIT signals according to
/proc/pid/status. The debugging wasn't very useful on travis
either; just totally remove it to get CI working again.
---
 e2e/ctl_v3_elect_test.go |  5 -----
 e2e/ctl_v3_lock_test.go  | 28 ----------------------------
 2 files changed, 33 deletions(-)

diff --git a/e2e/ctl_v3_elect_test.go b/e2e/ctl_v3_elect_test.go
index 410c00f813c..02c7090f7f9 100644
--- a/e2e/ctl_v3_elect_test.go
+++ b/e2e/ctl_v3_elect_test.go
@@ -32,10 +32,6 @@ func TestCtlV3Elect(t *testing.T) {
 }
 
 func testElect(cx ctlCtx) {
-	// debugging for #6934
-	sig := cx.epc.WithStopSignal(debugLockSignal)
-	defer cx.epc.WithStopSignal(sig)
-
 	name := "a"
 
 	holder, ch, err := ctlV3Elect(cx, name, "p1")
@@ -112,7 +108,6 @@ func ctlV3Elect(cx ctlCtx, name, proposal string) (*expect.ExpectProcess, <-chan
 		close(outc)
 		return proc, outc, err
 	}
-	proc.StopSignal = debugLockSignal
 	go func() {
 		s, xerr := proc.ExpectFunc(func(string) bool { return true })
 		if xerr != nil {
diff --git a/e2e/ctl_v3_lock_test.go b/e2e/ctl_v3_lock_test.go
index ddda3fc03cc..6bcbe4de2c5 100644
--- a/e2e/ctl_v3_lock_test.go
+++ b/e2e/ctl_v3_lock_test.go
@@ -16,36 +16,13 @@ package e2e
 
 import (
 	"os"
-	"runtime"
 	"strings"
-	"syscall"
 	"testing"
 	"time"
 
 	"github.com/coreos/etcd/pkg/expect"
 )
 
-// debugLockSignal forces SIGQUIT to debug etcdctl elect and lock failures
-var debugLockSignal os.Signal
-
-func init() {
-	// hacks to ignore SIGQUIT debugging for some builds
-	switch {
-	case os.Getenv("COVERDIR") != "":
-		// SIGQUIT interferes with coverage collection
-		debugLockSignal = syscall.SIGTERM
-	case runtime.GOARCH == "ppc64le":
-		// ppc64le's signal handling won't kill processes with SIGQUIT
-		// in the same way as amd64/i386, so processes won't terminate
-		// as expected. Since this debugging code for CI, just ignore
-		// ppc64le.
-		debugLockSignal = syscall.SIGKILL
-	default:
-		// stack dumping OK
-		debugLockSignal = syscall.SIGQUIT
-	}
-}
-
 func TestCtlV3Lock(t *testing.T) {
 	oldenv := os.Getenv("EXPECT_DEBUG")
 	defer os.Setenv("EXPECT_DEBUG", oldenv)
@@ -55,10 +32,6 @@ func TestCtlV3Lock(t *testing.T) {
 }
 
 func testLock(cx ctlCtx) {
-	// debugging for #6464
-	sig := cx.epc.WithStopSignal(debugLockSignal)
-	defer cx.epc.WithStopSignal(sig)
-
 	name := "a"
 
 	holder, ch, err := ctlV3Lock(cx, name)
@@ -135,7 +108,6 @@ func ctlV3Lock(cx ctlCtx, name string) (*expect.ExpectProcess, <-chan string, er
 		close(outc)
 		return proc, outc, err
 	}
-	proc.StopSignal = debugLockSignal
 	go func() {
 		s, xerr := proc.ExpectFunc(func(string) bool { return true })
 		if xerr != nil {