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

Upgrade default etcd version to 3.5.17 #654

Merged
merged 2 commits into from
Jan 30, 2025

Conversation

GrahamCampbell
Copy link
Contributor

@GrahamCampbell GrahamCampbell commented Jan 17, 2025

This PR upgrades the default etcd version to 3.5.17.

Corresponding PR on vitess: vitessio/vitess#17653

@GuptaManan100
Copy link
Collaborator

The DCO check seems to be failing 😕. Did you add the -s flag while committing?

@GrahamCampbell
Copy link
Contributor Author

Made the commit from the GitHub webapp. 🤷

Copy link
Member

@frouioui frouioui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The etcd version should be changed everywhere both in vitess and vtop.

@frouioui
Copy link
Member

Have you tested upgrade paths with a vitess cluster on K8S using the previous etcd version, upgrading to the other etcd version? Same thing with downgrades.

@GrahamCampbell
Copy link
Contributor Author

Not tested. PR based on #447.

@frouioui
Copy link
Member

In this case we need to do #447 (comment) too :)

@frouioui frouioui added the WIP [Do not merge] Not ready for review or merging. label Jan 17, 2025
@GrahamCampbell
Copy link
Contributor Author

Re-pushed with a signed off commit.

@frouioui frouioui removed the WIP [Do not merge] Not ready for review or merging. label Jan 21, 2025
frouioui
frouioui previously approved these changes Jan 21, 2025
Copy link
Member

@frouioui frouioui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Can you modify the release notes summary for the next release (2.15.0) to include a note on how to proceed with this newer etcd version, #447 (comment).

Copy link
Member

@frouioui frouioui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's open a PR on vitess to make the etcd version match too please, we use v3.3.10 in the tests. We also use the ETCD_VER env variable in several places, in the vitess repo, which should be set to the same version as here.

Signed-off-by: Graham Campbell <[email protected]>
@frouioui frouioui merged commit 63f2ac7 into planetscale:main Jan 30, 2025
12 checks passed
@GrahamCampbell GrahamCampbell deleted the patch-1 branch January 30, 2025 17:48
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.

3 participants