Skip to content

feat: add slurm_bin_prefix for non-standard Slurm binary paths#9

Merged
seilk merged 1 commit intoseilk:mainfrom
kochanha:feat/slurm-bin-prefix
Mar 23, 2026
Merged

feat: add slurm_bin_prefix for non-standard Slurm binary paths#9
seilk merged 1 commit intoseilk:mainfrom
kochanha:feat/slurm-bin-prefix

Conversation

@kochanha
Copy link

Summary

  • Some Slurm clusters (e.g. evgl1) don't have sinfo/squeue/scontrol in the remote shell PATH, causing ksh: sinfo: not found errors when running commands via SSH
  • Added configurable slurm_bin_prefix per Slurm cluster to specify the full binary directory path (e.g. /opt/slurm-21.08/bin)
  • Clusters without the setting continue to use bare commands as before

Changes

  • models.py: Added slurm_bin_prefix field to SlurmClusterConfig
  • slurm.py: Added slurm_bin_prefix parameter to collect_slurm_snapshot(), prefixes sinfo/squeue/scontrol with the configured path
  • config.py: Parse slurm_bin_prefix from config JSON
  • cli.py: Prompt for slurm_bin_prefix during opensmi onboard, display in review, pass through to runtime

Usage

In opensmi.json:

{
  "name": "evgl1",
  "login_node": "evgl1",
  "user": "ckim91",
  "slurm_bin_prefix": "/opt/slurm-21.08/bin"
}

Or enter the path interactively during opensmi onboard.

Test plan

  • Verify opensmi slurm works on clusters without slurm_bin_prefix (no regression)
  • Verify opensmi slurm works on evgl1 with slurm_bin_prefix: /opt/slurm-21.08/bin
  • Verify opensmi onboard prompts for Slurm bin path and saves it correctly

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@seilk seilk merged commit 67b22a7 into seilk:main Mar 23, 2026
5 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