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

/bin/sh default when using extraContainers: #3465

Open
BuckinghamIO opened this issue Jul 26, 2024 · 2 comments
Open

/bin/sh default when using extraContainers: #3465

BuckinghamIO opened this issue Jul 26, 2024 · 2 comments
Labels

Comments

@BuckinghamIO
Copy link

Bug description

When adding an extra containers for the single user environment the image-puller has a hardcoded command that it trys to execute to feedback if the process has been completed.

command:
  - /bin/sh
  - -c
  - echo "Pulling complete"

A issue arrises where when a extra container may not have bash and this causes an error preventing the startup of a container.

How to reproduce

Add any container that does not have /bin/bash to the singleUser environment, in my experience I was trying to add cloud-sql-proxy.

Expected behaviour

The image should run as expected and not be prevented by a JupyterHub requirement that all extra containers should have bash installed.

Actual behaviour

Error within the image-puller daemon set.

Error: failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unab │
│ le to start container process: exec: "/bin/sh": stat /bin/sh: no such file or directory: unknown

@BuckinghamIO BuckinghamIO changed the title /bin/bash default when using extraContainers: /bin/sh default when using extraContainers: Jul 26, 2024
@manics manics transferred this issue from jupyterhub/jupyterhub Jul 26, 2024
@manics
Copy link
Member

manics commented Jul 26, 2024

Running a minimal command means we can pre-pull an image by running it as a daemonset.

Do you have a suggestion for a better way to pre-pull images? Would you be happy if the command was configurable for each pulled image?

@BuckinghamIO
Copy link
Author

Well the issue with this command or even I guess the requirement for a command is some images are distroless which this clearly wont work with.

A configurable command may allow more options for sure, although Im not sure in my case it would be a solution either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants