Investigate/suppress errors in #reflector-logs #25

New issue

Closed

opened 2019-03-08 23:23:35 +01:00 by tzarebczan · 5 comments

tzarebczan commented

2019-03-08 23:23:35 +01:00

(Migrated from github.com)

#reflector-logs is full of errors like below. If we know they aren't causing any issues (I know we thought they might be at one point), we should suppress them.

net.OpError write tcp4 172.31.89.3:5567->86.161.0.74:39276: write: connection reset by peer
/home/travis/gopath/pkg/mod/github.com/lbryio/lbry.go@v0.0.0-20190109223729-30c312501602/extras/errors/errors.go:98 (0x526d75)
/home/travis/gopath/src/github.com/lbryio/reflector.go/peer/server.go:302 (0xb4691c)
/home/travis/gopath/src/github.com/lbryio/reflector.go/peer/server.go:164 (0xb45d85)
/home/travis/gopath/src/github.com/lbryio/reflector.go/peer/server.go:99

#reflector-logs is full of errors like below. If we know they aren't causing any issues (I know we thought they might be at one point), we should suppress them. ``` net.OpError write tcp4 172.31.89.3:5567->86.161.0.74:39276: write: connection reset by peer /home/travis/gopath/pkg/mod/github.com/lbryio/lbry.go@v0.0.0-20190109223729-30c312501602/extras/errors/errors.go:98 (0x526d75) /home/travis/gopath/src/github.com/lbryio/reflector.go/peer/server.go:302 (0xb4691c) /home/travis/gopath/src/github.com/lbryio/reflector.go/peer/server.go:164 (0xb45d85) /home/travis/gopath/src/github.com/lbryio/reflector.go/peer/server.go:99 ```

lyoshenka commented

2019-03-14 18:24:34 +01:00

(Migrated from github.com)

@jackrobison still waiting for details from you

lyoshenka commented

2019-03-14 18:24:52 +01:00

(Migrated from github.com)

assign it back to me after

jackrobison commented

2019-03-15 14:59:45 +01:00

(Migrated from github.com)

@lyoshenka this seemed to be caused by something not being handled correctly when a connection is closed abruptly (while bytes are being written). It seems to be initiated during a download of a stream by the client downloading from reflector plus another fast peer, where the peer the blob downloads from oscillates between the two (and thus has a lot of cancellations mid-transfer). Another condition that may be required is where the stream is only partially uploaded to reflector (and had a similar mid-blob-upload cancellation)

jackrobison commented

2019-03-15 15:33:23 +01:00

(Migrated from github.com)

A possible issue reproducing locally is that the local reflector.go blob server will reply (at least for me) with frames up to 262144 bytes long vs < 20000 bytes for the real one.

lyoshenka commented

2019-03-18 21:30:48 +01:00

(Migrated from github.com)

The correct answer seems to be to just ignore these. They happen when the other side closes the connection without letting us know (because it has no way to let us know). Fixed in 7a618f4.

The correct answer seems to be to just ignore these. They happen when the other side closes the connection without letting us know (because it has no way to let us know). Fixed in 7a618f4.

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/reflector.go#25

No description provided.

Rows
Columns