Skip to content

Commit

Permalink
Revert "HHH-15533 Introduce versionless CI matrix entries for the Pos…
Browse files Browse the repository at this point in the history
…tgreSQL and MySQL minimum supported version"

This reverts commit aae3513.
  • Loading branch information
beikov committed Sep 21, 2022
1 parent f8c25fd commit 8302b06
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 15 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/contributor-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
- rdbms: derby
- rdbms: mysql8
- rdbms: mariadb
- rdbms: postgresql
- rdbms: postgresql_9_5
- rdbms: postgresql_13
- rdbms: oracle
- rdbms: db2
Expand Down
8 changes: 4 additions & 4 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ stage('Configure') {
// new BuildEnvironment( dbName: 'derby' ),
// new BuildEnvironment( dbName: 'mysql8' ),
// new BuildEnvironment( dbName: 'mariadb' ),
// new BuildEnvironment( dbName: 'postgresql' ),
// new BuildEnvironment( dbName: 'postgresql_9_5' ),
// new BuildEnvironment( dbName: 'postgresql_13' ),
// new BuildEnvironment( dbName: 'oracle' ),
new BuildEnvironment( dbName: 'oracle_ee' ),
Expand Down Expand Up @@ -130,12 +130,12 @@ stage('Build') {
sh "./docker_db.sh mariadb"
state[buildEnv.tag]['containerName'] = "mariadb"
break;
case "postgresql":
case "postgresql_9_5":
// use the postgis image to enable the PGSQL GIS (spatial) extension
docker.withRegistry('https://index.docker.io/v1/', 'hibernateci.hub.docker.com') {
docker.image('postgis/postgis:9.5-2.5').pull()
}
sh "./docker_db.sh postgresql"
sh "./docker_db.sh postgresql_9_5"
state[buildEnv.tag]['containerName'] = "postgres"
break;
case "postgresql_13":
Expand Down Expand Up @@ -197,7 +197,7 @@ stage('Build') {
case "tidb":
runTest("-Pdb=tidb -DdbHost=localhost:4000${state[buildEnv.tag]['additionalOptions']}", 'TIDB')
break;
case "postgresql":
case "postgresql_9_5":
case "postgresql_13":
runTest("-Pdb=pgsql_ci${state[buildEnv.tag]['additionalOptions']}")
break;
Expand Down
4 changes: 2 additions & 2 deletions README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ The script `docker_db.sh` allows you to start a pre-configured database which ca
All you have to do is run the following command:

----
./docker_db.sh postgresql
./docker_db.sh postgresql_9_5
----

omitting the argument will print a list of possible options.
Expand Down Expand Up @@ -174,7 +174,7 @@ The following table illustrates a list of commands for various databases that ca
|`./gradlew test -Pdb=mariadb_ci`

|PostgreSQL 9.5
|`./docker_db.sh postgresql`
|`./docker_db.sh postgresql_9_5`
|`./gradlew test -Pdb=pgsql_ci`

|PostgreSQL 13
Expand Down
2 changes: 1 addition & 1 deletion ci/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ elif [ "$RDBMS" == "mysql" ]; then
goal="-Pdb=mysql_ci"
elif [ "$RDBMS" == "mariadb" ]; then
goal="-Pdb=mariadb_ci"
elif [ "$RDBMS" == "postgresql" ]; then
elif [ "$RDBMS" == "postgresql_9_5" ]; then
goal="-Pdb=pgsql_ci"
elif [ "$RDBMS" == "postgresql_13" ]; then
goal="-Pdb=pgsql_ci"
Expand Down
4 changes: 2 additions & 2 deletions ci/database-start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ elif [ "$RDBMS" == 'mysql8' ]; then
bash $DIR/../docker_db.sh mysql_8_0
elif [ "$RDBMS" == 'mariadb' ]; then
bash $DIR/../docker_db.sh mariadb
elif [ "$RDBMS" == 'postgresql' ]; then
bash $DIR/../docker_db.sh postgresql
elif [ "$RDBMS" == 'postgresql_9_5' ]; then
bash $DIR/../docker_db.sh postgresql_9_5
elif [ "$RDBMS" == 'postgresql_13' ]; then
bash $DIR/../docker_db.sh postgresql_13
elif [ "$RDBMS" == 'db2' ]; then
Expand Down
5 changes: 0 additions & 5 deletions docker_db.sh
Original file line number Diff line number Diff line change
Expand Up @@ -86,10 +86,6 @@ mariadb() {
fi
}

postgresql() {
postgresql_9_5
}

postgresql_9_5() {
$CONTAINER_CLI rm -f postgres || true
$CONTAINER_CLI run --name postgres -e POSTGRES_USER=hibernate_orm_test -e POSTGRES_PASSWORD=hibernate_orm_test -e POSTGRES_DB=hibernate_orm_test -p5432:5432 -d docker.io/postgis/postgis:9.5-2.5
Expand Down Expand Up @@ -531,7 +527,6 @@ if [ -z ${1} ]; then
echo -e "\tpostgis"
echo -e "\tpostgresql_13"
echo -e "\tpostgresql_9_5"
echo -e "\tpostgresql"
echo -e "\tsybase"
else
${1}
Expand Down

0 comments on commit 8302b06

Please sign in to comment.