Patch to add SLURM partition support to pdsh slurm module about pdsh

[GoogleCodeExporter]

Nice work, you even added a test.
My only comment would be to use -P not -p which conflicts with pdcp option.
(I don't think -P conflicts with anything but I did not check)

If you make that change I'll include your patch.

Thanks!
mark

Original comment by [email protected] on 25 Oct 2012 at 4:07

• Changed state: Accepted

[GoogleCodeExporter]

No problem, here's the patch with the change.  I also added a bit to the man 
page documenting -P.

Original comment by [email protected] on 26 Oct 2012 at 3:52

Attachments:

• pdsh-add-slurm-partition-target.patch

[GoogleCodeExporter]

Just checking in on this.

Original comment by [email protected] on 6 Nov 2012 at 2:45

[GoogleCodeExporter]

Sorry, I applied your patch to my local git tree, but ran into
problems when running ./bootstrap.

I don't think the check for slurm_load_partitions is strictly necessary
at this time. OK if I remove it, and thus forgo re-run of ./bootstrap for
now?

Original comment by [email protected] on 16 Nov 2012 at 10:16

[GoogleCodeExporter]

That's fine.  If slurm_load_jobs is available, then slurm_load_partitions 
almost certainly is as well.

Original comment by [email protected] on 16 Nov 2012 at 10:41

[GoogleCodeExporter]

This issue was closed by revision c71ef9c13a97.

Original comment by [email protected] on 19 Nov 2012 at 7:02

• Changed state: Fixed