-
Notifications
You must be signed in to change notification settings - Fork 193
Fix CI test failures due to insufficient disk space #1511
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
|
👍 👍 👍 💯 |
ed9009e to
dcf8c2f
Compare
leborchuk
left a comment
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.
See failed pax-ic-good-opt-off. But the failure should not be in the changes that have been made.
Error shows unexists relation
Found regression.diffs at: ./contrib/pax_storage/src/test/regress/regression.diffs
diff -I HINT: -I CONTEXT: -I GP_IGNORE: -U3 /__w/cloudberry/cloudberry/contrib/pax_storage/src/test/regress/expected/index_constraint_naming_partition.out /__w/cloudberry/cloudberry/contrib/pax_storage/src/test/regress/results/index_constraint_naming_partition.out
--- /__w/cloudberry/cloudberry/contrib/pax_storage/src/test/regress/expected/index_constraint_naming_partition.out 2025-12-28 20:14:29.244660945 -0800
+++ /__w/cloudberry/cloudberry/contrib/pax_storage/src/test/regress/results/index_constraint_naming_partition.out 2025-12-28 20:14:29.293661640 -0800
@@ -178,9 +178,10 @@
$fn$;
-- validate that there are no constraints when we start
SELECT * FROM partition_tables_show_all('r');
-ERROR: relation "r" does not exist
-CONTEXT: SQL function "partition_tables" statement 1
-SQL function "partition_tables_show_all" statement 1
+ partition_name | parent_name | root_name | constraint_name | index_name | constraint_type
+----------------+-------------+-----------+-----------------+------------+-----------------
+(0 rows)
+
-- UNIQUE constraint: validate we correctly add it and can only drop from root
SELECT recreate_two_level_table();
recreate_two_level_table
A set approve since traditional good-opt-on and good-opt-off do not fail anymore.
Thanks @leborchuk . I will continue to improve this PR later for a better code style. |
dcf8c2f to
1f396a3
Compare
|
Updates: In this PR, just improve two obvious places:
Let's see how the tests will go. |
The changes work for |
81efdda to
8ff5b73
Compare
|
Finally! After a cleanup work, we can get 30+G disk space for each test job! |
This commit fixes the disk space issue in tests by: - Adding Docker volume mounts to expose host directories to containers, add removing some pre-installed tools from host, including: - GitHub Actions tool cache - Android SDK, .NET SDK, Haskell (GHC + GHCup), Swift - PowerShell, Chromium, Miniconda, Azure CLI, and Scala BUild Tool - Cleaning up RPM artifacts and source tarballs after extraction in the test job to reclaim additional space This approach frees ~30G of disk space per job, ensuring sufficient space for build and test operations.
0da25cb to
6bfd6e3
Compare
Clean up RPM and source tarball artifacts after installation/extraction
to free disk space for subsequent test execution.
Changes:
Fixes #ISSUE_Number
What does this PR do?
Type of Change
Breaking Changes
Test Plan
make installcheckmake -C src/test installcheck-cbdb-parallelImpact
Performance:
User-facing changes:
Dependencies:
Checklist
Additional Context
CI Skip Instructions