Commit 7e78b12
committed
Fix mssql containers setup
Failures could be seen both on the developer machine and on CI.
Microsoft has changed the path to the sqlcmd utility in the latest releases of the image.
Besides, they have made connecting on an encrypted connection with sqlcmd mandatory by default.
Therefore, we must configure the utility to make this optional (-No).
Also, when now certificate are always validated but test servers use self-signed certificates.
For this reason, the utility must not try to validate the server certificate (-C).
Lastly, unlike the mcr.microsoft.com/mssql/server image, the mcr.microsoft.com/mssql-tools image has not been updated.
So the latest image release still uses the old sqlcmd utility path and options.
To avoid discrepancies in our setup, we'll use the mcr.microsoft.com/mssql/server image everywhere from now on.
We don't care too much about the size, because we need this image to run the server anyway.
Signed-off-by: Thomas Segismont <tsegismont@gmail.com>1 parent bbc049e commit 7e78b12
3 files changed
Lines changed: 8 additions & 7 deletions
File tree
- vertx-mssql-client
- docker
- src/test/java/io/vertx/mssqlclient/junit
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
| 13 | + | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
| 16 | + | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
| 15 | + | |
Lines changed: 3 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
115 | 115 | | |
116 | 116 | | |
117 | 117 | | |
118 | | - | |
| 118 | + | |
119 | 119 | | |
120 | 120 | | |
121 | 121 | | |
122 | | - | |
| 122 | + | |
| 123 | + | |
123 | 124 | | |
124 | 125 | | |
125 | 126 | | |
| |||
0 commit comments