-
Notifications
You must be signed in to change notification settings - Fork 362
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
Cannot use with Podman when short-name resolution is enforced #1146
Comments
Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗 |
(This cannot be fixed in |
Good point! The only people it'd affect are those who have a locally tagged @betatim @yuvipanda @minrk are you aware of any installations that would break if we use the full image name |
Nah let's do it! |
Non-default engines may require the full path including registry Closes jupyterhub#1146
Fixes jupyterhub#1146 Co-authored-by: Simon Li <orpheus+devel@gmail.com>
Fixes jupyterhub#1146 Co-authored-by: Simon Li <orpheus+devel@gmail.com>
Fixes jupyterhub#1146 Co-authored-by: Simon Li <orpheus+devel@gmail.com>
Bug description
When using the podman engine (with https://pypi.org/project/repo2podman/), an exception occurs when the base image is retrieved:
This is caused by this line:
repo2docker/repo2docker/buildpacks/base.py
Line 15 in 813d28b
Because the base image uses a short name (i.e. does not state what repository it is from), it will search the registries in the registries.conf file (see https://www.redhat.com/sysadmin/container-image-short-names ). Because the user cannot be prompted to select the registry to pull from, it fails (since short-name resolution is set to enforced).
Expected behaviour
jupyter-repo2docker --engine podman .
would launch a Jupyter notebook sucessfully.Actual behaviour
How to reproduce
python -m venv venv
. venv/bin/activate
pip install jupyter-repo2docker repo2podman
jupyter-repo2docker --engine podman .
Your personal set up
Possible Fixes
. Provide an option to state what registry to pull from (which will be inserted into the Dockerfile string)
. Change the FROM line to
FROM docker.io/buildpack-deps:bionic
The text was updated successfully, but these errors were encountered: