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

Images missing rm #301

Closed
chris-ricketts opened this issue Dec 5, 2024 · 3 comments · Fixed by #307
Closed

Images missing rm #301

chris-ricketts opened this issue Dec 5, 2024 · 3 comments · Fixed by #307
Assignees

Comments

@chris-ricketts
Copy link

I found it surprising that the docker images do not have rm

$ docker run --rm -it ghcr.io/strangelove-ventures/heighliner/gaia:v21.0.0 /bin/sh -c rm
/bin/sh: rm: not found

I discovered this when trying to use the images with neutron-org/cosmopark.

This tool tries to execute rm within the container here.

@agouin
Copy link
Member

agouin commented Dec 5, 2024

Heighliner images specifically only have read only utilities for security.

This is a common enough ask though that we may want to consider adding useful write utils such as rm, mv, mkdir, a text editor such as vi, etc.

@vimystic
Copy link
Contributor

vimystic commented Dec 20, 2024

Good first issue for @Zak4sad . In addition to adding aria2 into the infra-toolkit (which is used by heighliners Docker file)

@Zak4sad
Copy link
Contributor

Zak4sad commented Jan 9, 2025

@vimystic , @agouin , The PR is already merged here #307 , shall I mark this issue as done ?
ps: aria2c will be done in a separate PR strangelove-ventures/infra-toolkit#18

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 a pull request may close this issue.

6 participants