Skip to content

chore: update Elasticsearch image and remove unused RabbitMQ - #72

Merged
turegjorup merged 3 commits into
developfrom
feature/docker-images-update
Nov 8, 2026
Merged

turegjorup merged 3 commits into
developfrom
feature/docker-images-update

Conversation

@turegjorup

Copy link
Copy Markdown
Contributor

Update the images that the ITK Dev template does not control. This is step 2 of the quarterly update.

Changes

  • Update elasticsearch from 8.19.18 to 8.19.23 in docker-compose.override.yml. This is the latest 8.19 patch, and the same version as in event-database-imports.
  • Remove the rabbit service (rabbitmq:3.12.4-management-alpine) and the depends_on of phpfpm from docker-compose.override.yml.
  • Remove rabbit from the comments in pr.yaml and api-spec.yml.

No change for these images:

  • itkdev/markdownlint and jauderho/prettier have no tag (latest).
  • itkdev/ansible-plugin:1 in .woodpecker/ uses a major-version tag.

Why

The API does not use RabbitMQ. There is no Messenger or AMQP configuration, and no code uses it. An update from 3.12 to 4 would update a service that nothing uses, so this PR removes the service.

The elasticsearch service is only for development and tests. Production uses the shared Elasticsearch cluster.

Checks

  • PHPUnit on PHP 8.5.12 against Elasticsearch 8.19.23: all tests pass.
  • Prettier (YAML): no changes.

After the merge

  • Run docker compose up --detach --wait --remove-orphans locally, to remove the old rabbit container.

Elasticsearch 8.19.23 is the same version as in event-database-imports. The API does not use RabbitMQ (no messenger or amqp).
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.39%. Comparing base (76b8e6f) to head (9543626).
⚠️ Report is 23 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop      #72   +/-   ##
==========================================
  Coverage      74.39%   74.39%           
  Complexity       264      264           
==========================================
  Files             30       30           
  Lines            738      738           
==========================================
  Hits             549      549           
  Misses           189      189           
Flag Coverage Δ
unittests 74.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@turegjorup
turegjorup merged commit 2cfec40 into develop Nov 8, 2026
9 checks passed
@turegjorup
turegjorup deleted the feature/docker-images-update branch November 8, 2026 20:24
@turegjorup turegjorup mentioned this pull request Nov 8, 2026
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