-
-
Notifications
You must be signed in to change notification settings - Fork 478
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(mssql): bump Docker image version #2786
Conversation
Disable all mssql tests as the container is currently crashing, see #2785
✅ Deploy Preview for testcontainers-go ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
The lint failure is handled by #2787 |
@stevenh I've updated the PR adding the changes proposed by @HofmeisterAn in the .NET project (thank you!) |
@@ -128,25 +128,3 @@ func TestMSSQLServerWithInvalidPassword(t *testing.T) { | |||
testcontainers.CleanupContainer(t, ctr) | |||
require.NoError(t, err) | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm removing this test because it's doing exactly the same as another one, using an image that at the moment has no alternative. We can remove, or skip, whatever you prefer.
Tests are passing again! https://github.com/testcontainers/testcontainers-go/actions/runs/10949418982/job/30403422765?pr=2786 |
@stevenh I'm merging this to unblock other PRs. PLMK anything you consider about this. We can discuss in a follow up PR. Thanks! |
* main: feat: support databend module (#2779) chore: golangci-lint 1.61.0 (#2787) fix(mssql): bump Docker image version (#2786) fix: handle 127 error code for podman compatibility (#2778) fix: do not override ImageBuildOptions.Labels when building from a Dockerfile (#2775) feat(mongodb): Wait for mongodb module with a replicaset to finish (#2777) fix(postgres): Apply default snapshot name if no name specified (#2783)
Disable all mssql tests as the container is currently crashing, see #2785This PR uses the MSSQL image that does not crash with the current update in the GH workers.
Related issues