Skip to content

Commit

Permalink
Merge 13de053 into backport/stevendpclark/better-logging-mssql-tests/…
Browse files Browse the repository at this point in the history
…eminently-present-snail
  • Loading branch information
hc-github-team-secure-vault-core authored Sep 23, 2024
2 parents 683fd73 + 13de053 commit 20f783f
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions helper/testhelpers/mssql/mssqlhelper.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,8 @@ func PrepareMSSQLTestContainer(t *testing.T) (cleanup func(), retURL string) {
var err error
for i := 0; i < numRetries; i++ {
var svc *docker.Service
runner, err := docker.NewServiceRunner(docker.RunOptions{
var runner *docker.Runner
runner, err = docker.NewServiceRunner(docker.RunOptions{
ContainerName: "sqlserver",
ImageRepo: "mcr.microsoft.com/mssql/server",
ImageTag: "2017-latest-ubuntu",
Expand All @@ -48,7 +49,8 @@ func PrepareMSSQLTestContainer(t *testing.T) (cleanup func(), retURL string) {
},
})
if err != nil {
t.Fatalf("Could not start docker MSSQL: %s", err)
t.Logf("Could not start docker MSSQL: %v", err)
continue
}

svc, err = runner.StartService(context.Background(), connectMSSQL)
Expand All @@ -57,7 +59,7 @@ func PrepareMSSQLTestContainer(t *testing.T) (cleanup func(), retURL string) {
}
}

t.Fatalf("Could not start docker MSSQL: %s", err)
t.Fatalf("Could not start docker MSSQL: %v", err)
return nil, ""
}

Expand Down

0 comments on commit 20f783f

Please sign in to comment.