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

Make sackd deb package create /etc/default/sackd rather than slurm_ops #54

Open
NucciTheBoss opened this issue Nov 20, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@NucciTheBoss
Copy link
Member

The sackd package from Debian currently does not create its own /etc/default/sackd package during installation. This means that sackd will fail to start as $SACKD_OPTIONS is left undefined in the systemd service file.

We can get around this by just having slurm_ops create the environment file with the proper file mode permissions in the _AptManager(...)._apply_overrides() method, but this should be something that's ideally handled by the upstream package instead as all sackd's will need this environment file to ensure that $SACKD_OPTIONS is defined.

Related issue in the Debian Bug Tracker for sackd: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1087927

@NucciTheBoss NucciTheBoss added the enhancement New feature or request label Nov 20, 2024
@dsloanm
Copy link

dsloanm commented Dec 16, 2024

This is fixed as part of Debian Sid's release of slurm-wlm/24.11.0-1: https://packages.debian.org/sid/amd64/sackd/filelist so just a matter of time until the creation step can be removed from slurm_ops.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants