Merged PR 1555841: [OSS] fix regress sql test workflow

[OSS] fix regress sql test workflow

----
Bug fix for the SQL test workflow.
This pull request fixes the regression in the SQL test workflow by updating the GitHub Actions configuration and related scripts.
- Updated `.github/workflows/regress_tests.yml` to improve the build matrix, install dependencies, and run tests for multiple PostgreSQL versions and Ubuntu versions.
- Modified `scripts/start_oss_server.sh` to correct the help message and default directory.
- Minor cleanup in `scripts/install_setup_pg_cron.sh` to remove an unnecessary blank line.

Author: shuaitian-git

.github/workflows/regress_tests.yml

@@ -40,7 +40,7 @@ jobs:
       - name: Install package dependencies
         run: |
           export PG_VERSION=${{ matrix.pg_version }}
-          export CITUS_VERSION=12.1
+          export CITUS_VERSION=12
 
           export LC_ALL=en_US.UTF-8
           export LANGUAGE=en_US
@@ -65,33 +65,72 @@ jobs:
           pkg-config \
           cmake \
           git \
-          postgresql-${PG_VERSION} \
-          postgresql-server-dev-${PG_VERSION} \
-          libpq-dev \
-          libicu-dev \
+          locales \
+          gcc \
+          gdb \
+          libipc-run-perl \
+          unzip \
+          apt-transport-https \
+          bison \
+          flex \
+          libreadline-dev \
+          zlib1g-dev \
           libkrb5-dev \
-          postgresql-${PG_VERSION}-cron \
-          postgresql-${PG_VERSION}-pgvector \
-          postgresql-${PG_VERSION}-postgis-3 \
-          postgresql-${PG_VERSION}-rum \
-          postgresql-${PG_VERSION}-citus-${CITUS_VERSION}
+          software-properties-common \
+          libtool \
+          libicu-dev \
+          libssl-dev
 
           export CLEAN_SETUP=1
           export INSTALL_DEPENDENCIES_ROOT=/tmp/install_setup
           mkdir -p /tmp/install_setup
-          cp ./scripts/* /tmp/install_setup
+
+          # Install dependencies
+          cp ./scripts/setup_versions.sh /tmp/install_setup
+
+          cp ./scripts/install_setup_libbson.sh /tmp/install_setup
           sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT MAKE_PROGRAM=cmake /tmp/install_setup/install_setup_libbson.sh
-          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT /tmp/install_setup/install_setup_pcre2.sh
-          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT /tmp/install_setup/install_setup_intel_decimal_math_lib.sh
+          cp ./scripts/utils.sh /tmp/install_setup
+          cp ./scripts/install_setup_postgres.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT /tmp/install_setup/install_setup_postgres.sh -d /usr/lib/postgresql/${PG_VERSION} $POSTGRES_INSTALL_ARG -v ${PG_VERSION}
+          cp ./scripts/install_setup_rum_oss.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT PGVERSION=$PG_VERSION /tmp/install_setup/install_setup_rum_oss.sh
+          cp ./scripts/install_setup_citus_core_oss.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT PGVERSION=$PG_VERSION /tmp/install_setup/install_setup_citus_core_oss.sh ${CITUS_VERSION}
+          cp ./scripts/install_citus_indent.sh /tmp/install_setup/
           sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT /tmp/install_setup/install_citus_indent.sh
+          cp ./scripts/install_setup_system_rows.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT PGVERSION=$PG_VERSION /tmp/install_setup/install_setup_system_rows.sh
+          cp ./scripts/install_setup_pg_cron.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT PGVERSION=$PG_VERSION /tmp/install_setup/install_setup_pg_cron.sh
+          cp ./scripts/install_setup_intel_decimal_math_lib.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT /tmp/install_setup/install_setup_intel_decimal_math_lib.sh
+          cp ./scripts/install_setup_pcre2.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT /tmp/install_setup/install_setup_pcre2.sh
+          cp ./scripts/install_setup_pgvector.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT PGVERSION=$PG_VERSION /tmp/install_setup/install_setup_pgvector.sh
+
+          sudo add-apt-repository -y ppa:ubuntugis/ppa 
+          sudo apt-get update
+          sudo apt-get install -qy \
+          libproj-dev \
+          libxml2-dev \
+          libjson-c-dev \
+          libgeos++-dev \
+          libgeos-dev
+
+          cp ./scripts/install_setup_postgis.sh /tmp/install_setup/
+          sudo INSTALL_DEPENDENCIES_ROOT=$INSTALL_DEPENDENCIES_ROOT PGVERSION=$PG_VERSION /tmp/install_setup/install_setup_postgis.sh
+          
+          export PATH=$PATH:/usr/lib/postgresql/$PG_VERSION/bin
 
-      - name: Configure, Build and Install HelioDB
+      - name: Configure, Build and Install DocumentDB
         run: |
           make
           sudo make install
 
       - name: Run Tests
-        run: |
+        run: |          
           export LC_ALL=en_US.UTF-8
           export LANGUAGE=en_US
           export LC_COLLATE=en_US.UTF-8

scripts/install_setup_pg_cron.sh

@@ -1,4 +1,3 @@
-
 #!/bin/bash
 
 # fail if trying to reference a variable that is not set.

scripts/start_oss_server.sh

@@ -38,11 +38,11 @@ reset=`tput sgr0`
 
 if [ "$help" == "true" ]; then
     echo "${green}sets up and launches a postgres server with extension installed on port $coordinatorPort."
-    echo "${green}start_oss_server -d <postgresDir> [-t <serverType>] [-c] [-s]"
+    echo "${green}start_oss_server -d <postgresDir> [-c] [-s]"
     echo "${green}<postgresDir> is the data directory for your postgres instance with extension"
     echo "${green}[-c] - optional argument. removes all existing data if it exists"
     echo "${green}[-s] - optional argument. Stops all servers and exits"
-    echo "${green}if postgresDir not specified assumed to be ~/${serverType}_test"
+    echo "${green}if postgresDir not specified assumed to be ~/documentdb_test"
     exit 1;
 fi