Skip to content

Release initial batch of Halo Box Playbooks#326

Merged
danielholanda merged 9 commits into
mainfrom
dholanda/hb_release
Jun 12, 2026
Merged

Release initial batch of Halo Box Playbooks#326
danielholanda merged 9 commits into
mainfrom
dholanda/hb_release

Conversation

@danielholanda

@danielholanda danielholanda commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Releasing the following playbooks.

Core (5)

  • comfyui-image-gen
  • lmstudio-rocm-llms
  • n8n-automation-gpt-oss
  • pytorch-rocm-llms
  • vscode-qwen3-coder

Supplemental (10)

  • amd-sync
  • gaia-agents
  • lemonade-getting-started
  • llama-factory-finetuning
  • ollama-getting-started
  • open-webui-chat
  • pytorch-finetuning
  • pytorch-kernels
  • speech2speech-translation
  • unsloth-llms-finetuning

Remaining (4) — will be pushed through separate PR's

  • cvml
  • openclaw-lemonade-server
  • vllm-inference
  • clustering-rpc-server

@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

sreeram-11 commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

@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
@sreeram-11 sreeram-11 requested a review from adamlam2-amd June 11, 2026 18:21
@sreeram-11

Copy link
Copy Markdown
Collaborator

#326 (comment)

@danielholanda danielholanda merged commit 9291d24 into main Jun 12, 2026
242 of 249 checks passed
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