Skip to content

chore: use "docker compose" (v2) instead of "docker-compose" (v1)#2464

Merged
mdelapenya merged 3 commits intotestcontainers:mainfrom
mdelapenya:docker-compose-v2
Apr 3, 2024
Merged

chore: use "docker compose" (v2) instead of "docker-compose" (v1)#2464
mdelapenya merged 3 commits intotestcontainers:mainfrom
mdelapenya:docker-compose-v2

Conversation

@mdelapenya
Copy link
Member

  • chore: migrate from docker-compose (v1) to compose (v2)
  • chore: lowercase error message
  • chore: lint

What does this PR do?

This PR changes the implementation of the local docker compose support to shell-out to the docker binary, with compose as first argument, instead of to the docker-compose binary.

At the same time, we are deprecating all the methods and types from the LocalDockerCompose implementation.

Why is it important?

compose v1 was deprecated back in July '23, and the Github workers are not installing docker-compose in the workers by default.

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner April 3, 2024 07:32
@mdelapenya mdelapenya added chore Changes that do not impact the existing functionality compose Docker Compose. labels Apr 3, 2024
@mdelapenya mdelapenya self-assigned this Apr 3, 2024
@netlify
Copy link

netlify bot commented Apr 3, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 55d2320
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/660d06062a544c000852c601
😎 Deploy Preview https://deploy-preview-2464--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.

Copy link

@glours glours left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 😉

@mdelapenya mdelapenya merged commit f599979 into testcontainers:main Apr 3, 2024
@mdelapenya mdelapenya deleted the docker-compose-v2 branch April 3, 2024 08:25
mdelapenya added a commit to gustavosbarreto/testcontainers-go that referenced this pull request Apr 3, 2024
* main:
  bug:Fix AMQPS url (testcontainers#2462)
  chore: more compose updates in comments
  chore: use "docker compose" (v2) instead of "docker-compose" (v1) (testcontainers#2464)
  chore(deps): bump github/codeql-action from 2.22.12 to 3.24.9 (testcontainers#2459)
mdelapenya added a commit to coffeegoddd/testcontainers-go that referenced this pull request Apr 12, 2024
* main: (115 commits)
  chore: create TLS certs in a consistent manner (testcontainers#2478)
  chore(deps): bump idna from 3.6 to 3.7 (testcontainers#2480)
  Elasticsearch disable CA retrieval when ssl is disabled (testcontainers#2475)
  fix: handle dockerignore exclusions properly (testcontainers#2476)
  chore: prepare for next minor development cycle (0.31.0)
  chore: use new version (v0.30.0) in modules and examples
  Fix url creation to handle query params when using HTTP wait strategy (testcontainers#2466)
  fix: data race on container run (testcontainers#2345)
  fix: logging deadlock (testcontainers#2346)
  feat(k6):Add remote test scripts (testcontainers#2350)
  feat: optimizes file copies to and from containers (testcontainers#2450)
  fix(exec): updates the `Multiplexed` opt to combine stdout and stderr (testcontainers#2452)
  Upgrade neo4j module to use features from v0.29.1 of testcontainers-go (testcontainers#2463)
  bug:Fix AMQPS url (testcontainers#2462)
  chore: more compose updates in comments
  chore: use "docker compose" (v2) instead of "docker-compose" (v1) (testcontainers#2464)
  chore(deps): bump github/codeql-action from 2.22.12 to 3.24.9 (testcontainers#2459)
  refactor: Add Weaviate modules tests (testcontainers#2447)
  feat(exitcode): Add exit code sugar method (testcontainers#2342)
  feat: add module to support InfluxDB v1.x (testcontainers#1703)
  ...
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 compose Docker Compose.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants