-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
ImportError: No module named docker #9
Comments
Hi @TG9541, thanks for reporting, I will give a look, I fear they updated their install method again. I suggest you to pull the image I built 21 days ago: https://hub.docker.com/r/moul/readthedocs/ instead of rebuilding a new one |
Hi @moul, thanks, I'll try that. This sure is a moving target :-) |
@moul: I pulled the image above, changed the docker-compose.yml to use the image, and I still get the same error when I run the docker-compose line above. |
Ok perfect, No you are not supposed to add this file by yourself, this file used to exist in the readthedocs codebase, here is where I hardcode import it: https://github.com/moul/docker-readthedocs/blob/master/config/etc/default/rtd-config.py I just removed the deprecated import (automated docker build in progress) Can you give a new try ? |
Thanks, that fixed it :-) By the way, the docker-as-the-build-environment feature was what initially caught my interest. I'd have to pass-in the docker daemon socket (e.g. like |
@TG9541 happy I could help :) I didn't heard about this docker-as-the-build-environment feature before, Good luck ! |
I tried running docker-readthedocs, but it looks like a docker related dependency is missing. I'm new to RTD, and hence I can't easily say how far off the Dockerfile in moul/docker-readthedocs is from building successfully.
Commands to reproduce the problem:
Here is the "tail" of the build log with the error message:
The text was updated successfully, but these errors were encountered: