mirror of
https://github.com/etcd-io/etcd.git
synced 2024-09-27 06:25:44 +00:00
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.
This commit is contained in:
parent
033c0cbdd8
commit
5072530a80
@ -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 {
|
||||
|
@ -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 {
|
||||
|
Loading…
x
Reference in New Issue
Block a user