-
Notifications
You must be signed in to change notification settings - Fork 11
Antalya 25.8 - Move stateless crossout logic into clickhouse-test #1227
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
Conversation
|
Crossouts are still not working reliably when the match message is on the first line. The first line in the praktika report is created from both The crossout logic cannot be moved to after the merge happens, because the test status must be set before then. |
5a2248c to
29d97c9
Compare
Test Plan – PR #1227ObjectiveVerify that known broken tests are marked as BROKEN by 1️⃣ Local execution (core change)
2️⃣ Newly added broken tests (Azure filesystem cache)These tests fail locally and in contributor CI when Azure storage is not configured, and should be marked as BROKEN when the Azure-related error occurs.
3️⃣ CI validation
✅ Acceptance criteria
|
|
The aforementioned issue has been resolved, see |
Changelog category (leave one):
Moving the crossout logic into clickhouse-test allows for the broken tests list to function locally and show BROKEN status in actions logs.
Added a new
clickhouse-testargument--known-fails-file-pathThis PR is required for #1185
CI/CD Options
Exclude tests:
Regression jobs to run: