ci(fuzz): drop sccache from Fuzz Smoke (Windows server crashes on GNS TUs)#582
Merged
Merged
Conversation
… TUs) Fuzz Smoke has failed every scheduled run since 2026-07-03 with the identical failure signature the Windows ASan job hit before d93b5af dropped sccache there: the Windows sccache server crashes mid-build on GameNetworkingSockets' largest TUs (steamnetworkingsockets_shared/certs/certstore.cpp) with "An existing connection was forcibly closed by the remote host (os error 10054)". A local clang-cl build of GNS with no sccache completes fine, so sccache is the failing component, not our code. Mirror the proven fix: build both matrix legs (asan, ubsan) without the sccache launcher — direct compile, full cold build every run. Fixes #573. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe fuzz smoke GitHub Actions workflow removes sccache-based compiler caching entirely: the global environment variable, the setup step, the CMake compiler-cache flag, and the stats-reporting step are deleted, with an added comment explaining the intentional omission for Windows builds. ChangesFuzz Workflow sccache Removal
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
d93b5af5dropped sccache there: the Windows sccache server crashes mid-build on GameNetworkingSockets' largest TUs (steamnetworkingsockets_shared/certs/certstore.cpp) withAn existing connection was forcibly closed by the remote host (os error 10054).d93b5af5: build both matrix legs (asan,ubsan) without the sccache launcher — direct compile, full cold build every run. Same tradeoff already accepted for the ASan job.Test plan
py -c "import yaml; yaml.safe_load(...)")workflow_dispatchrun of "Fuzz Smoke" and confirm bothasanandubsanmatrix legs go green🤖 Generated with Claude Code
Summary by CodeRabbit