Skip to content

Commit

Permalink
Include the patch from Issue 255.
Browse files Browse the repository at this point in the history
This brings in a test which fails without the patch, and passes with the
patch. Still to be tested, manual tcpkill of a socket.
  • Loading branch information
jameinel committed Feb 15, 2018
1 parent dc846a7 commit ce5938e
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 8 deletions.
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
_harness

1 change: 1 addition & 0 deletions cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -682,6 +682,7 @@ func (cluster *mongoCluster) AcquireSocket(mode Mode, slaveOk bool, syncTimeout
time.Sleep(100 * time.Millisecond)
continue
} else {
// We've managed to successfully reconnect to the master, we are no longer abnormaly ended
server.Lock()
server.abended = false
server.Unlock()
Expand Down
2 changes: 1 addition & 1 deletion export_test.go
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package mgo

import (
"time"
"net"
"time"
)

func HackPingDelay(newDelay time.Duration) (restore func()) {
Expand Down
2 changes: 1 addition & 1 deletion harness/setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ start() {
UP=$(svstat daemons/* | grep ' up ' | grep -v ' [0-3] seconds' | wc -l)
echo "$UP processes up..."
if [ x$COUNT = x$UP ]; then
echo "Running setup.js with mongo..."
echo "Running init.js with mongo..."
mongo --nodb ../harness/mongojs/init.js
exit 0
fi
Expand Down
13 changes: 7 additions & 6 deletions server_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ import (
"time"

. "gopkg.in/check.v1"
"gopkg.in/mgo.v2"
"github.com/globalsign/mgo"
)

func (s *S) TestServerRecoversFromAbend(c *C) {
Expand All @@ -42,22 +42,23 @@ func (s *S) TestServerRecoversFromAbend(c *C) {
server := cluster.Server("127.0.0.1:40001")
sock, abended, err := server.AcquireSocket(100, time.Second)
c.Assert(err, IsNil)
c.Check(abended, Equals, false)
c.Assert(sock, NotNil)
sock.Close()
sock.Release()
c.Check(abended, Equals, false)
// Forcefully abend this socket
sock.Close()
server.AbendSocket(sock)
// Next acquire notices the connection was abnormally ended
sock, abended, err = server.AcquireSocket(100, time.Second)
c.Assert(err, IsNil)
sock.Release()
c.Check(abended, Equals, true)
sock.Close()
// cluster.AcquireSocket should fix the abended problems
sock, err = cluster.AcquireSocket(mgo.Primary, false, time.Minute, time.Second, nil, 100)
c.Assert(err, IsNil)
sock.Close()
sock.Release()
sock, abended, err = server.AcquireSocket(100, time.Second)
c.Assert(err, IsNil)
c.Check(abended, Equals, false)
sock.Close()
sock.Release()
}

0 comments on commit ce5938e

Please sign in to comment.