Skip to content

Conversation

@eaydingol
Copy link
Collaborator

@eaydingol eaydingol commented Dec 19, 2025

Adds support for loading citus and columnar so files from a different directory during tests.

usage:
CITUSLIBDIR=/tmp/opt/citus_versions/v13.2.1 make check-minimal
CITUSLIBDIR=/tmp/opt/citus_versions/v13.2.1 citus_tests/run_test.py columnar_create

Note to reviewer:
The sample test in build_and_test.yml will be removed before merge (using a dev image from citusdata/the-process#180).
It generates the following test, see logs in Checks tab.
[Build & Test / Test Citus Lib N-1 / PG17 - check-minimal - lib-v13.2.0 (pull_request)]

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.77%. Comparing base (ee3812d) to head (204aceb).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8390      +/-   ##
==========================================
- Coverage   88.79%   88.77%   -0.02%     
==========================================
  Files         287      287              
  Lines       63237    63247      +10     
  Branches     7928     7932       +4     
==========================================
+ Hits        56149    56150       +1     
- Misses       4756     4765       +9     
  Partials     2332     2332              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eaydingol eaydingol force-pushed the eag/mvtests-preso-workflow branch from 16b7478 to b4269b6 Compare December 19, 2025 06:41
@eaydingol eaydingol changed the title CI/CD test using alternative citus library directory Alternative lib directory for tests Dec 19, 2025
eaydingol added a commit to citusdata/the-process that referenced this pull request Dec 25, 2025
Add the library files from the latest minor release to exttester image. 
These files will be used in mixed version testing, see draft PR
citusdata/citus#8369

Tested locally for 17.7

```
make build-exttester-17.7
// in container
ls /opt/citus-versions/v13.2.0/
citus.so  citus_columnar.so  citus_decoders  citus_pgoutput.so  citus_wal2json.so

```
Slight increase in the image size:
```
ghcr.io/citusdata/exttester            17.7-dev-15cd08f      ae73e573b28d   7 seconds ago   3.02GB   
ghcr.io/citusdata/exttester            17.7-dev-076d6fb      be3fc5177d09   3 minutes ago   3.05GB  // from this PR
```

See related PR in citus repo, testing the dev image as well :
citusdata/citus#8390
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants