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 main to go1.18.1 #10101

Merged
merged 1 commit into from
Apr 16, 2022
Merged

Conversation

frouioui
Copy link
Member

Description

This Pull Request upgrades the golang version used on main to go1.18.1. The Go team has recently released a new patch release for go1.18 that contains vulnerability fixes, that impact Vitess.

Snippet from the Go release history website:

go1.18.1 (released 2022-04-12) includes security fixes to the crypto/elliptic, crypto/x509, and encoding/pem packages, as well as bug fixes to the compiler, linker, runtime, the go command, vet, and the bytes, crypto/x509, and go/types packages. See the Go 1.18.1 milestone on our issue tracker for details.

Note that the minimum required Go version is 1.18.1, if this requirement is not met, the following error will appear when building Vitess:

Version 1.18.1+ required. See https://vitess.io/contributing/build-from-source for install instructions.

Checklist

  • "Backport me!" label has been added if this change should be backported
  • Tests were added or are not required
  • Documentation was added or is not required

@@ -33,17 +33,17 @@ func TestVersionString(t *testing.T) {
buildTimePretty: "time is now",
buildGitRev: "d54b87ca0be09b678bb4490060e8f23f890ddb92",
buildGitBranch: "gitBranch",
goVersion: "1.18",
goVersion: "1.18.1",
goOS: "amiga",
Copy link
Collaborator

Choose a reason for hiding this comment

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

goOS: "amiga" 😆

Copy link
Member

@deepthi deepthi 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 change bootstrap_version to 6, and upload new bootstrap images.
I can re-review and approve once that is done.

.github/docker/cluster_test_12/Dockerfile Outdated Show resolved Hide resolved
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
@deepthi deepthi merged commit 9fc6d52 into vitessio:main Apr 16, 2022
@deepthi deepthi deleted the upgrade-to-go1.18.1 branch April 16, 2022 00:36
@frouioui frouioui mentioned this pull request Jun 6, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: General Changes throughout the code base Type: CI/Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants