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
feat: Pull MongoDB and Redis from shared-services repo #10266
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: John Gomersall <thegoms@gmail.com>
Signed-off-by: John Gomersall <thegoms@gmail.com>
Signed-off-by: John Gomersall <thegoms@gmail.com>
Signed-off-by: John Gomersall <thegoms@gmail.com>
Signed-off-by: John Gomersall <thegoms@gmail.com>
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.
I am a bit sceptical about duplicating so much to avoid pulling deps.
Why not have instead "test_deps" and "deps" in the Makefile to avoid pulling too much, still keeping things DRY and consistent ?
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.
Hi @alexgarel , I've tweaked it so the deps are cloned for tests but not run so that we can reference the docker yml in the test targets. Let me know what you think. Note there are changes to shared-services that go with this, but clone deps should do a pull anyway
Signed-off-by: John Gomersall <thegoms@gmail.com>
Signed-off-by: John Gomersall <thegoms@gmail.com>
Quality Gate passedIssues Measures |
What
Create a test.yml for integration tests but otherwise MongoDB and Redis come from the shared-services repo.
https://github.com/openfoodfacts/openfoodfacts-shared-services