-
Notifications
You must be signed in to change notification settings - Fork 197
[Test] CI: add binary swap CI for REL_2_STABLE #1522
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
base: REL_2_STABLE
Are you sure you want to change the base?
Conversation
df3d64c to
8cac6b5
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.
Hi, amazing work! No doubt! I made some comments. They are optional. I just do not understand all the nuances and asked for more details.
| - name: Checkout Baseline Code | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| ref: ${{ needs.resolve-baseline.outputs.baseline_version }} |
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.
Well, if it's the baseline version and has a tag. Maybe the RPM has already been uploaded and we don't need to build it again?
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.
Now we don't have the RPM package for release 2.0 yet. So we need to create it when running the tests.
|
|
||
| CREATE TABLE sales_partition_heap_part3 | ||
| PARTITION OF sales_partition_heap | ||
| FOR VALUES IN ('Processing'); |
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.
Should we also check if AO/AOCO tables work as expected?
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.
NP. Let me see how to add these tests to the workflow.
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.
Now we have added these tests.
cc181b0 to
ababd95
Compare
Add comprehensive binary swap CI testing for REL_2_STABLE branch: 1. Update test_binary_swap.sh script for testing 2. Integrate with parallel build workflows 3. Add the upgrading_compatibility for robust testing
a6cbe5a to
4e80654
Compare
Binary Swap Testing OverviewThis PR enhances the binary swap testing infrastructure that validates Cloudberry version compatibility. Core Mechanism: Uses Key Improvements: Strengthened the
This ensures safe version upgrades/downgrades while maintaining full functionality. |
|
Hi @leborchuk @my-ship-it @congxuebin @edespino guys, could you help review this PR again? Thanks! |
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