Skip to content
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

docker: move to Docker compose v2 #1460

Merged

Conversation

Bastian-Krause
Copy link
Member

@Bastian-Krause Bastian-Krause commented Aug 1, 2024

Description
Docker compose v1 has been deprecated since July 2023 [1]. Now the GitHub actions Ubuntu runner images removed it, too [2]. So move to v2.

labgrid is obviously not affected by any changes between v1 and v2 other than the compose call docker-compose -> docker compose.

While at it, drop the obsolete version element [3] from the docker compose config.

[1] https://docs.docker.com/compose/migrate/
[2] actions/runner-images#9692
[3] https://docs.docker.com/compose/compose-file/04-version-and-name/#version-top-level-element-optional

Checklist

  • PR has been tested (CI only)

Docker compose v1 has been deprecated since July 2023 [1]. Now the
GitHub actions Ubuntu runner images removed it, too [2]. So move to v2.

labgrid is obviously not affected by any changes between v1 and v2 other
than the compose call docker-compose -> docker compose.

[1] https://docs.docker.com/compose/migrate/
[2] actions/runner-images#9692

Signed-off-by: Bastian Krause <[email protected]>
@Bastian-Krause
Copy link
Member Author

@hoyes Since you've contributed #1307 before, could you take a look at this?

@Emantor Emantor merged commit 476ca9e into labgrid-project:master Aug 6, 2024
9 checks passed
@hoyes
Copy link
Contributor

hoyes commented Aug 6, 2024

LGTM!

@Bastian-Krause
Copy link
Member Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants