-
Notifications
You must be signed in to change notification settings - Fork 180
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
rocker/binder broken (jupyter
not in $PATH)
#739
Labels
bug
Something isn't working
Comments
@yuvipanda On second thought, unless we update all previous stack files, the next build will break all |
cboettig
added a commit
that referenced
this issue
Dec 29, 2023
I think this has been solved. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Container image name
rocker/binder:4.3
Container image digest
sha256:64f7b60a8d03f291b42e76d8250aaa525c2fe30e5a9af2217965a9dd90557a93
What operating system are you seeing the problem on?
macOS
System information
Server: Docker Engine - Community
Engine:
Version: 24.0.7
API version: 1.43 (minimum version 1.12)
Go version: go1.20.10
Git commit: 311b9ff
Built: Thu Oct 26 09:07:41 2023
OS/Arch: linux/amd64
Experimental: false
containerd:
Version: 1.6.25
GitCommit: d8f198a4ed8892c764191ef7b3b06d8a2eeb5c7f
runc:
Version: 1.1.10
GitCommit: v1.1.10-0-g18a0cb0
docker-init:
Version: 0.19.0
GitCommit: de40ad0
rootlesskit:
Version: 1.1.1
ApiVersion: 1.1.1
NetworkDriver: slirp4netns
PortDriver: builtin
StateDir: /tmp/rootlesskit3151596074
slirp4netns:
Version: 1.0.1
GitCommit: 6a7b16babc95b6a3056b33fb45b74a6f62262dd4
Bug description
Since #718 was merged,
jupyter
is now installed in/opt/venv
. However, theDockerfile
does not have the appropriatePATH
variable set, so therocker/binder
image no longer starts asjupyter
is no longer in$PATH
and it's the default entrypoitn.I'm not actually sure how to generate the
Dockerfile
s from the json, so not sure how to go about fixing it. I don't fully understand the JSON -> Dockerfile -> built images pipeline.How to reproduce this bug?
The text was updated successfully, but these errors were encountered: