Skip to content

Stable 24.8 - CI fixes: timeouts, batch split, and test-result noise #2106

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.

Sign up for GitHub

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

Merged
strtgbb merged 5 commits into from
Jul 23, 2026
+9 −3
Merged

Stable 24.8 - CI fixes: timeouts, batch split, and test-result noise #2106

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d1e3d8f
Pass --cicd to regression suite so command timeout is 600s instead of…
CarlosFelipeOR Jul 22, 2026
c04db13
Integration tests (release): split into 6 batches and raise timeout t…
CarlosFelipeOR Jul 22, 2026
0d0728e
broken_tests: mark integration test_user_overcommit as known-broken o…
CarlosFelipeOR Jul 22, 2026
b6343a7
stress_tests.lib: filter self-match in "Killed by signal (output file…
CarlosFelipeOR Jul 22, 2026
80c29de
stress_tests.lib: fix "Killed by signal (output files)" false positiv…
CarlosFelipeOR Jul 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/regression-reusable-suite.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,7 @@ jobs:
--collect-service-logs
--output classic
--parallel 1
--cicd
--log raw.log
--with-analyzer
artifact_paths: |
Expand Down
2 changes: 1 addition & 1 deletion docker/test/stateless/stress_tests.lib
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,7 @@ function check_logs_for_critical_errors()
# Remove file fatal_messages.txt if it's empty
[ -s /test_output/fatal_messages.txt ] || rm /test_output/fatal_messages.txt

rg -Faz "########################################" /test_output/* > /dev/null \
rg -Faz "########################################" /test_output/* | rg -v "rg -Fa" > /dev/null \
&& echo -e "Killed by signal (output files)$FAIL" >> /test_output/test_results.tsv

function get_gdb_log_context()
Expand Down
5 changes: 5 additions & 0 deletions tests/broken_tests.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,5 +49,10 @@
"02098_hashed_array_dictionary_simple_key": {
"reason": "KNOWN: timeout with sanitizer",
"check_types": ["tsan", "asan", "msan", "ubsan"]
},
"test_overcommit_tracker/test.py::test_user_overcommit": {
"reason": "INVESTIGATE - AssertionError on tsan",
"message": "AssertionError: all tasks are killed",
"check_types": ["tsan"]
}
}
4 changes: 2 additions & 2 deletions tests/ci/ci_config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -430,9 +430,9 @@ class CI:
),
JobNames.INTEGRATION_TEST: CommonJobConfigs.INTEGRATION_TEST.with_properties(
required_builds=[BuildNames.PACKAGE_RELEASE],
num_batches=4,
num_batches=6,
# release_only=True,
timeout=12000, # the job timed out with default value (7200)
timeout=4 * 3600, # the job timed out with default value (7200)
),
JobNames.INTEGRATION_TEST_FLAKY: CommonJobConfigs.INTEGRATION_TEST.with_properties(
required_builds=[BuildNames.PACKAGE_ASAN],
Expand Down
Loading