Skip to content

Fix: Add Yarn installation and correct Redis host configuration in init.sh#14

Open
3aluw wants to merge 1 commit intoankush:masterfrom
3aluw:master
Open

Fix: Add Yarn installation and correct Redis host configuration in init.sh#14
3aluw wants to merge 1 commit intoankush:masterfrom
3aluw:master

Conversation

@3aluw
Copy link

@3aluw 3aluw commented Oct 3, 2025

This PR closes #13
This PR fixes two issues in the init.sh setup script that were preventing a smooth ERPNext/Frappe setup.

Changes made

  1. Install Yarn automatically

    • Added npm install -g yarn after Node.js is set up.
    • Ensures bench build works without manual installation.
  2. Fix Redis host configuration

    • Removed the :6379 port suffix from Redis host values.
    • Correctly uses only the hostname (redis-cache, redis-queue, redis-socketio), since bench set-redis-*-host expects just the hostname.

Before

bench set-redis-cache-host redis-cache:6379
bench set-redis-queue-host redis-queue:6379
bench set-redis-socketio-host redis-socketio:6379

After

npm install -g yarn

bench set-redis-cache-host redis-cache
bench set-redis-queue-host redis-queue
bench set-redis-socketio-host redis-socketio

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.

Issues with init.sh: Yarn missing + incorrect Redis URLs

1 participant