Skip to content

chore(deps): bump docker dependencies to v24.0.5+incompatible#1394

Merged
mdelapenya merged 14 commits intotestcontainers:mainfrom
mdelapenya:docker-upgrade
Aug 2, 2023
Merged

chore(deps): bump docker dependencies to v24.0.5+incompatible#1394
mdelapenya merged 14 commits intotestcontainers:mainfrom
mdelapenya:docker-upgrade

Conversation

@mdelapenya
Copy link
Copy Markdown
Member

@mdelapenya mdelapenya commented Aug 1, 2023

What does this PR do?

It bumps the core library and all the modules to Docker v24.0.5+incompatible.

Besides that, we were forced to introduce a breaking change, as this PR updates the references to the deprecated Docker's types.AuthConfig, using Docker's registry.AuthConfig instead. The code that was affected was the one related to building containers from a Dockerfile. We will update the release notes in the upcoming release.

As a consequence of the Docker bump in the compose module, we were forced to bump certain dependencies in the compose module that needs Go 1.20, therefore, we are temporarily retiring the GH workflow for running the compose module using Go 1.19. Once Go 1.21 is out, we will update that file to include the classic two-version window for compiling the project. A positive change here is that the replace directive when using the compose module has been simplified.

Why is it important?

See #1359 and how the Go versions broke how the host headers are validated by Docker and us.

Related issues

@mdelapenya mdelapenya self-assigned this Aug 1, 2023
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 1, 2023

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 2250836
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/64c9fc2ef721cd0008f63607
😎 Deploy Preview https://deploy-preview-1394--testcontainers-go.netlify.app/features/docker_compose
📱 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.

…mpatible

It will avoid the deprecation of types.AuthConfig, causing a breaking change in the library
@mdelapenya mdelapenya changed the title chore(deps): bump docker dependencies to v24.0.5+incompatible chore(deps): bump docker dependencies to v23.0.7-0.20230718082441-f860ed7c77fc+incompatible Aug 1, 2023
All consumers building from a Dockerfile must check that they are not affected
@mdelapenya mdelapenya changed the title chore(deps): bump docker dependencies to v23.0.7-0.20230718082441-f860ed7c77fc+incompatible chore(deps): bump docker dependencies to v24.0.5+incompatible Aug 1, 2023
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Aug 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mdelapenya mdelapenya added the breaking change Causing compatibility issues. label Aug 2, 2023
@mdelapenya mdelapenya marked this pull request as ready for review August 2, 2023 07:46
@mdelapenya mdelapenya requested a review from a team as a code owner August 2, 2023 07:46
@mdelapenya mdelapenya added the dependencies Dependencies or external services label Aug 2, 2023
@mdelapenya mdelapenya merged commit 659a096 into testcontainers:main Aug 2, 2023
@mdelapenya mdelapenya deleted the docker-upgrade branch August 2, 2023 09:48
@davideme
Copy link
Copy Markdown

davideme commented Aug 3, 2023

Any ETA on a release that include the fix?

@mdelapenya
Copy link
Copy Markdown
Member Author

Any ETA on a release that include the fix?

Hi @davideme I'm planning it for the end of this week, so hopefully we can have it tomorrow 🤞

cuppett added a commit to cuppett/mysql-dba-operator that referenced this pull request Sep 7, 2023
cuppett added a commit to cuppett/mysql-dba-operator that referenced this pull request Sep 10, 2023
Resolves #33: Adds character set and collation validation & constraints to CRs

* Adding basic fields for character set and collation
* Allowing collation to be null
* Populate AvailabeCharSets struct and add some tests for nonEmpty
* Refactor adminConnection to ServerAdminConnection in controller tests
* Bumps to fix testcontainers with newer golang

See also: testcontainers/testcontainers-go#1394
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Causing compatibility issues. dependencies Dependencies or external services

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: go 1.20.6: "http: invalid Host header, host port waiting failed"

3 participants