-
Notifications
You must be signed in to change notification settings - Fork 190
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
Update Go to v1.17 #473
Update Go to v1.17 #473
Conversation
4f86aa3
to
02f325c
Compare
go.mod
Outdated
github.com/docker/go-metrics v0.0.1 // indirect | ||
github.com/docker/libtrust v0.0.0-20160708172513-aabc10ec26b7 // indirect | ||
github.com/fluxcd/pkg/apis/meta v0.10.1 | ||
github.com/fluxcd/pkg/apis/meta v0.10.2 | ||
github.com/fluxcd/pkg/gittestserver v0.4.2 | ||
github.com/fluxcd/pkg/gitutil v0.1.0 | ||
github.com/fluxcd/pkg/helmtestserver v0.2.0 |
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.
There are new releases for both gittestserver and helmtestserver with Go 1.17
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.
Oops, seems I missed these. Thanks!
1bdc2ac
to
952223e
Compare
Signed-off-by: Aurel Canciu <aurelcanciu@gmail.com>
The mod replaces are no longer required since helm v3.7.1. Signed-off-by: Aurel Canciu <aurelcanciu@gmail.com>
Signed-off-by: Aurel Canciu <aurelcanciu@gmail.com>
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.
LGTM
Thanks @relu
I've successfully tested the build on multi-arch using buildx with qemu. I suppose some more thorough testing will be needed though to evaluate the changes.