Skip to content

ci: pull phpfpm and its dependencies up front in workflow templates#147

Open
turegjorup wants to merge 1 commit into
developfrom
feature/gha-pull-images-upfront
Open

ci: pull phpfpm and its dependencies up front in workflow templates#147
turegjorup wants to merge 1 commit into
developfrom
feature/gha-pull-images-upfront

Conversation

@turegjorup

Copy link
Copy Markdown
Contributor

Pull the phpfpm service and its dependencies up front with docker compose pull --quiet --include-deps phpfpm, so the images download in parallel before the job runs instead of lazily during docker compose run.

Changes

  • Add a "Pull container images" step to the workflow templates whose jobs run in phpfpm:
    • github/workflows/composer.yaml (validate, normalize, audit)
    • github/workflows/twig.yaml
    • github/workflows/drupal/php.yaml
    • github/workflows/drupal-module/php.yaml
    • github/workflows/symfony/php.yaml

Why

phpfpm declares depends_on: mariadb (and memcached for Drupal), so docker compose run --rm phpfpm … brings up several services. Pulling with --include-deps up front fetches those images in parallel before the run, for faster and cleaner jobs.

Notes

  • Only applied where it helps. The Prettier/markdownlint workflows (YAML, Markdown, JavaScript, Styles) run a single dependency-less image, so an up-front pull would add nothing over the pull run already does — left unchanged.
  • github/workflows/drupal/site.yaml already pulls the full stack up front (docker compose pull) and is unchanged.
  • Mirrors the up-front pull strategy adopted in ci: cache vendor, pre-pull images, drop Task from Review workflow event-database-imports#98.

@turegjorup turegjorup self-assigned this Jul 8, 2026
@turegjorup turegjorup requested a review from rimi-itk July 8, 2026 09:49
@turegjorup turegjorup force-pushed the feature/gha-pull-images-upfront branch from dc041bd to 2df739c Compare July 8, 2026 09:54
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.

1 participant