-
Notifications
You must be signed in to change notification settings - Fork 17
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
docker/docker-compose.yaml : Add fix for mongodb container on WSL #45
Conversation
Tip: If you add something like "Fixes #43" in the PR description it should automatically link it with the issue. |
Thanks for the tip. I will use that from now onwards :) |
I'm hitting this error:
Maybe it's just a problem with my local setup, I'll try again from a clean setup but still it would be interesting to understand why this is happening. |
OK I just had to remove an old stale container. For some reason it wasn't a problem with the volume using a bind-mounted directory. |
docker/docker-compose.yaml
Outdated
@@ -17,11 +17,11 @@ services: | |||
env_file: | |||
- ../.env | |||
|
|||
db: | |||
db1: |
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.
Why was this renamed to db1
? I think it should be OK to leave it as db
.
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 hit the same issue of ERROR: for db Cannot create container for service db: Conflict. The container name "/kernelci-db" is already in use
. So I tried to rename the service to db1 and check.
It was for testing only.
Yes, we don't need to rename the service name.
I renamed the service back to |
Signed-off-by: Jeny Sadadia <jeny.sadadia@gmail.com>
53a9267
to
92cc2a9
Compare
Okay. I pushed the changes with 'db' service name. |
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.
Tested OK locally using kernelci-pipeline.
Added docker volume for mongodb.
Signed-off-by: Jeny Sadadia jeny.sadadia@gmail.com