Skip to content

fix: compare docker images with latest release tag - #1736

Merged
dunglas merged 3 commits into
mainfrom
fix/rebuild-docker-images-based-on-latest-release
Jul 9, 2025
Merged

fix: compare docker images with latest release tag#1736
dunglas merged 3 commits into
mainfrom
fix/rebuild-docker-images-based-on-latest-release

Conversation

@AlliBalliBaba

Copy link
Copy Markdown
Contributor

This should fix #1730 by always comparing upstream docker images to the actual latest release tag instead of the 'latest' dunglas/frankenphp.

(assuming this works as intended, since I can't test it locally)

The issue is that currently the 'latest' Docker version is 1.7, while the latest release is 1.8. This leads to the CI not upgrading the 1.8 version.

@AlliBalliBaba
AlliBalliBaba requested a review from dunglas July 9, 2025 16:23
@dunglas
dunglas merged commit d5544bb into main Jul 9, 2025
48 checks passed
@dunglas
dunglas deleted the fix/rebuild-docker-images-based-on-latest-release branch July 9, 2025 21:41
@dunglas

dunglas commented Jul 9, 2025

Copy link
Copy Markdown
Member

Thank you!!

@jduan00

jduan00 commented Jul 9, 2025

Copy link
Copy Markdown

When will new docker images be built? e.g. dunglas/frankenphp:php8.4

Thanks!

@AlliBalliBaba

Copy link
Copy Markdown
Contributor Author

The new images (1.8.0 with PHP 8.4.10) should now be on Docker Hub

@PhilETaylor

PhilETaylor commented Jul 10, 2025

Copy link
Copy Markdown

Confirmed

~ docker run --rm -it dunglas/frankenphp:1.8.0-php8.4 frankenphp -v
FrankenPHP v1.8.0 PHP 8.4.10 Caddy v2.10.0 h1:fonubSaQKF1YANl8TXqGcn4IbIRUDdfAkpcsfI/vX5U=

~ docker run --rm -it dunglas/frankenphp:1.8.0 frankenphp -v
FrankenPHP v1.8.0 PHP 8.4.10 Caddy v2.10.0 h1:fonubSaQKF1YANl8TXqGcn4IbIRUDdfAkpcsfI/vX5U=

~ docker run --rm -it dunglas/frankenphp:latest frankenphp -v
FrankenPHP v1.8.0 PHP 8.4.10 Caddy v2.10.0 h1:fonubSaQKF1YANl8TXqGcn4IbIRUDdfAkpcsfI/vX5U=

Thanks!

@jduan00

jduan00 commented Jul 10, 2025

Copy link
Copy Markdown

Confirmed. Thanks!

@PhilETaylor

Copy link
Copy Markdown

Sorry but this introduced an issue - details: #1742

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.

Docker images confusion...

4 participants