Skip to content
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

clustermesh: update etcd version to v3.5.4 #1417

Closed
wants to merge 0 commits into from

Conversation

kahirokunn
Copy link
Contributor

@kahirokunn kahirokunn commented Feb 26, 2023

With helm, etcd uses v3.5.4.
There is no reason to separate cilium-cli, so we will match them.

https://github.com/cilium/cilium/blob/e8104547227af8fa53ed197e13f5879957d8297c/install/kubernetes/cilium/values.yaml#L2288

The container image in v3.5.4 of etcd is multi-platform.
I am using EKS, which is a managed control plane, so I am running on amd64.

Therefore, when using EKS, there are many configurations where the control plane is amd64, but the node runs on arm64.
Currently, the -arm64 suffix of etcd only refers to the architecture of the control plane, so it does not cover many usecases.
However, upgrading etcd will solve all the problems.

Server Version: version.Info{Major:"1", Minor:"25+", GitVersion:"v1.25.6-eks-48e63af", GitCommit:"9f22d4ae876173884749c0701f01340879ab3f95", GitTreeState:"clean", BuildDate:"2023-01-24T19:19:02Z", GoVersion:"go1.19.5", Compiler:"gc", Platform:"linux/amd64"}

https://quay.io/repository/coreos/etcd?tab=tags&tag=latest

@tklauser tklauser changed the title ci: update etcd version to v3.5.4 clustermesh: update etcd version to v3.5.4 Feb 28, 2023
@tklauser tklauser closed this Feb 28, 2023
@tklauser tklauser temporarily deployed to ci February 28, 2023 12:48 — with GitHub Actions Inactive
@tklauser
Copy link
Member

Hi @kahirokunn. Sorry, for some reason I messed up merging this PR and the branch now seems empty. Would you mind pushing your change again?

Sorry for the inconvenience!

@kahirokunn
Copy link
Contributor Author

@tklauser #1421
👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants