Skip to content

Fix: bundle Redis in CE Docker image and fix oracledb plugin lockfile#15900

Merged
gsmithun4 merged 9 commits intolts-3.16from
fix/ce-redis
Apr 13, 2026
Merged

Fix: bundle Redis in CE Docker image and fix oracledb plugin lockfile#15900
gsmithun4 merged 9 commits intolts-3.16from
fix/ce-redis

Conversation

@adishM98
Copy link
Copy Markdown
Collaborator

@adishM98 adishM98 commented Apr 13, 2026

Problem

Two issues were blocking CE single-instance deployments:

  1. CE image failed to boot with ECONNREFUSED 127.0.0.1:6379 — BullMQ and CustomDomainCacheService both connect to Redis on startup. CE docs only mention external
    Redis for multi-pod setups, but single-instance CE also requires it. Rather than requiring operators to run a Redis sidecar, we now bundle it in the image.
  2. Docker build failed during lerna run build — the oracle wallet PR (added wallet connectivity to oracle. #15771) added unzipper as a dependency to @tooljet-plugins/oracledb but didn't update
    plugins/package-lock.json. The Dockerfile installs from the lockfile, so unzipper was never installed and tsc failed with a missing module error.

Changes

docker/ce-production.Dockerfile

  • Added redis-server to the apt-get install step
  • Created /var/lib/redis and /var/log/redis dirs with correct appuser ownership — merged into the useradd block so appuser exists before chown runs
  • Wrote /app/redis.conf (localhost-only, daemonized) under the appuser-owned /app path to avoid /etc/redis permission issues

docker/ce-entrypoint.sh

  • Starts redis-server (daemonized via conf) at container boot before DB wait/migrations

plugins/package-lock.json

  • Regenerated to include unzipper@0.12.3 and its transitive deps (duplexer2, bluebird)

@adishM98 adishM98 requested a review from slancerk as a code owner April 13, 2026 07:49
@adishM98 adishM98 changed the title Fix: Enhance Redis connection handling in CustomDomainCacheService Fix: bundle Redis in CE Docker image and fix oracledb plugin lockfile Apr 13, 2026
ganesh8056
ganesh8056 previously approved these changes Apr 13, 2026
Copy link
Copy Markdown
Collaborator

@ganesh8056 ganesh8056 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved for Plugins changes

@gsmithun4 gsmithun4 merged commit 2448f8b into lts-3.16 Apr 13, 2026
57 checks passed
@gsmithun4 gsmithun4 deleted the fix/ce-redis branch April 13, 2026 17:38
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.

3 participants