-
Notifications
You must be signed in to change notification settings - Fork 885
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-2492 use consistent versions of crypt_shared and server #1021
Conversation
this matches behavior of run-orchestration.sh
get_distro uses . /etc/os-release On some hosts, that overwrites the VERSION environment variable.
This reverts commit 2810078.
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.
Looks good with a few suggestions and questions 👍
. $DRIVERS_TOOLS/.evergreen/download-mongodb.sh | ||
get_distro | ||
get_mongodb_download_url_for "$DISTRO" "$MONGODB_VERSION" | ||
if [ -z "$MONGO_CRYPT_SHARED_DOWNLOAD_URL" ]; then |
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: Add a note about where $MONGO_CRYPT_SHARED_DOWNLOAD_URL
and $EXTRACT
come from. I had to do some searching to figure out that get_mongodb_download_url_for
sets those env vars.
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.
Good thinking. Done.
echo "Downloading crypt_shared package from $MONGO_CRYPT_SHARED_DOWNLOAD_URL" | ||
download_and_extract_crypt_shared "$MONGO_CRYPT_SHARED_DOWNLOAD_URL" "$EXTRACT" | ||
touch expansion.yml | ||
fi |
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.
Nice find with expansions.update
! Should we move the find
command from run-tests.sh
here and add CRYPT_SHARED_LIB_PATH
to the expansions instead of setting SKIP_CRYPT_SHARED_LIB_DOWNLOAD
if we didn't download it?
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.
Good idea. I have updated to move the find
command here and set a CRYPT_SHARED_LIB_PATH
expansion.
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.
LGTM!
The expansion is set on the task function, not the variant.
This reverts commit 4e445a6.
Summary
Background & Motivation
This PR validates changes proposed in mongodb-labs/drivers-evergreen-tools#213.
Queryable Encryption is in Technical Preview and is subject to backwards breaking changes. If backwards breaking changes are made, using different version of the server and crypt_shared may complicate diagnosing driver test failures.