Bug#850891: Additional patch, to cover slurmdbd's duplicate code

2017-07-28 Thread Gennaro Oliva
tags 850891 + wontfix Hi Rémi and Karl, since the discussion about this possible feature is over I'm tagging this old bug with wontfix. Thanks for your contribution -- Gennaro Oliva

Bug#850891: Additional patch, to cover slurmdbd's duplicate code

2017-01-28 Thread Gennaro Oliva
Hi Rémi and Karl, On Wed, Jan 11, 2017 at 09:23:14PM +, Karl Kornel wrote: > 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 >

Bug#850891: Additional patch, to cover slurmdbd's duplicate code

2017-01-11 Thread Karl Kornel
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

Bug#850891: Additional patch, to cover slurmdbd's duplicate code

2017-01-11 Thread Rémi Palancher
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

Bug#850891: Additional patch, to cover slurmdbd's duplicate code

2017-01-10 Thread Karl Kornel
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