Skip to content

CXX-2783 use Docker image to run mongohouse #1085

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 7 commits into from
Jan 22, 2024

Conversation

kevinAlbs
Copy link
Collaborator

Summary

Similar to mongodb/mongo-c-driver#1517, this PR uses the mongohouse Docker image added in DRIVERS-2543 to fix test_mongohouse task.

Verified with this patch build: https://spruce.mongodb.com/version/65aeae81c9ec44ca5f410888

Background & Motivation

DRIVERS-2543 adds scripts to use the Docker image maintained by the Atlas Data Lake team. mongodb-labs/drivers-evergreen-tools@95ad5f3 removed run-mongohouse-local.sh from drivers-evergreen-tools.

A new RUN_MONGOHOUSE_TESTS environment variable is passed to the test runner to conditionally run the serverStatus command. mongohoused does not return an expected version field in the response to serverStatus:

AtlasDataFederation test> db.runCommand({serverStatus: 1})
{ ok: 1, host: 'localhost' }

The test mongohoused uses a local store. I expect this does not requiring a running mongod or mongos. I expect the version check is not applicable.

To resolve error:
```
.evergreen/atlas_data_lake/pull-mongohouse-image.sh: 7: Bad substitution
```
`mongohoused` responds with `{ "ok" : 1, "host" : "localhost" }`.
@kevinAlbs kevinAlbs marked this pull request as ready for review January 22, 2024 18:58
@kevinAlbs kevinAlbs requested a review from eramongodb January 22, 2024 18:58
@kevinAlbs kevinAlbs merged commit f9f48a1 into mongodb:master Jan 22, 2024
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