Hi Karl,

Le 10/01/2017 à 22:31, Karl Kornel a écrit :
Hello!

I have a second patch, to make another similar change to my first patch.

I noticed that slurmdbd’s preinst script is doing the same user-creation.  
Although slurmdbd does not directly relate to slurm-client, both slurm-client 
and slurmdbd depend directly on the slurm-wlm-basic-plugins package.  So, my 
second patch does two things:

1) Move the preinst script from the slurm-client package to the 
slurm-wlm-basic-plugins package.
2) Remove the duplicate user-creation code from slurmdbd.

FWIW, we took the same approach in our EDF-specific packages:

https://github.com/edf-hpc/slurm-llnl

It works fine for us so far!

I just don't remember why we didn't propose the patch back into Debian official packages though :(

Best,
Rémi

Reply via email to