kubeadm manifests should use multi-arch images

Created on 2 Aug 2018  Â·  8Comments  Â·  Source: kubernetes/kubeadm

Just like we started using the pause image with fat manifest, we should switch to using all the kubernetes components to use images without say the -amd64 in their name. Starting with the items below:

https://github.com/kubernetes/kubernetes/blob/master/cmd/kubeadm/app/images/images.go#L70-L73

TODO(@neolit123): this page needs a bump, possibly others too:
https://kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm-init/

areecosystem kindocumentation kinfeature lifecyclactive prioritimportant-longterm

Most helpful comment

I'll prepare a PR for this tomorrow.

All 8 comments

these end up calling GetGenericArchImage() which was a recent addition and it uses runtime.GOARCH.
we do have manifest lists nowadays in the gcr bucket, but do we have them for everything?

@rosti WDYT?

I'll prepare a PR for this tomorrow.

@neolit123 if we find something missing then we fix it :)

/lifecycle active

^ testing new labels :sunglasses:

edit: hmm, halp doesn't work.
adding manually.

I don't think anyone has deployed a new prow recently.

On Thu, Aug 2, 2018, 18:20 Lubomir I. Ivanov notifications@github.com
wrote:

/lifecycle active

^ testing new labels 😎

edit: hmm, halp doesn't work. @spiffxp https://github.com/spiffxp
@BenTheElder https://github.com/BenTheElder
adding manually.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubeadm/issues/1030#issuecomment-410114729,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4Bq6GuPAc_SkB6AG9k6F8FOH2GaG6bks5uM6V3gaJpZM4VrkLE
.

got it, thanks.

Closing this in favor for the oldest open issue we have atm: #51 _sigh_
Sooooon we're there :sunglasses: :tada:

Was this page helpful?
0 / 5 - 0 ratings

Related issues

ep4eg picture ep4eg  Â·  3Comments

jbrandes picture jbrandes  Â·  4Comments

atoato88 picture atoato88  Â·  4Comments

bruceauyeung picture bruceauyeung  Â·  4Comments

ggee picture ggee  Â·  4Comments