Skip to content

IBX-10495: [Backport] Included Elasticsearch 8 on CI#55

Merged
micszo merged 5 commits into4.6from
ibx-10495-4.6
Feb 2, 2026
Merged

IBX-10495: [Backport] Included Elasticsearch 8 on CI#55
micszo merged 5 commits into4.6from
ibx-10495-4.6

Conversation

@micszo
Copy link
Contributor

@micszo micszo commented Jan 23, 2026

🎫 Issue IBX-10495

Related PRs:

Description:

Backport of #54 to 4.6.

Added new file for Elasticsearch 8 (8.19.9).

Fixed Dockerfiles for Node (12, 14, 16, 18) to resolve issues with building and testing images on CI.

For QA:

Please see:

Documentation:

@micszo micszo marked this pull request as ready for review January 28, 2026 15:48
@micszo micszo requested a review from a team January 29, 2026 13:12
@micszo micszo merged commit 125a999 into 4.6 Feb 2, 2026
14 checks passed
@micszo micszo deleted the ibx-10495-4.6 branch February 2, 2026 11:05
@kossuth-wt
Copy link

kossuth-wt commented Mar 19, 2026

@micszo is there a reason why yarn@stable is used in ‎php/Dockerfile-node18? (The other dockerfiles have yarn@1.22.22). This caused in one of our projects gitlab CI fail because yarn@4.12.0 was fetched as the "stable", but in the project "current global version of Yarn is 1.22.22".

We use ghcr.io/ibexa/docker/php:8.3-node18 as our PHP image. We had to add this to our Dockerfile-app to make the project use yarn 1.22.22 and not fail on install.

RUN apt-get update -q -y \
&& apt-get install -q -y --no-install-recommends gnupg \
&& curl -sL https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add - \
&& echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list \
&& sudo apt-get update && sudo apt-get install yarn=1.22.22-1 \
&& rm -rf /var/lib/apt/lists/*

https://doc.ibexa.co/en/latest/getting_started/requirements/#asset-manager also states 1.x version is used.

@micszo
Copy link
Contributor Author

micszo commented Mar 19, 2026

Hi @kossuth-wt, this wasn't intended and must have slipped in unnoticed during development. I will fix this. Thanks for reporting!

@kossuth-wt
Copy link

@micszo Another thing is that two dockerfiles use npm install and two uses corepack to install yarn, that also might worth a look.

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.

5 participants