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

multi: bump Go versions #7944

Merged
merged 2 commits into from
Sep 5, 2023

Conversation

positiveblue
Copy link
Collaborator

Bump all build go versions to v1.21.0
Bump the minimum build package version to v1.19.0

Debian "buster" is not longer supported. Security updates have been discontinued since June 30th 2022. We will build using the next version, "bullseye".

Copy link
Collaborator

@guggero guggero left a comment

Choose a reason for hiding this comment

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

Needs some updates to fully work, but other than that looks good.

lnrpc/Dockerfile Outdated Show resolved Hide resolved
@@ -1,8 +1,11 @@
module github.com/lightningnetwork/lnd/cert

go 1.16
go 1.19
Copy link
Collaborator

Choose a reason for hiding this comment

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

We'll need to push up tags for all modules and bump their version in the main go.mod, but I guess that's always nice to make sure we include all the latest changes.

@@ -1,105 +1,30 @@
github.com/aead/siphash v1.0.1/go.mod h1:Nywa3cDsYNNK3gaciGTWPwHt0wlpNV15vwmswBAUSII=
Copy link
Collaborator

Choose a reason for hiding this comment

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

Interesting that there's such a big diff in the sum files... Did we just never run go mod tidy or did the more recent Go versions change what needs to be validated?

cert/go.mod Outdated
@@ -2,7 +2,10 @@ module github.com/lightningnetwork/lnd/cert

go 1.19

require github.com/stretchr/testify v1.5.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: commit message should probably read "build: run go mod tidy"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I should have escaped the ` from git commit -m "build: run `go mod tidy`" 👍

@guggero
Copy link
Collaborator

guggero commented Sep 1, 2023

Looks like the tools/go.mod file isn't complete. Was able to get the CI check commits step working by doing:

cd tools
go get github.com/btcsuite/btcd/wire@v0.23.3
go install -trimpath github.com/btcsuite/btcd
go mod tidy

@JssDWt
Copy link
Contributor

JssDWt commented Sep 3, 2023

Debian "buster" is not longer supported.

This probably needs some good advertisement. I imagine some early noderunners are running on buster, because that was raspberry pi os until 2021-10-30.

@guggero
Copy link
Collaborator

guggero commented Sep 4, 2023

Debian "buster" is not longer supported.

This probably needs some good advertisement. I imagine some early noderunners are running on buster, because that was raspberry pi os until 2021-10-30.

This only affects the base image of the Docker build. That doesn't have any impact on anyone outside of the docker image build process. You can run a bookworm based docker image on a buster host (and vice versa) without any issues.

Bump all build go versions to v1.21.0
Bump the minimum build package version to v1.19.0

Debian "buster" is not longer supported. Security updates have been
discontinued since June 30th 2022. We will build using the latest
version, "bookworm".
The docker image have been updated so we are using another protobuf
version to generate the files. The generate files include the version of
the compiler used to creating them, so we need this commit to pass the
`rpc-check` step in our CI.
Copy link
Member

@yyforyongyu yyforyongyu left a comment

Choose a reason for hiding this comment

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

LGTM🎉

@guggero guggero merged commit f7cb443 into lightningnetwork:master Sep 5, 2023
21 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants