Skip to content
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

Add --debug & --no-build options #68

Merged
merged 4 commits into from
Sep 6, 2017

Conversation

yuvipanda
Copy link
Collaborator

@yuvipanda yuvipanda commented Aug 27, 2017

If --debug is passed, it'll print the Dockerfile generated.

Note that the Dockefile thus produced is not in any way usefu outside of debuggingl -
we do a bunch of path munging and adding scripts in here (such as install-miniconda).
If it isn't useful as is now, I don't think we should encourage people to use this to print
Dockerfiles for basing their Dockerfiles on from here, and should just write that as docs
instead.

Either way, this is still very useful for Debugging.

@choldgraf
Copy link
Member

lgtm - if you use it w/ no-build will it stop relatively quickly after printing the dockerfile? (or will it get buried quickly w/ a wall of log messages)

@yuvipanda
Copy link
Collaborator Author

well, it'll build however long it takes to build if you use it without --no-build. If you use it with --no-build then the build step is skipped :)

@yuvipanda
Copy link
Collaborator Author

I'm now owndering if we should even have --print-dockerfile here. None of these dockerfiles are actually going to build outside of repo2docker - we inject scripts, move the actual checkout into a 'src' directory, etc. We'll probably continue doing this kinda munging in the future :D

I might just make it print Dockerfile output if loglevel is DEBUG, and leave --no-build in there. I am convinced you shouldn't be using repo2docker to 'bootstrap' a copy of your Dockerfile

The Dockerfile we generate won't really work at all outside
of repo2docker. This makes it very clear that these are purely
for debugging purposes.
@yuvipanda yuvipanda changed the title Add --print-dockerfile & --no-build options Add --debug & --no-build options Aug 27, 2017
@choldgraf
Copy link
Member

sounds good - you don't imagine that the dockerfile contents will be useful for anything outside of debugging?

@arokem
Copy link
Contributor

arokem commented Aug 28, 2017

If repo2docker is called with a Dockerfile as input, should this generate an identical output? I would expect this to have some calls to jupyter to start up the notebook and so on. But that's not what I've gotten when I tried running this.

@yuvipanda
Copy link
Collaborator Author

yuvipanda commented Aug 28, 2017 via email

@arokem
Copy link
Contributor

arokem commented Aug 28, 2017

Thanks! In particular, it's not clear to me right now what starts jupyter here? Is repo2docker itself starting jupyter? How does this then integrate with z2jh? Do we need to install repo2docker on all the machines in the cluster?? Or would kubernetes run jupyterhub for us in the right configuration?

@yuvipanda
Copy link
Collaborator Author

@arokem kuberentes specifies the correct cmdline to start jupyter - it just requires that jupyterhub package is installed in the container image. There is no direct relation between z2jh and repo2docker - repo2docker is just an easy way to build a Docker image for use with z2jh if you don't have a Dockerfile. If you already have a Dockerfile, there is 0 advantage to using repo2docker with z2jh.

@yuvipanda yuvipanda merged commit 8db1d43 into jupyterhub:master Sep 6, 2017
@yuvipanda yuvipanda deleted the print-dockerfile branch September 6, 2017 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants