-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[Proposal] Use kube_image_repo as default for etcd when using etcd_kubeadm_enabled #7025
Comments
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
/remove-lifecycle stale |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
/remove-lifecycle rotten |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
/remove-lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close |
@k8s-triage-robot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Environment:
Cloud provider or hardware configuration: Arm64
OS (
printf "$(uname -srm)\n$(cat /etc/os-release)\n"
): Ubuntu 20.04Kubespray version (commit) (
git rev-parse --short HEAD
): 9c1e082Network plugin used: Calico
I would like to propose we use the k8s.gcr.io(kube_image_repo) repo as default for etcd. By default Google's image of etcd has multi-arch support, and will pass
ETCD_UNSUPPORTED_IMAGE
environment variable on arm/unsupported platforms. The efforts for finding this was inspired from #6921.I would like to create a PR for this, but would like some input from the maintainers first.
Google's/Kubernetes image tag for etcd is
3.4.13-0
regardless of the architecture isk8s.gcr.io/3.4.13-0
compared tov3.4.14
v3.4.14-arm64
respectively. Some logic would need to be created if the kube_image_repo is in use. Would we be okay doing something like this?Currently for testing purposes I have set in
roles/download/defaults/main.yml
On top of this
kubespray/roles/kubernetes/master/templates/kubeadm-config.v1beta2.yaml.j2
Line 67 in aeaa876
etcd logs show:
Since it seems it's a requirement to pass
ETCD_UNSUPPORTED_ARCH
as a environment variable instead of a flag even using the correct syntaxETCD_UNSUPPORTED_ARCH=arm64
likely won't succeed. Regardless we have set currentlykubespray/roles/kubernetes/master/templates/kubeadm-config.v1beta2.yaml.j2
Line 68 in aeaa876
It's likely we can't even use this or need it
In my opinion etcd should be passing this environment variable by default in there images or at least have a binary flag to support it. Currently an upstream issue is opened etcd-io/etcd#12543.
It seems it's not possible to pass
ETCD_UNSUPPORTED_ARCH
as a binary flag https://github.com/etcd-io/etcd/blob/fdb3f897308be59bd132fcf2585d46108a778a5a/Documentation/op-guide/supported-platform.md#32-bit-and-other-unsupported-systemsConsidering for the control plane we're using kube_image_repo for images would it make sense to default etcd to it?
The text was updated successfully, but these errors were encountered: