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
chore: Bump various dependency versions #128
base: master
Are you sure you want to change the base?
Conversation
* docker images to ubuntu:24.04 * ns-3 to 3.4.1 * go to 1.22.3
sim/Dockerfile
Outdated
ENV GO_VERS 1.21.5 | ||
RUN wget -O go.tar.gz https://dl.google.com/go/go$GO_VERS.linux-$TARGETARCH.tar.gz | ||
ENV GO_VERS 1.22.3 | ||
ADD --checksum=sha256:6c33e52a5b26e7aa021b94475587fce80043a727a54ceb0eee2f9fc160646434 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't realize Docker allowed verification by SHA256. How cool is that?!
Copy-paste error I assume:
ADD --checksum=sha256:6c33e52a5b26e7aa021b94475587fce80043a727a54ceb0eee2f9fc160646434 \ | |
ADD --checksum=sha256: 8920ea521bad8f6b7bc377b4824982e011c19af27df88a815e3586ea895f1b36 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh wait, not a copy-paste error. This is the SHA256 for the arm64, which is why it fails on CI (amd64).
@larseggert How do we fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm. Not sure if there is a way to do this. Do we actually need to install a custom go? 24.04 has go-1.22 via apt
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They used to have really outdated (multiple years) Go versions. If they have 1.22, we can use that.
Fixes #125