Skip to content

Commit

Permalink
Include PostgreSQL in the testing matrix
Browse files Browse the repository at this point in the history
  • Loading branch information
ekohl authored and evgeni committed Nov 23, 2023
1 parent 33dcbbb commit 7484c4a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 7 deletions.
1 change: 1 addition & 0 deletions .github/matrix.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{
"postgresql": ["12"],
"ruby": ["2.7"],
"node": ["14"]
}
12 changes: 5 additions & 7 deletions .github/workflows/foreman.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,21 +21,19 @@ jobs:
id: build_matrix
uses: theforeman/gha-matrix-builder@v0
test:
name: "Ruby ${{ matrix.ruby }} and Node ${{ matrix.node }}"
name: "Ruby ${{ matrix.ruby }} and Node ${{ matrix.node }} on PostgreSQL ${{ matrix.postgresql }}"
runs-on: ubuntu-latest
needs: setup_matrix
services:
postgres:
image: 'postgres:12'
image: 'postgres:${{ matrix.postgresql }}'
ports: ['5432:5432']
options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
env:
POSTGRES_PASSWORD: password
strategy:
fail-fast: false
matrix:
ruby: ${{ fromJson(needs.setup_matrix.outputs.matrix).ruby }}
node: ${{ fromJson(needs.setup_matrix.outputs.matrix).node }}
matrix: ${{ fromJson(needs.setup_matrix.outputs.matrix) }}
steps:
- run: sudo apt-get update
- run: sudo apt-get -qq -y install build-essential libcurl4-openssl-dev zlib1g-dev libpq-dev libvirt-dev
Expand All @@ -49,10 +47,10 @@ jobs:
- name: Archive Gemfile.lock
uses: actions/upload-artifact@v3
with:
name: Gemfile.lock
name: Gemfile-ruby-${{ matrix.ruby }}-node-${{ matrix.node }}-pg-${{ matrix.postgresql }}.lock
path: Gemfile.lock
- name: "Set up Node ${{ matrix.node }}"
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
- name: Prepare test env
Expand Down

0 comments on commit 7484c4a

Please sign in to comment.