external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixes param error

create_drop_db_gh
gindibay 2023-10-13 11:42:45 +03:00
parent 701c4be79c
commit 3abc40a55a
2 changed files with 47 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.github/workflows/build_and_test.yml vendored
View File

@ -243,7 +243,7 @@ jobs:
}
print substr(parts[X], 1, length(parts[X])-1)
}')
echo ${tests}
echo ${TESTS}
gosu circleci \
make -C src/test/regress \
check-arbitrary-configs parallel=4 CONFIGS=${tests}

46
array_fix.patch Normal file
View File

@ -0,0 +1,46 @@
diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml
index 90a4b1432..5a24dc02d 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -90 +90 @@ jobs:
- run: echo "PG_MAJOR=${PG_MAJOR}" >> $GITHUB_ENV
+ run: echo "PG_MAJOR=${PG_MAJOR}" >> ${GITHUB_ENV}
@@ -239 +239 @@ jobs:
- tr '\n' ',' | awk -v N="$N" -v X="$X" -F, '{
+ tr '\n' ',' | awk -v N="${N}" -v X="${X}" -F, '{
@@ -242 +242 @@ jobs:
- parts[i % N] = parts[i % N] $i ","
+ parts[i % N] = parts[i % N] ${i} ","
@@ -246 +246 @@ jobs:
- echo $TESTS
+ echo ${tests}
@@ -249 +249 @@ jobs:
- check-arbitrary-configs parallel=4 CONFIGS=$TESTS
+ check-arbitrary-configs parallel=4 CONFIGS=${tests}
@@ -331 +331 @@ jobs:
- citus-post-tar=${GITHUB_WORKSPACE}/install-$PG_MAJOR.tar; \
+ citus-post-tar=${GITHUB_WORKSPACE}/install-${PG_MAJOR}.tar; \
@@ -342 +342 @@ jobs:
- citus-post-tar=${GITHUB_WORKSPACE}/install-$PG_MAJOR.tar; \
+ citus-post-tar=${GITHUB_WORKSPACE}/install-${PG_MAJOR}.tar; \
@@ -435,3 +435,3 @@ jobs:
- detected_changes=$(git diff origin/main... --name-only --diff-filter=AM | (grep 'src/test/regress/sql/.*\.sql\|src/test/regress/spec/.*\.spec\|src/test/regress/citus_tests/test/test_.*\.py' || true))
- tests=${detected_changes}
- if [ -z "$tests" ]; then
+ detected_changes=$(git diff origin/main... --name-only --diff-filter=AM | grep 'src/test/regress/sql/.*\.sql\|src/test/regress/spec/.*\.spec\|src/test/regress/citus_tests/test/test_.*\.py' | tr '\n' ',')
+ tests=$(echo "${detected_changes}" | sed 's/,$//')
+ if [ -z "${tests}" ]; then
@@ -440 +440 @@ jobs:
- echo "Detected tests " $tests
+ echo "Detected tests " ${tests}
@@ -442 +442 @@ jobs:
- echo tests="$tests" >> "$GITHUB_OUTPUT"
+ echo tests="${tests}" >> "${GITHUB_OUTPUT}"
@@ -466 +466 @@ jobs:
- tests_array=($tests)
+ IFS=',' read -ra tests_array <<< "$tests"
@@ -469,2 +469,2 @@ jobs:
- test_name=$(echo "$test" | sed -r "s/.+\/(.+)\..+/\1/")
- gosu circleci src/test/regress/citus_tests/run_test.py $test_name --repeat ${{ env.runs }} --use-base-schedule --use-whole-schedule-line
+ test_name=$(echo "${test}" | sed -r "s/.+\/(.+)\..+/\1/")
+ gosu circleci src/test/regress/citus_tests/run_test.py ${test_name} --repeat ${{ env.runs }} --use-base-schedule --use-whole-schedule-line