Skip to content

Conversation

sjpb
Copy link
Collaborator

@sjpb sjpb commented Sep 22, 2023

This PR provides an install-generic.yml task book which can use "user-supplied" Slurm binaries/libraries rather than installing OpenHPC packages.

Note that the ./configure command for the Slurm build used with this must:

  • Set --sysconfdir to match the directory of openhpc_slurm_conf_path (default: /etc/slurm/slurm.conf)
  • Set --prefix to match the directory the slurm daemon binaries are deployed at (see openhpc_sbin_dir). This is because PluginDir defaults to being based on the build prefix. Although it can be overriden in slurm.conf, the slurmds do not appear to get this parameter when running configless, so they will fail on start with an error that the (default) plugin dir doesn't exist.

This PR also:

  • Allows the slurmd spool directory to be customised
  • Allows default templates to be overriden

@sjpb sjpb force-pushed the feat/no-ohpc branch 6 times, most recently from a5709a6 to 34606b0 Compare March 19, 2024 15:34
@sjpb sjpb force-pushed the feat/no-ohpc branch 2 times, most recently from 4200fa3 to 9f8e765 Compare July 24, 2024 14:44
sjpb and others added 5 commits February 25, 2025 14:34
* Add group.node_params to partitions/groups. (#182)

Allows Features, etc., to be added to partitions.

* update SelectType from legacy to current default (#167)

---------

Co-authored-by: Kurt Bendl <[email protected]>
@sjpb sjpb force-pushed the feat/no-ohpc branch 2 times, most recently from 1a2841d to db53850 Compare September 4, 2025 14:23
@sjpb
Copy link
Collaborator Author

sjpb commented Sep 4, 2025

See appliance PR testing this here: stackhpc/ansible-slurm-appliance#769

@sjpb sjpb marked this pull request as ready for review September 4, 2025 15:32
@bertiethorpe bertiethorpe self-requested a review September 8, 2025 08:55
Copy link
Member

@bertiethorpe bertiethorpe left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants