Skip to content

Docker build failing on Linux (Xubuntu 16.04) #10

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

Closed
jlpalomino opened this issue Jun 1, 2018 · 8 comments
Closed

Docker build failing on Linux (Xubuntu 16.04) #10

jlpalomino opened this issue Jun 1, 2018 · 8 comments
Assignees

Comments

@jlpalomino
Copy link
Member

The instructions do not say to create a new directory called earth-analytics-binder, but in order to cd to it, I created it and copied environment.yml and Dockerfile into this new folder.

When running the provided command (docker build -t earthlab/earth-analytics .), the build seems to fail when trying to download repodata.json.bz2 file from https://repo.anaconda.com/pkgs/main/linux-64/. I checked that the file exists, but it appears to have been updated today and may have an error.

@jlpalomino
Copy link
Member Author

After installing the http transport package for docker, the build was successful.

To run the docker image, the provided command (docker run -it -p 8888:8888 earthlab/earth-analytics) resulted in a URL that did not resolve. Instead, I used http://localhost:8888 and manually entered the token from the original URL. The image now runs successfully.

@lwasser
Copy link

lwasser commented Jun 5, 2018

oh @jlpalomino this is another update. the repo used to be called earth-analytics-binder. so if you cloned the repo you'd have that dir. just last week - we renamed the repo to earth-analytics-python-env so please update the instructions to cd to that dir which a user will have after cloning the repo!

@lwasser
Copy link

lwasser commented Jun 5, 2018

also it's interesting you needed the token and i didn't. can we try this again together so i can see please? also i think we renamed the docker image earth-analytics-python so there may be a few dated things here in the documentation. let's figure it out. Also @mbjoseph may be able to help test on his linux machine.

@lwasser
Copy link

lwasser commented Jun 5, 2018

when we sort it out you can submit a pr to update the docs!! i've assigned this to you.

@mbjoseph
Copy link
Member

mbjoseph commented Jun 6, 2018

Bah. As @jlpalomino noticed and @lwasser points out, out the docs are out of date ☹️

The instructions to build the image should be:

cd earth-analytics-python-env
docker build -t earthlab/earth-analytics-python-env .

I can confirm that the URL generated substitutes the container ID for localhost. I have no idea why this is. But if we can fix it, that would be great. I posted something on Twitter tagging the Jupyter team and Docker but all I got was crickets. We could probably open an issue here to get help: https://github.com/jupyter/docker-stacks

We can always check the build status on Docker Hub here if we need to: https://hub.docker.com/r/earthlab/earth-analytics-python-env/builds/

We also might want to add these badges to the README to point to the automated build and get the status:

Docker Automated build

^ in markdown is [![Docker Automated build](https://img.shields.io/docker/automated/earthlab/earth-analytics-python-env.svg)](https://hub.docker.com/r/earthlab/earth-analytics-python-env/)

Docker Build Status

^ is [![Docker Build Status](https://img.shields.io/docker/build/earthlab/earth-analytics-python-env.svg)](https://hub.docker.com/r/earthlab/earth-analytics-python-env/)

@lwasser
Copy link

lwasser commented Jun 6, 2018

🐐 baaaaaaaaah
thanks @mbjoseph i can ask Tim about this too :) i'm curious about how that all works and sometimes a key is really not necessary anyway. @jlpalomino let's update, add the badges and such as max suggests above! it will be a good exercise for you to get used to our GH workflow :) i find each place i work is slightly different so good to practice for everyone! :)

@jlpalomino
Copy link
Member Author

To resolve this issue about the install itself, I will create a pull request to update the README.md, per the response by Max.

I'll create new issues for the URL generated by Docker (i.e. URL generated substitutes the container ID for localhost) in both earthlab/earth-analytics-python-env and jupyter/docker-stacks.

@jlpalomino
Copy link
Member Author

No need for a new issue on jupyter/docker-stacks. I found an existing issue on jupyter/docker-stacks (jupyter/docker-stacks#642) that addresses this issue with the URL generated by Docker (i.e. URL generated substitutes the container ID for localhost), and it works.

docker run --hostname localhost -it -p 8888:8888 earthlab/earth-analytics-python-env

I'll create a new issue and pull request to update the README.md to update the command.

jlpalomino pushed a commit to jlpalomino/earth-analytics-python-env that referenced this issue Jun 6, 2018
These changes update the commands to build the docker image and add badges that point to the automated build and get the status. Fixes earthlab#10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants