fix(php8): remove duplicate OPcache loader on PHP <= 8.4

[lussoluca]

🤖 This was written by an AI agent on behalf of @lussoluca.

Problem

Since e9b6e49 ("feat(php8): add PHP 8.5.7 and refresh image dependencies"), the 8.x image emits at every PHP invocation:

Cannot load Zend OPcache - it was already loaded

Reproduced on the 8.4.22-fpm-alpine3.24-rootless tag. The 8.4.2 tag (built before that commit) is clean.

Root cause

The official parent php:8.x-fpm-alpine image already enables OPcache via docker-php-ext-opcache.ini (zend_extension=opcache). Commit e9b6e49 added a second loader:

RUN if [ -f "$(php -r 'echo ini_get("extension_dir");')/opcache.so" ]; then \ echo "zend_extension=opcache.so" > /usr/local/etc/php/conf.d/00-opcache-load.ini; \ fi

On PHP <= 8.4 both loaders run → OPcache loads twice → the warning. On PHP 8.5 OPcache is statically compiled, opcache.so is absent, so the guard never created the file there. The block only ever fired on <= 8.4, exactly where the parent already loads OPcache — always redundant.

Fix

Remove the block. opcache.ini still tunes OPcache in both cases (it carries no zend_extension line); the parent loads it on <= 8.4 and it is static on 8.5.

Additional cleanup: drop dead opcache.fast_shutdown

8/conf/opcache.ini set opcache.fast_shutdown=1. That directive was removed in PHP 7.2 — fast shutdown became always-on and the option no longer exists. On PHP 8.4 it is absent from phpinfo() and ini_get("opcache.fast_shutdown") returns false, so the line is silently ignored dead config. The 8/ image only targets PHP 8.x, so it can never apply. Removed.

Verification

In the affected 8.4.22 image, two loaders are present:

/usr/local/etc/php/conf.d/00-opcache-load.ini:     zend_extension=opcache.so
/usr/local/etc/php/conf.d/docker-php-ext-opcache.ini: zend_extension=opcache

After this change only the parent's docker-php-ext-opcache.ini remains, OPcache stays enabled, and the warning is gone. Verified locally by building the PR branch (--build-arg PHPVER=8.4.22-fpm-alpine3.24, targets dist and dev): single loader, no warning, extension_loaded("Zend OPcache") → true, and opcache_get_status()["opcache_enabled"] → true with opcache.enable_cli=1.

Assisted-by: claude-code/claude-opus-4-8

[sparkfabrik-ai-bot]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review OPcache on PHP 8.5 After removing the block, verify that OPcache is properly enabled and functional on PHP 8.5 builds. The PR description states OPcache is statically compiled on 8.5, but it should be confirmed that the opcache.ini tuning file still applies correctly and OPcache is active without any additional loader configuration. COPY conf/*.ini /usr/local/etc/php/conf.d/ COPY conf.disabled /usr/local/etc/php/conf.disabled COPY fpm-conf-templates/ /templates/ # Make folders writable for the root group RUN chmod 775 /usr/local/etc/php && \ chmod 775 /usr/local/etc/php/conf.d && \ chmod 775 /usr/local/etc/php-fpm.d && \

[sparkfabrik-ai-bot]

PR Code Suggestions ✨

No code suggestions found for the PR.