-
Notifications
You must be signed in to change notification settings - Fork 726
Fix daily-tests failures #2065
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
Fix daily-tests failures #2065
Conversation
There was no separate test for this, so it was dropped in the recent refactor. However connect-tests `too_many_sans` does actually exercise this. Restore the behaviour, and add a local test.
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesClick to expand
Other differencesClick to expand
Additional informationCheckout details:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2065 +/- ##
=======================================
Coverage 94.46% 94.47%
=======================================
Files 100 100
Lines 23229 23248 +19
=======================================
+ Hits 21944 21963 +19
Misses 1285 1285 ☔ View full report in Codecov by Sentry. |
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.
Thanks 👍
Unfortunately feature unification turned on fips mode for every workspace item. Rather than chase around making that work, just avoid it by specifying the workspace item each time.
Fixing that uncovered a test that wanted to see the
HandshakePayloadTooLarge
error which didn't carry-over in the recent refactor. Restore that.See passing run: https://github.com/rustls/rustls/actions/runs/10197371458