Skip to content

Conversation

nbbeeken
Copy link
Contributor

Adds the LOADBALANCED=ON setting to the create-instance.sh script usage.
Syncs the load-balancers specs and runs them in our serverless testing

There's just a few more assertions a debug helpers that came up while figuring out the issues with these tests. No driver changes were needed.

  • I added the EJSON helper to the utils.js file, it came in real handy figuring out which clients were failing to connect
  • I added a check to our run-checks.sh that will fail if you pushed changes without running generate_evergreen_tasks.js

@nbbeeken nbbeeken requested review from dariakp and durran October 13, 2021 16:40
@nbbeeken nbbeeken marked this pull request as ready for review October 13, 2021 16:40
@nbbeeken nbbeeken requested a review from dariakp October 14, 2021 18:02
Copy link
Contributor

@dariakp dariakp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for all the clean up!

@dariakp dariakp merged commit 644ac56 into 4.1 Oct 15, 2021
@dariakp dariakp deleted the NODE-3552/serverless-behind-lb branch October 15, 2021 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants