-
Notifications
You must be signed in to change notification settings - Fork 887
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
GODRIVER-2916 Test against updated FaaS scripts #1343
Conversation
@prestonvasquez I tried increasing the timeout to 600 seconds, but I am still failing with this error:
|
Note: I was able to start a remote cluster and connect to it with the local makefile. |
@@ -21,7 +21,7 @@ import ( | |||
"go.mongodb.org/mongo-driver/mongo/options" | |||
) | |||
|
|||
const timeout = 10 * time.Second | |||
const timeout = 600 * time.Second |
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.
Optional: consider using a smaller timeout. If there is an issue with selecting a server for an operation, it would take 10 minutes for the operation to timeout.
.evergreen/config.yml
Outdated
@@ -163,7 +163,7 @@ functions: | |||
# If this was a patch build, doing a fresh clone would not actually test the patch | |||
cp -R ${PROJECT_DIRECTORY}/ $DRIVERS_TOOLS | |||
else | |||
git clone https://github.com/mongodb-labs/drivers-evergreen-tools.git $DRIVERS_TOOLS | |||
git clone --branch DRIVERS-2384 https://github.com/blink1073/drivers-evergreen-tools.git $DRIVERS_TOOLS |
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.
Will this be updated once mongodb-labs/drivers-evergreen-tools PR #331 is merged ?
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.
Yes, I'll remove this and try a smaller timeout once that is merged.
API Change ReportNo changes found! |
GODRIVER-2916
Summary
Run against the updated scripts in mongodb-labs/drivers-evergreen-tools#331, which check for errors.
Background & Motivation
The script had been failing silently.