Skip to content

Release initial batch of Halo Box Playbooks#326

Open
danielholanda wants to merge 5 commits into
mainfrom
dholanda/hb_release
Open

Release initial batch of Halo Box Playbooks#326
danielholanda wants to merge 5 commits into
mainfrom
dholanda/hb_release

Conversation

@danielholanda
Copy link
Copy Markdown
Collaborator

Description

Releasing the following:

  • comfyui-image-gen
  • lmstudio-rocm-llms
  • n8n-automation-gpt-oss
  • pytorch-rocm-llms
  • vscode-qwen3-coder
  • lemonade-getting-started
  • pytorch-finetuning
  • speech2speech-translation
  • unsloth-llms-finetuning

@sreeram-11 sreeram-11 self-requested a review June 2, 2026 17:36
@danielholanda danielholanda self-assigned this Jun 2, 2026
@sreeram-11 sreeram-11 self-requested a review June 2, 2026 17:56
@sreeram-11 sreeram-11 marked this pull request as ready for review June 2, 2026 19:29
@sreeram-11
Copy link
Copy Markdown
Collaborator

sreeram-11 commented Jun 3, 2026

@danielholanda,

I added a commit here that fixes the n8n --version check on the Linux Halo runners.

On Linux Halo runners, there were two Node.js installations:

  1. /usr/bin/node -> v24.15.0 (package-managed Node.js installation from NodeSource)
  2. /usr/local/bin/node -> v20.19.2 (older manual Node.js installation)

Background

  • The GitHub Actions shell was picking up /usr/local/bin/node first because /usr/local/bin appears before /usr/bin in PATH
  • So earlier, n8n --version was passing because it was using the n8n installation associated with the older Node.js v20.19.2
  • After removing the older manual Node.js installation, the runner correctly used Node.js v24.15.0
  • However, the n8n installation for this Node.js setup is located at: /home/user/.npm-global/bin/n8n
  • The GitHub Actions shell did not include this directory in PATH, so n8n --version started failing even though n8n was installed

Changes

  • Cleaned up the Linux Halo runners to use the package-managed Node.js installation from NodeSource (Node.js v24.15.0)
  • Removed the older manual Node.js installation from /usr/local/bin
  • Updated the n8n test to include the npm global bin directory in PATH before running n8n

Validation

  • Confirmed the Linux Halo runners now use Node.js v24.15.0.
  • Confirmed the n8n test can find and run n8n --version successfully.

@danielholanda danielholanda mentioned this pull request Jun 5, 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