-
Notifications
You must be signed in to change notification settings - Fork 45
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
Upgrade mongo driver to latest #52
Upgrade mongo driver to latest #52
Conversation
21ec17b
to
ceb7215
Compare
b3c7687
to
0c887f0
Compare
0c887f0
to
d271e6c
Compare
- image: circleci/golang:1.14 | ||
- image: circleci/mongo:4.2 | ||
- image: cimg/go:1.18 | ||
- image: circleci/mongo:5.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We were also behind on golang and mongo so I figured I'd bump them both "while I'm in there"
steps: | ||
- checkout | ||
|
||
- restore_cache: | ||
keys: | ||
- v1-pkg-cache | ||
|
||
- run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/fc0a898a6ae297c0ef59e2f1d824713d6f1cd222/install.sh | sh -s -- -d -b $(go env GOPATH)/bin v1.27.0 | ||
- run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/a31485fe906612b2884239fdf7c55aec76ea7596/install.sh | sh -s -- -d -b $(go env GOPATH)/bin v1.50.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The old version no longer worked and was consistently failing on CI so this needed to be upgraded as well
@@ -20,6 +20,10 @@ func (m *mockServer) MinRTT() time.Duration { | |||
return time.Duration(0) | |||
} | |||
|
|||
func (m *mockServer) RTT90() time.Duration { | |||
return time.Duration(0) | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the new version of the driver the Server
interface added this RTT90
function that needed to be added to appease the type system
In preparation for some incoming bugfixes to the golang driver (see https://jira.mongodb.org/browse/GODRIVER-2620) this PR upgrades the mongo driver to the latest version.