-
Notifications
You must be signed in to change notification settings - Fork 716
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
Document how to pre-pull images #34
Comments
From @errordeveloper on October 24, 2016 11:18 @bulletRush thank you very much for this proposal. We have considered this already, and we will very likely implement it soon. |
From @bulletRush on October 24, 2016 14:25 @errordeveloper, may i do something for this feature? |
@bulletRush can you send a documentation patch how to pre-pull the images? |
This documentation need at least one request: change image pull policy from default(which is always) to IfNotPresent. Second, kubeadm need a new flag(e.g. My refused pr is here: [kubeadm] specify an alternate location for all images and pre pull them. I still think a support for pre pull directly is more friendly for newbee. |
and what's |
I need this kind of thing too. |
See: #101 |
hope the command |
I think that it would be useful to know what images the new config needs, not the current one. The process of the software update is as follows:
|
just to clarify; has this feature been implemented and is awaiting documentation? or it has not been implemented? what's the workflow to pre-pull an image? if not, do you need help to move this along? |
Seems like the work items here are:
Is that a fair assessment @bulletRush ? |
Have been the case since v1.5 at least, probably earlier as well
There is docs on what master images can be used per branch already
We decided to document on the site instead of building it into kubeadm @jamiehannaford Fair enough? |
@luxas Sounds good, didn't realise all this was already there 😄 If that's the case, does this issue need to stay open? |
I think it can be closed |
is there no kube command to pre-pull images? must it be done manually (ssh to node and |
@ilackarms Perfectly right. kubeadm doesn't care about which CRI runtime to use. |
@luxas maybe for that reason there should be a |
@ilackarms That is a sig-node feature in that case; to expose a "Pull API" in the Kubelet API (a quite big one). However, such a thing doesn't exist right now, so there is no client like kubeadm or kubectl that can consume it. |
understood @luxas. probably we would want to bubble up that api from the kubelet through the apiserver so a kube client can request that an image be pre-pulled (e.g. to eliminate cold start times) |
+1 for a pre-pull api. In the mean time, I've usually hacked around the lack by using a daemonset with the containers to be kept hot with the container command overridden to be: command: [/bin/sh,-c,'while true; do sleep 1000; done'] |
From @bulletRush on October 21, 2016 7:50
Is this a request for help? (If yes, you should use our troubleshooting guide and community support channels, see http://kubernetes.io/docs/troubleshooting/.):
No.
**What keywords did you search in Kubernetes issues before filing this one?:
kubeadm
,kubeadm pull
,kubeadm image
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
Kubernetes version (use
kubectl version
):Client Version: version.Info{Major:"1", Minor:"4+", GitVersion:"v1.4.0-beta.8", GitCommit:"3040f87c570a772ce94349b379f41f329494a4f7", GitTreeState:"clean", BuildDate:"2016-09-18T21:06:37Z", GoVersion:"go1.6.3", Compiler:"gc", Platform:"linux/amd64"}
Environment:
uname -a
): Linux 130 3.10.0-327.36.1.el7.x86_64 kubeadm join on slave node fails preflight checks #1 SMP Sun Sep 18 13:04:29 UTC 2016 x86_64 x86_64 x86_64 GNU/LinuxHi, I have implement pre pull image and configurable pods.
so you can:
To use this feature, you can run kubeadm in this command
./kubeadm init --config=config.json
. a short sample aboutconfig.json
:when you run kubeadm by
./kubeadm --config=config.json
, the output will show you the image pull process:If you config
kube-apiserver
,kube-controller-manager
in this config file, the static pod manifests genereted by kubeadm will use podSpec definition in this file.but I still have some confuse about my implementation:
Copied from original issue: kubernetes/kubernetes#35264
The text was updated successfully, but these errors were encountered: