-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
build: update go modules #12663
build: update go modules #12663
Conversation
go.mod
Outdated
github.com/kubernetes-incubator/external-storage => github.com/libopenstorage/external-storage v0.20.4-openstorage-rc3 | ||
github.com/portworx/sched-ops => github.com/portworx/sched-ops v0.20.4-openstorage-rc3 | ||
) | ||
replace github.com/googleapis/gnostic => github.com/googleapis/gnostic v0.4.1 |
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 tried updating this as well with no success. The latest is github.com/google/gnostic v0.6.9.
1662eff
to
33ccdc8
Compare
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 think the build issues could be resolved by applying anyone of the last two comment kubernetes/client-go#1084
#12672 will probably fix this issue |
4f0cf28
to
2dae2ec
Compare
@BlaineEXE ready to rebase? |
This pull request has merge conflicts that must be resolved before it can be merged. @BlaineEXE please rebase it. https://rook.io/docs/rook/latest/Contributing/development-flow/#updating-your-fork |
a059333
to
5452635
Compare
Update go modules. The primary goal is to remove the goproxy import which has security warnings. The library is not used directly or indirectly by any called Rook code, only dependency modules. Therefore, exclude it from the build. At the same time, portworx deps are starting to complicate the module update again. Rook also doesn't directly or indirectly use the portworx code, so exclude it also. Signed-off-by: Blaine Gardner <blaine.gardner@ibm.com>
5452635
to
87fc552
Compare
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.
Nice to see the go.sum shrunk by ~80 lines
build: update go modules (backport #12663)
Update go modules. The primary goal is to remove the goproxy import which has security warnings. The library is not used directly or indirectly by any called Rook code, only dependency modules. Therefore, exclude it from the build.
At the same time, portworx deps are starting to complicate the module update again. Rook also doesn't directly or indirectly use the portworx code, so exclude it also.
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
skip-ci
on the PR.