prometheus-slurm-exporter: mount Slurm client tools onto the container PATH - #1395
Open
100milliongold wants to merge 1 commit into
Open
prometheus-slurm-exporter: mount Slurm client tools onto the container PATH#1395100milliongold wants to merge 1 commit into
100milliongold wants to merge 1 commit into
Conversation
…r PATH The unit mounts sdiag, sinfo and squeue at the host's slurm_install_prefix inside the container as well. The exporter invokes them by name, and a custom prefix is not on the container's PATH, so every scrape run fails: exec: "sinfo": executable file not found in $PATH With the default prefix the destination happens to be /usr/local/bin, which is already on PATH, so the bug only appears when the operator sets slurm_install_prefix. Restart=always keeps the unit reported as active, so nothing in the play fails; only Prometheus shows it, as an EOF on the scrape. Pin the destination paths to /usr/local/bin. The library mount is left alone: shared objects are resolved by path, not by PATH. Observed on DGX B300, DGX OS 7.5.0, slurm_install_prefix=/raid/slurm/usr/local. Signed-off-by: Jea-Eok-Kim <je.kim@xiilab.com>
100milliongold
marked this pull request as ready for review
September 4, 2026 03:24
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
roles/prometheus-slurm-exporter/templates/docker.slurm-exporter.service.j2mounts the Slurm client tools into the container at the host path:The exporter invokes them by name, so the destination must be on the container's
PATH. With the defaultslurm_install_prefixthe destination is/usr/local/bin, which already is; with a custom prefix it is not, and every scrape run dies:Restart=alwaysmeanssystemctl statuskeeps reportingactiveand the play completes normally. The only visible symptoms are an empty Grafana dashboard and, in Prometheus,Get "http://<node>:8080/metrics": EOF. On our node the unit had restarted 3289 times before anyone noticed.Reproduced on DGX B300 / DGX OS 7.5.0 with
slurm_install_prefix: /raid/slurm/usr/local.Fix
Pin the three destination paths to
/usr/local/bin. Behaviour with the default prefix is unchanged, since that is where they already landed.The library mount is deliberately left as-is: shared objects are resolved by path, not by
PATH.Verification
After changing the destinations on the affected node:
The Grafana SLURM dashboard populates as expected.