Skip to content

Conversation

@mdelapenya
Copy link
Member

What does this PR do?

It adds a Go env var to the shell the tests are run in order to mitigate the recent flakiness we are seeing when parsing the TLS certs in the MSSQL server container:

TLS Handshake failed: tls: failed to parse certificate from server: x509: negative serial number

The PR is setting the env var in the Makefile for MSSQL.

Why is it important?

Reduce flakiness

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner April 2, 2025 15:26
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Apr 2, 2025
@mdelapenya mdelapenya self-assigned this Apr 2, 2025
@mdelapenya mdelapenya requested a review from stevenh April 2, 2025 15:26
@mdelapenya mdelapenya changed the title fix: reduce flakiness in MSSQL tests fix(mssql): reduce flakiness in tests Apr 2, 2025
@netlify
Copy link

netlify bot commented Apr 2, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 600af4d
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/67ee48377d3d430008b83f66
😎 Deploy Preview https://deploy-preview-3084--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mdelapenya mdelapenya merged commit 3bdcef9 into testcontainers:main Apr 3, 2025
16 checks passed
@mdelapenya mdelapenya deleted the mssql-flakiness branch April 3, 2025 08:39
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Apr 3, 2025
* main:
  fix(mssql): reduce flakiness in tests (testcontainers#3084)
  chore: bump golangci-lint to v2 (testcontainers#3082)
  chore(gcloud): deprecate old gcp containers, creating subpackages for them (testcontainers#3063)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Apr 3, 2025
* main:
  fix(mssql): reduce flakiness in tests (testcontainers#3084)
  chore: bump golangci-lint to v2 (testcontainers#3082)
  chore(gcloud): deprecate old gcp containers, creating subpackages for them (testcontainers#3063)
  fix(mongodb): replica set initialization & connection handling (testcontainers#2984)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Apr 3, 2025
* main:
  fix(mssql): reduce flakiness in tests (testcontainers#3084)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Apr 14, 2025
* main: (91 commits)
  chore(deps): bump github/codeql-action from 3.28.13 to 3.28.15 (testcontainers#3097)
  chore(deps): bump golang.org/x/crypto from 0.31.0 to 0.37.0 (testcontainers#3098)
  feat(aerospike): add Aerospike module (testcontainers#3094)
  security(compose): upgrade github.com/docker/compose/v2 to fix security vulnerability (testcontainers#3095)
  feat: add more functional options to the modules API (testcontainers#3070)
  chore(deps): bump golang.org/x/net in /modules/arangodb (testcontainers#3087)
  feat: add arangodb module (testcontainers#3083)
  chore(deps): bump actions/upload-artifact from 4.6.0 to 4.6.2 (testcontainers#3086)
  chore(deps): bump SonarSource/sonarqube-scan-action from 5.0.0 to 5.1.0 (testcontainers#3085)
  feat: add socat container (testcontainers#3071)
  fix(mssql): reduce flakiness in tests (testcontainers#3084)
  chore: bump golangci-lint to v2 (testcontainers#3082)
  chore(gcloud): deprecate old gcp containers, creating subpackages for them (testcontainers#3063)
  fix(mongodb): replica set initialization & connection handling (testcontainers#2984)
  chore(deps): bump docker/setup-docker-action from 4.2.0 to 4.3.0 (testcontainers#3077)
  chore(deps): bump github/codeql-action from 3.28.12 to 3.28.13 (testcontainers#3078)
  chore(deps): bump tj-actions/changed-files from 45.0.4 to 46.0.3 (testcontainers#3076)
  docs: add dependabot configuration (testcontainers#3074)
  chore(deps): replace `golang.org/x/exp/slices` with stdlib (testcontainers#3075)
  fix(dind): use docker image load (testcontainers#3073)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Changes that do not impact the existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant