Skip to content

Commit

Permalink
ci(ci.sh): fix Docker Compose presence check - migrate to sub-command
Browse files Browse the repository at this point in the history
1. The old way to use docker compose was through the standalone binary
`docker-compose`
2. This was working for a while but now the auto-upgrades that we cannot
seem to avoid have caught up with us and broke ci.sh in the GitHub action
runners because the standalone binary is no longer available at all and
therefore the migration must happen.
3. Point 2 is just a theory but one that is considered to be very likely
correct.
4. It is to be seen if we'll have any other downstream issues such as the
tests failing in other ways due to this underlying docker change.

Signed-off-by: Peter Somogyvari <[email protected]>
  • Loading branch information
petermetz committed Apr 3, 2024
1 parent ab9eedf commit f71c48e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ function mainTask()
fi

docker --version
docker-compose --version
docker compose version
node --version
npm --version
java -version
Expand Down

0 comments on commit f71c48e

Please sign in to comment.