-
Notifications
You must be signed in to change notification settings - Fork 55
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 Docker Container Image for pulp-cli #714
Comments
I just want to add, that we have packages in Fedora (not necessarily the newest version in anything but rawhide). I can see the value for having a container with pulp-cli installed as the entrypoint. It could be a community addition. Implementation considerations: I would base the image on some |
Maybe ... As I said this is what I came up quickly basically Copy-Paste of my Template 🤣 . There might surely be other Ways to do it. In MOST cases I usually use the At least in 1 Case I indeed used Not sure for You can for sure optimize 😃. This was just "what I usually do". It doesn't mean it can't be done with less space, complexity, etc. I just like it because it works for most cases and with the Cache it also reduces Build Time/Image Size. I usually fall back to the same "Standard" Structure I use everywhere (which can be debated, but I'm pretty happy in how it works), which is an Entrypoint Script which is "Killable" plus an App Launch Script which will get Killed when the Container gets Killed. This is to avoid that 10 Seconds Timeout or so. Everybody has its Ways and Best Practices. I didn't use Multi-Stage (or Multi-Arch) Images yet, so of course that's another Improvement Point. I didn't learn how to do that yet though. There are many other People who are (much) more experienced than me with Containers. So, if there is a better Way (one can debate what "better" means eh 🤣), that one can also be used instead. Folder Structure is of course Debatable, whether to use The Virtual Environment I also do as Standard, since at least on recent Versions of Debian/Ubuntu,
Not a "State" per se provided you have mounted the Config Folder as a Bind-Mount Volume I guess 👍 . |
Yeah, there are at least as many ways to do things "right" in containers as there are people building containers. |
I have created my own and running it in the same Pod as multiprocess pulp image.
.bashrc contains autocomplete |
Is your feature request related to a problem? Please describe.
I'd like to have
pulp-cli
available as a Container Image.It seems to me (at least according to the Documentation) that it's only possible to installl
pulp-cli
from Source or frompip
(pypi
). There doesn't seem to be a Docker Container Image for it, at least, not that I could find. Andpulp-minimal
seems to contain severalpulp-*
Commands, but crucially NOTpulp
. So e.g.pulp status
will not work ...Describe the solution you'd like
The best would probably be that it's done as part of your "general"
pulp
Framework.I did a proof-of-concept but that is based on my Build Scripts (with a lot of commented lines to reduce Dependencies to a minimum) and the Docker Bases I usually use (Alpine and Debian), so it really looks (and is) completely different.
You can find the Repository here: https://github.com/luckylinux/docker-pulp-cli
Describe alternatives you've considered
Well this is the alternative to a Virtual Environment basically 😃.
Additional context
As said, probably better that somebody more familiar with the Container base used by
pulp-oci-images
implements this in "their framework", as my approach based on Alpine / Debian is probably too different.The proof of Concept (I created a
cli.toml
File with the base Configuration, as per Documentation) seemed to work at leastpulp status
returned the expected Result.The text was updated successfully, but these errors were encountered: