-
Notifications
You must be signed in to change notification settings - Fork 546
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
Run tests using Go 1.18 #2093
Run tests using Go 1.18 #2093
Conversation
Signed-off-by: Jason Hall <[email protected]>
This hits #2091 😢 |
this is something we can enable only for the tests or does that affects the binary when users use this feature? |
I think we could Just building cosign from HEAD with Go 1.18 and testing locally, I'm able to |
quick ping on this if possible? it's blocking upgrading rekor so we can handle some incompatible rekor changes that prevent blob verification #2058 |
My understanding is we can either update our usage of the deprecated SHA-1 cert (#2091 (comment)) or temporarily set the env var to ignore it during tests. If you're blocked I'd say set the env var in tests and upgrade to Go 1.18+ to get unblocked. |
Added the suppression, this can unblock a few pending PRs. |
Codecov Report
@@ Coverage Diff @@
## main #2093 +/- ##
==========================================
- Coverage 26.33% 26.23% -0.11%
==========================================
Files 129 130 +1
Lines 7564 7617 +53
==========================================
+ Hits 1992 1998 +6
- Misses 5317 5362 +45
- Partials 255 257 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
We should fix this with new testdata but we're blocked on an upstream change. Signed-off-by: Dan Lorenc <[email protected]>
* Run tests using Go 1.18 Signed-off-by: Jason Hall <[email protected]> * Suppress SHA1_RSA failures, these only occur in tests. We should fix this with new testdata but we're blocked on an upstream change. Signed-off-by: Dan Lorenc <[email protected]> Signed-off-by: Jason Hall <[email protected]> Signed-off-by: Dan Lorenc <[email protected]> Co-authored-by: Dan Lorenc <[email protected]>
Summary
Go 1.19 will be out next month, which means Go 1.17 will be out of support.
Signed-off-by: Jason Hall [email protected]
Release Note
NONE
Documentation
NONE