Skip to content

normalize usage of docker-compose#774

Closed
ionous wants to merge 1 commit intoshift-org:mainfrom
ionous:compose-cmd-fixup
Closed

normalize usage of docker-compose#774
ionous wants to merge 1 commit intoshift-org:mainfrom
ionous:compose-cmd-fixup

Conversation

@ionous
Copy link
Copy Markdown
Contributor

@ionous ionous commented Jul 8, 2024

this change makes it so that macos uses the same "dash" compose command as production. ( docker-compose )

this is a "patch" until we can upgrade production. ( re: #772 ) once that's done, then this should be converted to the "space" version ( docker compose )

( even though the dash version is deprecated, the "dash" version works on macos with current versions of docker. the "space" version doesn't yet exist on production )

i've tested this on apple silicon with local Docker version 26.1.1, build 4cf5afa. and i've tested this previously on my older intel mac with ... a relatively recent version of docker.

( eta: production is 24.0.5 - 2023-07-24 )

see also this thread from @carrythebanner

We originally did this docker compose vs docker-compose dance to work around some other failures, but that was in 2021/2022 and Docker v20.10 — see commit notes here:
32d9bd2
7ab6a19

this change makes it so that macos uses the same compose command as production.

this is a "patch" until we can upgrade production. once that's done, then this should be converted to "docker compose" for all platforms. ( the "dash" version is deprecated, but the "space" version doesn't yet exist on production  )

i've tested this on apple silicon with local Docker version 26.1.1, build 4cf5afa. and i've tested this previously on my older intel mac with ... a relatively recent version of docker.
@ionous ionous marked this pull request as draft July 30, 2024 01:57
@ionous
Copy link
Copy Markdown
Contributor Author

ionous commented Jul 30, 2024

the very latest version of docker removes the docker-compose command.
some clever will be needed until we can upgrade docker on production

@nolessafool
Copy link
Copy Markdown
Member

I'm trying to get some motion on #772 in the coming weeks to unblock this. This feels like a good "off season" upgrade while we can afford to break production :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants