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 Go from 1.13 to 1.15 #370

Merged
merged 2 commits into from
Jan 28, 2022
Merged

Upgrade Go from 1.13 to 1.15 #370

merged 2 commits into from
Jan 28, 2022

Conversation

kzys
Copy link
Contributor

@kzys kzys commented Jan 18, 2022

  • The first commit is upgrading Go from 1.13 to 1.15. go-openapi needs Go 1.15, which we probably should adopt rather than stop updating. Go 1.15 and Go 1.13 are no longer supported according to https://go.dev/doc/devel/release. We need to discuss Define supported Go versions #364 later.
  • The second commit is making Firecracker's main branch build "soft fail", meaning that the build breakage wouldn't make the whole build broken. Firecracker is making some breaking changes before 1.0. So we cannot make the SDK work with the main branch and released versions.

Our Debian-based BuildKite host now has Go 1.15 to unblock
github.com/go-openapi/runtime upgrade.

Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
@kzys
Copy link
Contributor Author

kzys commented Jan 18, 2022

The main branch build wouldn't work due to #371.

@kzys kzys force-pushed the go115 branch 7 times, most recently from 5beb346 to a3fa2b8 Compare January 28, 2022 04:48
Firecracker is making some breaking changes before reaching 1.0.
Due to that, it is hard to make the SDK work with released versions and
main branch.

This commit makes Firecracker main branch's test failures "soft fail",
meaning, tests are still run against the branch, but the failures
wouldn't impact the overall test result.

Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
@kzys
Copy link
Contributor Author

kzys commented Jan 28, 2022

Another blocker is firecracker-microvm/firecracker#2876.

Firecracker 1.0 is coming. So we need to take these changes anyway later.

@kzys kzys merged commit 195bc24 into firecracker-microvm:main Jan 28, 2022
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.

4 participants