Skip to content

Commit

Permalink
Bring back testing with MySQL
Browse files Browse the repository at this point in the history
The original reason was after the `rank` becoming a protected keyword. As we depend on doctrine as abrstraction a lot it is good to verify that the translation is reasonable on backend SQL servers of whatever dialect.
  • Loading branch information
Michael Vasseur committed Nov 24, 2024
1 parent 32b8f12 commit 2899dcb
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,19 +14,24 @@ jobs:
strategy:
matrix:
SQL_ISOLATION: ['REPEATABLE-READ', 'SERIALIZABLE']
include:
- sql: mariadb
sql_options: --health-cmd="healthcheck.sh --connect --innodb_initialized" --health-interval=10s --health-timeout=5s --health-retries=3
- sql: mysql
sql_options: --health-cmd="mysqladmin ping" --health-interval=5s --health-timeout=2s --health-retries=3
container:
image: domjudge/gitlabci:24.04
options: --privileged --cgroupns=host --init
services:
sqlserver:
image: mariadb
image: ${{ matrix.sql }}
options: ${{ matrix.sql_options }}
ports:
- 3306:3306
env:
MYSQL_ROOT_PASSWORD: root
MYSQL_USER: domjudge
MYSQL_PASSWORD: domjudge
options: --health-cmd="healthcheck.sh --connect --innodb_initialized" --health-interval=10s --health-timeout=5s --health-retries=3
steps:
- uses: actions/checkout@v4
- name: info
Expand Down Expand Up @@ -86,13 +91,13 @@ jobs:
if: ${{ !cancelled() }}
uses: actions/upload-artifact@v4
with:
name: DB-dump-${{ matrix.SQL_ISOLATION }}
name: DB-dump-${{ matrix.SQL_ISOLATION }}-${{ matrix.sql }}
path: /tmp/db.sql
- name: Upload all logs/artifacts
if: ${{ !cancelled() }}
uses: actions/upload-artifact@v4
with:
name: Logs-${{ matrix.SQL_ISOLATION }}
name: Logs-${{ matrix.SQL_ISOLATION }}-${{ matrix.sql }}
path: |
/var/log/nginx
/opt/domjudge/domserver/webapp/var/log/*.log
Expand Down

0 comments on commit 2899dcb

Please sign in to comment.