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
Randomize disabled optimizations in CI #57315
Randomize disabled optimizations in CI #57315
Conversation
This is an automated comment for commit 89b9373 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
So many red tests, which is both nice (detecting issues) and bad (detecting issues).
Need time to review the rest of the failures |
How would random settings help to understand that? |
Yeah, I should probably have started by enabling them by default first and then we could leave them random. I'll update and keep analyzing the errors to report bugs found |
Much better results now, and by better I mean horrible 😄 Several crashes and sanitizer alerts. I'll start reporting |
Failures:
Before continuing it's pretty obvious that |
New batch of failures:
|
I'm removing the randomization of optimize_using_constraints and I expect the rest to be clean. I'm tempted to remove broken optimizations, at least for the old infra (not analyzer), but we can do that in other PRs if and when the proper fix is done. |
Only unrelated (flaky) failures. I'll set the PR to only enable randomization of these settings for now, so at least we know they get usage continuosly and it's easier to decide whether to enable them by default or not |
Changes look harmless so I'm merging. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Randomize these optimizations that are disabled by default. Do they work correctly? Should they be on instead? Let's see