Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Packaging: add a dependency on Slurm version for RPM packages #158

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kcgthb
Copy link

@kcgthb kcgthb commented Jan 31, 2025

Slurm SPANK plugins need to be recompiled for each major Slurm version. To make upgrades easier and avoid confusion, this PR proposes to add a dependency on the version of Slurm that's installed on the system, and add that major version in the generated RPM release info.

For instance, building RPMs for Pyxis 0.20 on a el9 host with Slurm 24.11 RPMs will create a nvslurm-plugin-pyxis-0.20.0-1.slurm2411.el9.x86_64.rpm package, with a dependency on libslurm.so.42()(64bit).

To do this, we get the version of the installed slurm-devel RPM package in the SPEC file, via a rpm -q command. Doing this uses the "%{VERSION}" RPM tag, which conflicts with VERSION variable that is used in the Makefile and passed to rpmbuild. So to avoid unwanted expansion of that %{VERSION} variable, it's been renamed to PYXIS_VER in the Makefile.

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.

1 participant