Patch to add SLURM partition support to pdsh slurm module

[GoogleCodeExporter]

I frequently find it useful to target all the nodes in a slurm partition with 
soem command.  To enable this, I wrote a patch to the pdsh slurm module that 
uses the slurm C API to retrieve a list of all the nodes in a partition (very 
similar to what the existing -j option does).  I've attached the patch in case 
you'd like to add it to the mainline.

Original issue reported on code.google.com by michaelf...@gmail.com on 25 Oct 2012 at 2:07

Attachments:

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

[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 mark.gro...@gmail.com 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 michaelf...@gmail.com on 26 Oct 2012 at 3:52

Attachments:

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

[GoogleCodeExporter]

Just checking in on this.

Original comment by michaelf...@gmail.com 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 mark.gro...@gmail.com 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 michaelf...@gmail.com on 16 Nov 2012 at 10:41

[GoogleCodeExporter]

This issue was closed by revision c71ef9c13a97.

Original comment by mark.gro...@gmail.com on 19 Nov 2012 at 7:02

• Changed state: Fixed