-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 offline flag to chectl server:start #14699
Comments
Moving to 7.3, hopefully we can get this done in time. WDYT @benoitf is this something you can do? |
I'm not sure to fully understand
where are the images ? it's a chectl command that is pulling them from tgz archives ? also as a side note, for now chectl has no visibility to all the images, it's inside helm and che-operator that the list is defined. Now the list will be stored as well inside chectl ? or it's dumb and take .tgz image with a given pattern name and just sent them so the list is not known. does that mean that we'll use |
@nickboldt @benoitf we can defer this to @benoitf images are pulled from the online registries (dockerhub and quay.io). The goal is to prepare Che to run in offline mode. We can maintain the list of images in chectl (the list should include plugin sidecars and stacks as well), I don't have a better idea. |
@tolusha we could consider to implement that same as crc use - use local dnsmasq server and configure minishift with our routing suffix and host. |
Do we have an ETA for this to be solved? It would be cool if it could be moved out of backlog and into a specific milestone, such as 7.12 or 7.13. |
7.13 is highly possible. |
Adding a line in /etc/hosts for the nip hostname? That is how I used to deal with it. |
@amisevsk |
@tolusha that makes a lot of sense -- it was something on my todo list at one point but I never got around to it. The reason we weren't/aren't pushing the images currently is due to licensing (can't redistribute vsixes downloaded from the VS Code marketplace, license issues in projects), but I think those are addressed now with the work to build problem vsixes in CI. My only concern with pushing offline registries is that (IIRC) some of the sample projects in the devfile registry still do not have licenses attached. Technically that means we have no rights to distribute those projects. |
@amisevsk |
@tolusha: For project sample licenses there's #14790; for the overall task of enabling offline registries, there's no current issue (but it would be a dozen line change in the CI scripts). Additionally, the offline image for the plugin reg can get very large due to old versions -- e.g. 1GB compressed for 7.8.0. There are a few options there (only cache latest plugins; remove old plugins from the registry altogether), but I don't think the approach is settled. |
Well, you could delete the old stuff from time to time. Why are we hoarding old versions we don't support? |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Is your enhancement related to a problem? Please describe.
We are making some changes to make it possible to run Che offline. But there is not chectl "offline" flag to automatically set this up.
Describe the solution you'd like
Add flag
--offline
to thechectl server:start
that should:imagePullPolicy
toNever
The deployment of Che should not be blocked while downloading the images: deployment and image pulling should happen in parallel.
If Che is already deployed the command should pull all the images and change the image pull policies (if needed).
Additional context
The images that need to be pulled are the default images for the given version of chectl (e.g. nightly/latest/7.x.y):
Subtasks
Minor tasks
The text was updated successfully, but these errors were encountered: