-
Notifications
You must be signed in to change notification settings - Fork 94
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
versions: Bump golang #1728
versions: Bump golang #1728
Conversation
f24f08e
to
fcc2e39
Compare
@stevenhorsman to fix the error from govulncheck for csi-wrapper please try follow
the output looks like:
After this
|
47650f1
to
33939ce
Compare
The golvuln is fixed now, but we are hitting typecheck errors in the linting check now, I guess due to differences in the package we've upgraded, so I'll dig into those more when I get a chance later. |
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
@liudalibj - I wonder if you can help with the issues I'm seeing on this PR now. After doing your suggested changes the govuln is clean, but I get thousands of lint errors now as some of them are in the generated csi-wrapper code I tried re-running |
I will take a look on this issue today. |
Bump to go 1.21.8 to resolve CVEs: - https://pkg.go.dev/vuln/GO-2024-2600 - https://pkg.go.dev/vuln/GO-2024-2599 - https://pkg.go.dev/vuln/GO-2024-2598 - https://pkg.go.dev/vuln/GO-2024-2610 Signed-off-by: stevenhorsman <steven@uk.ibm.com>
- Bump google.golang.org/protobuf dependency to 1.33 to fix: - https://pkg.go.dev/vuln/GO-2024-2611 Signed-off-by: stevenhorsman <steven@uk.ibm.com> Co-authored-by: Da Li Liu <liudali@cn.ibm.com> Signed-off-by: stevenhorsman <steven@uk.ibm.com>
a7aba24
to
6290099
Compare
try to fix go lint issue Signed-off-by: Da Li Liu <liudali@cn.ibm.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.
@stevenhorsman the lint issue is fixed by using the latest golint version |
Thanks so much - I was obviously following red herrings looking at the generated code! |
Bump to go 1.21.8 to resolve CVEs: