Skip to content

PHPLIB-911: Test with consistent version of crypt_shared #965

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

Merged
merged 1 commit into from
Sep 8, 2022

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Aug 31, 2022

https://jira.mongodb.org/browse/PHPLIB-911

Use the crypt_shared library provisioned by download-mongodb.sh and add an additional task to disable the library and fall back to mongocryptd.

This also revises how we pass environment variables into run-tests.sh and the test suite, and restructures run-tests.sh a bit.

Related to mongodb/mongo-php-driver#1353 for PHPC-2135.

Full patch build: https://spruce.mongodb.com/version/630faa3fe3c3315a7b34afd5/tasks

Use the crypt_shared library provisioned by download-mongodb.sh and add an additional task to disable the library and fall back to mongocryptd.

This also revises how we pass environment variables into run-tests.sh and the test suite, and restructures run-tests.sh a bit.
@jmikola jmikola changed the base branch from master to v1.13 September 7, 2022 15:34
@jmikola
Copy link
Member Author

jmikola commented Sep 8, 2022

Note: the two CI failures on latest are due to #967, which was merged to master. The spec tests in v1.13 have not been updated.

@jmikola jmikola merged commit 0fe1a0f into mongodb:v1.13 Sep 8, 2022
@jmikola jmikola deleted the phplib-911 branch September 8, 2022 00:54
@jmikola jmikola removed the request for review from alcaeus September 8, 2022 00:55
levon80999 pushed a commit to levon80999/mongo-php-library that referenced this pull request Sep 28, 2022
Use the crypt_shared library provisioned by download-mongodb.sh and add an additional task to disable the library and fall back to mongocryptd.

This also revises how we pass environment variables into run-tests.sh and the test suite, and restructures run-tests.sh a bit.
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.

1 participant