Skip to content
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

Update version of MongoDB used for testing to 7.0 #45160

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Dec 17, 2024

It is not the latest (8.0 was released in November) but it's probably the most common used and also it's consistent with the version used as a container.

Hopefully, this will be compatible with Ubuntu 22 and 24.

It is not the latest (8.0 was released in November) but it's probably
the most common used and also it's consistent with the version used as a
container.

Hopefully, this will be compatible with Ubuntu 22 and 24.
@gsmet
Copy link
Member Author

gsmet commented Dec 23, 2024

@loicmathieu this will be needed for the upcoming switch to Ubuntu 24. Unfortunately, things are not correctly stopped on Windows (or some other issues) which lead to CI stalling and being cancelled after 6 hours.

I think we are not the only one with the issue: flapdoodle-oss/de.flapdoodle.embed.mongo.spring#47 .

I'm not entirely sure why MongoDB 7 won't stop properly on Windows... but it seems something is off.

@loicmathieu
Copy link
Contributor

Flapdoodle on Windows has always been a pain, I remember having the same issue when update to a major version of Flapdoodle.
I wonder if we should move more tests to containers and only keep the test resource that uses Flapdoodl for cluster tests (with replicas).
As a workaround we may disable those tests on windows and create a followup issue to re-enable them later (we already did that for some months).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants