New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

etcd crashed by a panic #4171

Closed
jbking opened this Issue Jan 8, 2016 · 3 comments

Comments

4 participants
@jbking

jbking commented Jan 8, 2016

The problem is introduced by this.

1f97f2d#diff-8663bb0f53797ae80ef14f3e44668a55

We use a7287b6

How to reproduce

  1. build etcd.
  2. execute the etcd.
  3. execute a command with etcdctl.
$ etcdctl --peers "http://127.0.0.1:4001" ls
panic: interface conversion: net.Conn is *netutil.limitListenerConn, not *net.TCPConn

goroutine 69 [running]:
github.com/coreos/etcd/pkg/transport.(*keepaliveListener).Accept(0xc20810d0e0, 0x0, 0x0, 0x0, 0x0)
        /go/src/github.com/coreos/etcd/pkg/transport/keepalive_listener.go:53 +0xe3
net/http.(*Server).Serve(0xc208043a40, 0x7f5f675ab788, 0xc20810d0e0, 0x0, 0x0)
        /usr/local/go/src/net/http/server.go:1728 +0x92
github.com/coreos/etcd/etcdmain.serveHTTP(0x7f5f675ab788, 0xc20810d0e0, 0x7f5f675ac898, 0xc20812b920, 0x0, 0x0, 0x0)
        /go/src/github.com/coreos/etcd/etcdmain/http.go:39 +0x1be
github.com/coreos/etcd/etcdmain.func·006(0x7f5f675ab788, 0xc20810d0e0)
        /go/src/github.com/coreos/etcd/etcdmain/etcd.go:326 +0x80
created by github.com/coreos/etcd/etcdmain.startEtcd
        /go/src/github.com/coreos/etcd/etcdmain/etcd.go:327 +0x25a8

(snip)
@gyuho

This comment has been minimized.

Show comment
Hide comment
@gyuho

gyuho Jan 8, 2016

Member

Yes, this is a bug. Confirmed that I see the same error message. Thanks for reporting. We will investigate.

Member

gyuho commented Jan 8, 2016

Yes, this is a bug. Confirmed that I see the same error message. Thanks for reporting. We will investigate.

@xiang90

This comment has been minimized.

Show comment
Hide comment
@xiang90

xiang90 Jan 8, 2016

Contributor

@jbking It should be fixed. Please give it a try.

Contributor

xiang90 commented Jan 8, 2016

@jbking It should be fixed. Please give it a try.

@jbking

This comment has been minimized.

Show comment
Hide comment
@jbking

jbking Jan 8, 2016

It works. Thanks.

jbking commented Jan 8, 2016

It works. Thanks.

heyitsanthony added a commit to heyitsanthony/etcd that referenced this issue Jan 12, 2016

integration: etcd end-to-end tests
Uses gexpect to test the etcd binary directly. Tests etcd-io#4135, etcd-io#4171

heyitsanthony added a commit to heyitsanthony/etcd that referenced this issue Jan 13, 2016

integration: etcd end-to-end tests
Uses gexpect to test the etcd binary directly. Tests etcd-io#4135, etcd-io#4171

heyitsanthony added a commit to heyitsanthony/etcd that referenced this issue Jan 13, 2016

e2e: etcd end-to-end tests
Uses gexpect to test the etcd binary directly. Tests etcd-io#4135, etcd-io#4171

heyitsanthony added a commit to heyitsanthony/etcd that referenced this issue Jan 13, 2016

e2e: etcd end-to-end tests
Uses gexpect to test the etcd binary directly. Tests etcd-io#4135, etcd-io#4171
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment