Skip to content
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

net/http: read: connection reset by peer under high load #20960

Open
liranp opened this issue Jul 9, 2017 · 19 comments
Open

net/http: read: connection reset by peer under high load #20960

liranp opened this issue Jul 9, 2017 · 19 comments
Labels
help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@liranp
Copy link

liranp commented Jul 9, 2017

What version of Go are you using (go version)?

go1.8.3

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOOS="linux"

What did you do?

  1. Edit /etc/sysctl.conf and reload with sysctl -p.
  2. Edit /etc/security/limits.conf.
  3. Execute the following server and client (on different hosts) to simulate multiple HTTP requests:
    Server https://play.golang.org/p/B3UCQ_4mWm
    Client https://play.golang.org/p/_XQRqp8K5b

What did you expect to see?

No errors at all.

What did you see instead?

Errors like the following:

Get http://<addr>:8080: read tcp [::1]:65244->[::1]:8080: read: connection reset by peer
Get http://<addr>:8080: write tcp [::1]:62575->[::1]:8080: write: broken pipe
Get http://<addr>:8080: dial tcp [::1]:8080: getsockopt: connection reset by peer
Get http://<addr>:8080: readLoopPeekFailLocked: read tcp [::1]:51466->[::1]:8080: read: connection reset by peer
@ALTree ALTree added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Jul 11, 2017
@ALTree ALTree added this to the Go1.10 milestone Jul 11, 2017
@rsc rsc modified the milestones: Go1.10, Go1.11 Nov 22, 2017
@teejays
Copy link

teejays commented Dec 1, 2017

I've been seeing similar issues with a similar set up where the server is on Go 1.6 while the client, which is making many concurrent requests, is on Go 1.9.2. Are we aware of what causes this? Seems like some kind of unintentional in-build rate limiting?

@bradfitz
Copy link
Contributor

bradfitz commented Dec 1, 2017

Go has no built-in rate limiting.

If you get broken network connections, they might be real (a peer really did disconnect), or they might be from hitting kernel limits on one of the sides.

@bradfitz
Copy link
Contributor

bradfitz commented Jul 9, 2018

Can anybody reproduce this and investigate?

@bradfitz bradfitz modified the milestones: Go1.11, Unplanned Jul 9, 2018
@GeorgeErickson
Copy link

Adding a comment to remind myself but:

I’ve seen similar issues with a should have time to investigate in the next month or so.

@dubbelpunt
Copy link

dubbelpunt commented Aug 24, 2018

In my opinion this issue occurs when the response is streamed into a dynamic allocated buffer.

No error is thrown when conn.Read() is send to a buffer with a fixed length.
Error is thrown sporadic when response is read via:
io.Copy(&buf, conn)
ioutil.ReadAll(conn)
conn.Read() streamed until EOF

Notice: even when the error is thrown, the response is buffered correct and complete.

GO Version I used: 1.10.3
OS: FreeBSD 11.2 (64 bit)

@drasko
Copy link

drasko commented Sep 17, 2018

I am also seeing this issue on http load testing.

Myabe I am hitting kernel tcp stack limits. @bradfitz any idea how to check if this is kernel issue?

@dxjones
Copy link

dxjones commented Nov 21, 2018

I am seeing this error frequently (read: connection reset by peer) when testing an HTTP 1.1 client that is generating heavy load on a server. I have not checked if the problem goes away with HTTP/2. (It could, since requests would be interleaved on a single connection).

I am running the client and server both on the same "localhost", and both are running a managed pool of "worker" goroutines in an attempt to limit the demand for resources.

The client and server each have 350+ goroutines (as reported by runtime.NumGoroutine())and are running as root on a Mac laptop with the max number of open files and sockets maxed out at 8192 (ulimit -n 8192).

I can run the same code on Ubuntu 18.04 LTS with more cores, memory, sockets, etc, and I eventually run into the same error message, although at a higher traffic load.

If there is a kernel limit, it would be nice to know what it is, so we could write our code so that it intentionally stays below that limit, rather than tweaking parameters and hoping it won't be a problem.

I see this topic was active a year ago, ... has there been any progress on this in the past year @bradfitz ?

@mcauto
Copy link

mcauto commented Nov 27, 2018

Can I retry the HTTP request in second?

@creker
Copy link

creker commented Dec 17, 2018

I also see this issue under macOS where server and client talk over localhost. I don't know the exact cause of the problem but running netstat during high load displays very large number of connections in TIME_WAIT state. Either I exhaust file descriptors or local ports.

Or both. I keep seeing connection reset by peer until the number of TIME_WAIT connections is about 7k. If I let the test run even further, I eventually get connect: can't assign requested address error which suggests that I exhausted local ports. The number of TIME_WAIT connections is at 15k.

For my project that's not needed and I could solve the problem by limiting number of concurrent connections. Solving the problem in my code by limiting number of goroutines seemed arbitrary and still occasionally failed. Limiting MaxConnsPerHost in http transport did the trick and problem went away completely.

@REPTILEHAUS
Copy link

Also hitting this issue. Im running 1000 concurrent connections at it eventually hits this also on OSX with file descriptors bumped up from default (256) to 4096. See a lot of stuff about this, possibly only an issue on OSX

@gaby
Copy link

gaby commented Feb 8, 2019

I'm also running into this issue with the latest release of Go. No matter what I try, I keep getting these "connection reset by peer". Netstat show's hundreds of connections in TIME_WAIT state, even after shutting down my client.

transport = &http.Transport{ MaxIdleConns: 1, MaxIdleConnsPerHost: 1, IdleConnTimeout: 5 * time.Second }

The problem still happens with a Max Idle Connections of 1, and using runtime.GOMAXPROCS(1). I also try calling transport.CloseIdleConnections() after each goroutine, but according to netstat they still show up as "TIME_WAIT".

@gaby
Copy link

gaby commented Feb 10, 2019

I manage to solve my TIME_WAIT problem using this article: http://tleyden.github.io/blog/2016/11/21/tuning-the-go-http-client-library-for-load-testing/

@creker
Copy link

creker commented Feb 10, 2019

@gabrielcalderon it doesn't fix the problem completely. Idle connections do not limit the number of connections Go could open. If you exhausted idle connections for any reason, Go will start opening new ones and cause TIME_WAIT problem again. What fixes the problem is MaxConnsPerHost knob. It limits the number of connections Go could open and blocks subsequent requests if it's exhausted. You want to put it equal or higher than idle connections.

@REPTILEHAUS
Copy link

Just to update. my issue was related to OSX and its defaults.. having tweakes the following settings I can now push 1000 concurrent transactions
sudo ulimit -n 6049
sudo sysctl -w kern.ipc.somaxconn=1024

@gaby
Copy link

gaby commented Feb 21, 2019

The only way I was able to fix it was to limit the number of concurrent transactions to be half the number of MaxIdleConns.

@pnsvinodkumar
Copy link

@REPTILEHAUS Thanks for that.. This worked well for me.. Don't think "ulimit" was an issue, as it was set to "unlimited" already.

Can someone give an insight into, what could be the side-effects, if I were to increase kern.ipc.somaxconn to an abnormally high number ?

@iwind
Copy link

iwind commented Aug 31, 2019

Just to update. my issue was related to OSX and its defaults.. having tweakes the following settings I can now push 1000 concurrent transactions
sudo ulimit -n 6049
sudo sysctl -w kern.ipc.somaxconn=1024

Works for me on Mac OS @REPTILEHAUS .

@gatspy
Copy link

gatspy commented Nov 25, 2021

have same issue

@AnubhavUjjawal
Copy link

Just to update. my issue was related to OSX and its defaults.. having tweakes the following settings I can now push 1000 concurrent transactions sudo ulimit -n 6049 sudo sysctl -w kern.ipc.somaxconn=1024

For me, after doing this change, the error changed to connect: resource temporarily unavailable

cunnie added a commit to cloudfoundry/bosh-dns-release that referenced this issue Feb 20, 2023
This commit prints out an error with a breadcrumb on macOS to tweak a
kernel setting (`kern.ipc.somaxconn`) to allow the test to pass.

Fixes:
```
  [FAILED] Unexpected error:
      <*net.OpError | 0xc0014887d0>: {
          Op: "read",
          Net: "tcp",
          Source: <*net.TCPAddr | 0xc0008929c0>{IP: [127, 0, 0, 1], Port: 57620, Zone: ""},
          Addr: <*net.TCPAddr | 0xc0008929f0>{IP: [127, 0, 0, 1], Port: 6587, Zone: ""},
          Err: <*os.SyscallError | 0xc00015e400>{
              Syscall: "read",
              Err: <syscall.Errno>0x36,
          },
      }
      read tcp 127.0.0.1:57620->127.0.0.1:6587: read: connection reset by peer
  occurred
  In [It] at: /Volumes/workspace/bosh-dns-release/src/bosh-dns/dns/main_test.go:1480 @ 02/19/23 17:10:05.539
```

Thanks <golang/go#20960 (comment)>
AliceInHunterland added a commit to nspcc-dev/neo-go that referenced this issue Mar 11, 2024
The default for `MaxConnsPerHost` is "no limit", so it'll use as many
connections as it can (potentially hitting MacOS/Windows limits
golang/go#20960 (comment) ).

Close #3300

Signed-off-by: Ekaterina Pavlova <ekt@morphbits.io>
AliceInHunterland added a commit to nspcc-dev/neo-go that referenced this issue Mar 11, 2024
The default for `MaxConnsPerHost` is "no limit", so it'll use as many
connections as it can (potentially hitting MacOS/Windows limits
golang/go#20960 (comment) ).

Close #3300

Signed-off-by: Ekaterina Pavlova <ekt@morphbits.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests