Skip to content

Commit

Permalink
fix(ci): fix SQL Server 2017 (#5003)
Browse files Browse the repository at this point in the history
* fix(ci): add fix for SQL Server 2017

* fix(ci): fix schema-engine for SQL Server 2017

* fix(ci): forward ubuntu version to test-query-engine-template

* chore: trigger Rust tests

* chore(core-tests): undo changes to build.rs

* chore: trigger Rust tests
  • Loading branch information
jkomyno authored Sep 23, 2024
1 parent c9ff577 commit c57dd2b
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 3 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/test-query-engine-template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ on:
version:
type: string
required: true
ubuntu:
type: string
default: 'latest'
single_threaded:
type: boolean
default: false
Expand Down Expand Up @@ -46,7 +49,7 @@ jobs:
PRISMA_ENGINE_PROTOCOL: ${{ matrix.engine_protocol }}
PRISMA_RELATION_LOAD_STRATEGY: ${{ matrix.relation_load_strategy }}

runs-on: ubuntu-latest
runs-on: "ubuntu-${{ inputs.ubuntu }}"
steps:
- uses: actions/checkout@v4
- uses: actions-rust-lang/setup-rust-toolchain@v1
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/test-query-engine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -126,11 +126,13 @@ jobs:
version: "2019"
- name: "mssql_2017"
version: "2017"
ubuntu: "20.04"
uses: ./.github/workflows/test-query-engine-template.yml
name: mssql ${{ matrix.database.version }}
with:
name: ${{ matrix.database.name }}
version: ${{ matrix.database.version }}
ubuntu: ${{ matrix.database.ubuntu }}
connector: "sqlserver"
relation_load_strategy: '["query"]'

Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/test-schema-engine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,7 @@ jobs:
database:
- name: mssql_2017
url: "sqlserver://localhost:1434;database=master;user=SA;password=<YourStrong@Passw0rd>;trustServerCertificate=true;socket_timeout=60;isolationLevel=READ UNCOMMITTED"
ubuntu: "20.04"
- name: mssql_2019
url: "sqlserver://localhost:1433;database=master;user=SA;password=<YourStrong@Passw0rd>;trustServerCertificate=true;socket_timeout=60;isolationLevel=READ UNCOMMITTED"
- name: mssql_2022
Expand Down Expand Up @@ -104,7 +105,7 @@ jobs:
is_vitess: true
single_threaded: true

runs-on: ubuntu-latest
runs-on: "ubuntu-${{ matrix.database.ubuntu || 'latest' }}"
steps:
- uses: actions/checkout@v4
- uses: actions-rust-lang/setup-rust-toolchain@v1
Expand Down
2 changes: 1 addition & 1 deletion schema-engine/cli/tests/cli_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ where
.or_else(|| panic_payload.downcast_ref::<String>().map(|s| s.to_owned()))
.unwrap_or_default();

panic!("Error: '{}'", res)
panic!("Error: '{}'", res);
}
}
}
Expand Down

0 comments on commit c57dd2b

Please sign in to comment.