Skip to content

Conversation

abr-egn
Copy link
Contributor

@abr-egn abr-egn commented Sep 10, 2024

RUST-2024

There were three independent problems happening here:

  • The serverless tests and serverless csfle tests were both trying to merge results into results.xml, and the second one would fail because the merge tool (appropriately) won't overwrite a file without an additional flag set. Fixed this by changing the test script to do a rolling merge, which also has the nice effect that our tests don't need to include output file names or an explicit merge step any more.
  • Many of the serverless tests were failing because CRYPT_SHARED_LIB_PATH wasn't being set; I tracked this down to a bug in the serverless setup scripts in drivers-evergreen-tools. I've sent RUST-2024 Fix CRYPT_SHARED_LIB_PATH expansion mongodb-labs/drivers-evergreen-tools#491 for that and in the meanwhile pointed our testing at my branch with the fix.
  • A few of the legacy tests were failing because of CLOUDP-267864; this just needed a test file sync to pick up the serverless skip for those tests.

@abr-egn abr-egn requested a review from kevinAlbs September 10, 2024 18:33
@abr-egn abr-egn marked this pull request as ready for review September 10, 2024 18:33
@abr-egn
Copy link
Contributor Author

abr-egn commented Sep 10, 2024

https://spruce.mongodb.com/version/66e06df35524ab00078b4b00/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC has a run with these fixes. The Ubuntu 22.04 (ARM64) failures are unrelated, those fail on main as well. I suspect a script breakage.


rm -rf $DRIVERS_TOOLS
git clone https://github.com/mongodb-labs/drivers-evergreen-tools.git $DRIVERS_TOOLS
#git clone https://github.com/mongodb-labs/drivers-evergreen-tools.git $DRIVERS_TOOLS
Copy link
Contributor

Choose a reason for hiding this comment

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

Can likely revert since mongodb-labs/drivers-evergreen-tools#491 was merged.

@abr-egn abr-egn merged commit 287985e into mongodb:main Sep 11, 2024
1 check was pending
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.

2 participants