-
Notifications
You must be signed in to change notification settings - Fork 807
Any other conditions are required of docker? #3
Comments
Did you build with the |
See also #2 which they required |
I confirm the same problem on MacOS. I solved the problem adding the Maybe this should be written on the README 👍 |
Yup, will do. Interestingly the original issue here was on Ubuntu, so I'm curious as to whether its the same problem. @Raxxor: does this fix it for you? |
Thanks for @samuelli & @LasaleFamine 's comments.
|
This patch does a few things: 1. Moves to Chrome Stable (since headless is now in 59). 2. Reduces the layer count in the docker file build. 3. Fixes lint errors in Dockerfile 4. Adds a non-privileged user called `botrender` to container (so you don't have to use `--unsafe-perm` with the npm install) 5. Expose 8080 for port binding and add documentation to allow running against service without `docker exec` 6. Document seccomp and SYS_ADMIN flags to reduce ECONNREFUSED confusion and use of `--disable-sandbox` as noted in #3
@justinribeiro has landed added some great recommendations to the readme which address this, so I'm closing this issue. Please file issues for more errors that you might be experiencing. @Raxxor: can you file separate issues for anything else you might be seeing? |
I'm on MacOS sierra and get I have done following commands:
|
ubuntu16.04 Docker version 17.03.1-ce, build c6d412e
When I tried
docker console output following logs
The text was updated successfully, but these errors were encountered: