Skip to content

Commit 7289809

Browse files
committed
Pg18 beta conf file updated
Pg18 beta conf file updated (cherry picked from commit c36410c) Update image suffix in build and test workflow Update image suffix in build configuration Update image suffix in build configuration Update image suffix in build configuration (cherry picked from commit 7dbb946) Update image suffix in build_and_test.yml to reflect latest development version
1 parent 6b6d959 commit 7289809

File tree

5 files changed

+49
-8
lines changed

5 files changed

+49
-8
lines changed

.devcontainer/Dockerfile

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,18 @@ RUN mkdir .pgenv-staging/
108108
RUN cp -r .pgenv/src .pgenv/pgsql-* .pgenv/config .pgenv-staging/
109109
RUN rm .pgenv-staging/config/default.conf
110110

111+
FROM base AS pg18
112+
RUN MAKEFLAGS="-j $(nproc)" pgenv build 18beta2
113+
RUN rm .pgenv/src/*.tar*
114+
RUN make -C .pgenv/src/postgresql-*/ clean
115+
RUN make -C .pgenv/src/postgresql-*/src/include install
116+
117+
# Stage the pgenv artifacts for PG18
118+
RUN mkdir .pgenv-staging/
119+
RUN cp -r .pgenv/src .pgenv/pgsql-* .pgenv/config .pgenv-staging/
120+
RUN rm .pgenv-staging/config/default.conf
121+
122+
111123
FROM base AS uncrustify-builder
112124

113125
RUN sudo apt update && sudo apt install -y cmake tree
@@ -201,6 +213,7 @@ COPY --link --from=uncrustify-builder /uncrustify/usr/ /usr/
201213
COPY --link --from=pg15 /home/citus/.pgenv-staging/ /home/citus/.pgenv/
202214
COPY --link --from=pg16 /home/citus/.pgenv-staging/ /home/citus/.pgenv/
203215
COPY --link --from=pg17 /home/citus/.pgenv-staging/ /home/citus/.pgenv/
216+
COPY --link --from=pg18 /home/citus/.pgenv-staging/ /home/citus/.pgenv/
204217

205218
COPY --link --from=pipenv /home/citus/.local/share/virtualenvs/ /home/citus/.local/share/virtualenvs/
206219

@@ -216,7 +229,7 @@ COPY --chown=citus:citus .psqlrc .
216229
RUN sudo chown --from=root:root citus:citus -R ~
217230

218231
# sets default pg version
219-
RUN pgenv switch 17.5
232+
RUN pgenv switch 18beta2
220233

221234
# make connecting to the coordinator easy
222235
ENV PGPORT=9700

.github/workflows/build_and_test.yml

Lines changed: 32 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,12 @@ jobs:
3232
style_checker_image_name: "ghcr.io/citusdata/stylechecker"
3333
style_checker_tools_version: "0.8.18"
3434
sql_snapshot_pg_version: "17.5"
35-
image_suffix: "-vb17c33b"
35+
image_suffix: "-dev-219b87c"
3636
pg15_version: '{ "major": "15", "full": "15.13" }'
3737
pg16_version: '{ "major": "16", "full": "16.9" }'
3838
pg17_version: '{ "major": "17", "full": "17.5" }'
39-
upgrade_pg_versions: "15.13-16.9-17.5"
39+
pg18_version: '{ "major": "18", "full": "18beta2" }'
40+
upgrade_pg_versions: "15.13-16.9-17.5-18beta2"
4041
steps:
4142
# Since GHA jobs need at least one step we use a noop step here.
4243
- name: Set up parameters
@@ -113,6 +114,7 @@ jobs:
113114
- ${{ needs.params.outputs.pg15_version }}
114115
- ${{ needs.params.outputs.pg16_version }}
115116
- ${{ needs.params.outputs.pg17_version }}
117+
- ${{ needs.params.outputs.pg18_version }}
116118
runs-on: ubuntu-latest
117119
container:
118120
image: "${{ matrix.image_name }}:${{ fromJson(matrix.pg_version).full }}${{ matrix.image_suffix }}"
@@ -144,6 +146,7 @@ jobs:
144146
- ${{ needs.params.outputs.pg15_version }}
145147
- ${{ needs.params.outputs.pg16_version }}
146148
- ${{ needs.params.outputs.pg17_version }}
149+
- ${{ needs.params.outputs.pg18_version }}
147150
make:
148151
- check-split
149152
- check-multi
@@ -173,6 +176,10 @@ jobs:
173176
pg_version: ${{ needs.params.outputs.pg17_version }}
174177
suite: regress
175178
image_name: ${{ needs.params.outputs.fail_test_image_name }}
179+
- make: check-failure
180+
pg_version: ${{ needs.params.outputs.pg18_version }}
181+
suite: regress
182+
image_name: ${{ needs.params.outputs.fail_test_image_name }}
176183
- make: check-enterprise-failure
177184
pg_version: ${{ needs.params.outputs.pg15_version }}
178185
suite: regress
@@ -185,6 +192,10 @@ jobs:
185192
pg_version: ${{ needs.params.outputs.pg17_version }}
186193
suite: regress
187194
image_name: ${{ needs.params.outputs.fail_test_image_name }}
195+
- make: check-enterprise-failure
196+
pg_version: ${{ needs.params.outputs.pg18_version }}
197+
suite: regress
198+
image_name: ${{ needs.params.outputs.fail_test_image_name }}
188199
- make: check-pytest
189200
pg_version: ${{ needs.params.outputs.pg15_version }}
190201
suite: regress
@@ -197,6 +208,10 @@ jobs:
197208
pg_version: ${{ needs.params.outputs.pg17_version }}
198209
suite: regress
199210
image_name: ${{ needs.params.outputs.fail_test_image_name }}
211+
- make: check-pytest
212+
pg_version: ${{ needs.params.outputs.pg18_version }}
213+
suite: regress
214+
image_name: ${{ needs.params.outputs.fail_test_image_name }}
200215
- make: installcheck
201216
suite: cdc
202217
image_name: ${{ needs.params.outputs.test_image_name }}
@@ -209,6 +224,10 @@ jobs:
209224
suite: cdc
210225
image_name: ${{ needs.params.outputs.test_image_name }}
211226
pg_version: ${{ needs.params.outputs.pg17_version }}
227+
- make: installcheck
228+
suite: cdc
229+
image_name: ${{ needs.params.outputs.test_image_name }}
230+
pg_version: ${{ needs.params.outputs.pg18_version }}
212231
- make: check-query-generator
213232
pg_version: ${{ needs.params.outputs.pg15_version }}
214233
suite: regress
@@ -221,6 +240,10 @@ jobs:
221240
pg_version: ${{ needs.params.outputs.pg17_version }}
222241
suite: regress
223242
image_name: ${{ needs.params.outputs.fail_test_image_name }}
243+
- make: check-query-generator
244+
pg_version: ${{ needs.params.outputs.pg18_version }}
245+
suite: regress
246+
image_name: ${{ needs.params.outputs.fail_test_image_name }}
224247
runs-on: ubuntu-latest
225248
container:
226249
image: "${{ matrix.image_name }}:${{ fromJson(matrix.pg_version).full }}${{ needs.params.outputs.image_suffix }}"
@@ -264,6 +287,7 @@ jobs:
264287
- ${{ needs.params.outputs.pg15_version }}
265288
- ${{ needs.params.outputs.pg16_version }}
266289
- ${{ needs.params.outputs.pg17_version }}
290+
- ${{ needs.params.outputs.pg18_version }}
267291
parallel: [0,1,2,3,4,5] # workaround for running 6 parallel jobs
268292
steps:
269293
- uses: actions/checkout@v4
@@ -314,6 +338,10 @@ jobs:
314338
new_pg_major: 17
315339
- old_pg_major: 15
316340
new_pg_major: 17
341+
- old_pg_major: 17
342+
new_pg_major: 18
343+
- old_pg_major: 16
344+
new_pg_major: 18
317345
env:
318346
old_pg_major: ${{ matrix.old_pg_major }}
319347
new_pg_major: ${{ matrix.new_pg_major }}
@@ -402,7 +430,7 @@ jobs:
402430
CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
403431
runs-on: ubuntu-latest
404432
container:
405-
image: ${{ needs.params.outputs.test_image_name }}:${{ fromJson(needs.params.outputs.pg17_version).full }}${{ needs.params.outputs.image_suffix }}
433+
image: ${{ needs.params.outputs.test_image_name }}:${{ fromJson(needs.params.outputs.pg18_version).full }}${{ needs.params.outputs.image_suffix }}
406434
needs:
407435
- params
408436
- test-citus
@@ -514,7 +542,7 @@ jobs:
514542
name: Test flakyness
515543
runs-on: ubuntu-latest
516544
container:
517-
image: ${{ needs.params.outputs.fail_test_image_name }}:${{ fromJson(needs.params.outputs.pg17_version).full }}${{ needs.params.outputs.image_suffix }}
545+
image: ${{ needs.params.outputs.fail_test_image_name }}:${{ fromJson(needs.params.outputs.pg18_version).full }}${{ needs.params.outputs.image_suffix }}
518546
options: --user root
519547
env:
520548
runs: 8

.github/workflows/packaging-test-pipelines.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
# Postgres versions are stored in .github/workflows/build_and_test.yml
3030
# file in json strings with major and full keys.
3131
# Below command extracts the versions and get the unique values.
32-
pg_versions=$(cat .github/workflows/build_and_test.yml | grep -oE '"major": "[0-9]+", "full": "[0-9.]+"' | sed -E 's/"major": "([0-9]+)", "full": "([0-9.]+)"/\1/g' | sort | uniq | tr '\n', ',')
32+
pg_versions=$(cat .github/workflows/build_and_test.yml | grep -oE '"major": "[0-9]+", "full": "[^"]+"' | sed -E 's/.*"major": "([0-9]+)".*/\1/' | sort -n | uniq | tr '\n' ',')
3333
pg_versions_array="[ ${pg_versions} ]"
3434
echo "Supported PG Versions: ${pg_versions_array}"
3535
# Below line is needed to set the output variable to be used in the next job

configure

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2588,7 +2588,7 @@ fi
25882588
if test "$with_pg_version_check" = no; then
25892589
{ $as_echo "$as_me:${as_lineno-$LINENO}: building against PostgreSQL $version_num (skipped compatibility check)" >&5
25902590
$as_echo "$as_me: building against PostgreSQL $version_num (skipped compatibility check)" >&6;}
2591-
elif test "$version_num" != '15' -a "$version_num" != '16' -a "$version_num" != '17'; then
2591+
elif test "$version_num" != '15' -a "$version_num" != '16' -a "$version_num" != '17' -a "$version_num" != '18'; then
25922592
as_fn_error $? "Citus is not compatible with the detected PostgreSQL version ${version_num}." "$LINENO" 5
25932593
else
25942594
{ $as_echo "$as_me:${as_lineno-$LINENO}: building against PostgreSQL $version_num" >&5

configure.ac

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ AC_SUBST(with_pg_version_check)
8080

8181
if test "$with_pg_version_check" = no; then
8282
AC_MSG_NOTICE([building against PostgreSQL $version_num (skipped compatibility check)])
83-
elif test "$version_num" != '15' -a "$version_num" != '16' -a "$version_num" != '17'; then
83+
elif test "$version_num" != '15' -a "$version_num" != '16' -a "$version_num" != '17' -a "$version_num" != '18'; then
8484
AC_MSG_ERROR([Citus is not compatible with the detected PostgreSQL version ${version_num}.])
8585
else
8686
AC_MSG_NOTICE([building against PostgreSQL $version_num])

0 commit comments

Comments
 (0)