Hi Rémi, (I hope the é comes through the email properly!)

If I would play devil’s advocate, then I would say this: “An action like this 
seems inappropriate to place into a package that does not clearly indicate why 
it is needed.  Instead, you should create a slurm-common package, and have the 
user creation happen there.”

I would be OK with that: I am fine putting together a patch to create a 
“slurm-common” package that does the user creation.  Some common documentation 
could also be moved to that package.  But, I did not know if that was 
appropriate, so I submitted the simpler change first!

~ Karl

On 1/11/17, 1:21 AM, "Rémi Palancher" <r...@rezib.org> wrote:

    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