search

wotwot / pdsh

Automatically exported from code.google.com/p/pdsh
GNU General Public License v2.0
0 stars 0 forks source link

node list from dshbak broken with 2.25 #19

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Since upgrading to 2.25, when I use dshbak to get lists of nodes, I can't 
simply copy/paste the list as it doesn't have any commas in it:

[root@a4bu-m1 updates]# pdsh -a "rpm -qa|grep dhcp-|sort" |dshbak -c
----------------
a4bu-a[001-002]a4bu-d[001-002]a4bu-io[01-02]a4bu-l[4-5]a4bu-n[301-308]n[001-078]
----------------

Thanks.

Original issue reported on code.google.com by justingw...@gmail.com on 19 Apr 2011 at 9:12

GoogleCodeExporter commented 9 years ago 
Can you paste output without |dshbak -c?
e.g.:

pdsh -a "rpm -qa|grep dhcp-|sort" 

You can just attach the output if you'd like.

Then I can use those lines to create a reproducer.
Thanks for finding this bug.
mark

Original comment by mark.gro...@gmail.com on 19 Apr 2011 at 9:25

GoogleCodeExporter commented 9 years ago 
[root@a4bu-m1 updates]# pdsh -a "rpm -qa|grep ^dhcp-3|sort"
a4bu-n303: dhcp-3.0.5-23.el5_6.4
a4bu-n307: dhcp-3.0.5-23.el5_6.4
a4bu-d001: dhcp-3.0.5-23.el5_6.4
a4bu-n301: dhcp-3.0.5-23.el5_6.4
a4bu-n306: dhcp-3.0.5-23.el5_6.4
a4bu-d002: dhcp-3.0.5-23.el5_6.4
a4bu-n308: dhcp-3.0.5-23.el5_6.4
a4bu-n305: dhcp-3.0.5-23.el5_6.4
a4bu-n304: dhcp-3.0.5-23.el5_6.4
n003: dhcp-3.0.5-23.el5_6.4
a4bu-a002: dhcp-3.0.5-23.el5_6.4
a4bu-a001: dhcp-3.0.5-23.el5_6.4
a4bu-n302: dhcp-3.0.5-23.el5_6.4
n016: dhcp-3.0.5-23.el5_6.4
n006: dhcp-3.0.5-23.el5_6.4
a4bu-io02: dhcp-3.0.5-23.el5_6.4
n008: dhcp-3.0.5-23.el5_6.4
n007: dhcp-3.0.5-23.el5_6.4
n012: dhcp-3.0.5-23.el5_6.4
a4bu-io01: dhcp-3.0.5-23.el5_6.4
n005: dhcp-3.0.5-23.el5_6.4
n013: dhcp-3.0.5-23.el5_6.4
n009: dhcp-3.0.5-23.el5_6.4
n010: dhcp-3.0.5-23.el5_6.4
n004: dhcp-3.0.5-23.el5_6.4
n014: dhcp-3.0.5-23.el5_6.4
n002: dhcp-3.0.5-23.el5_6.4
n015: dhcp-3.0.5-23.el5_6.4
...

Original comment by justingw...@gmail.com on 19 Apr 2011 at 9:33

GoogleCodeExporter commented 9 years ago 
ok, thanks, this was a very silly mistake added during the refactor of dshbak
code in r1289. I'm going to add a test and then the fix shortly. Thanks for 
reporting
it, it is sad that the existing tests missed it, but they all tested corner 
cases
not involving multiple host prefixes.

Original comment by mark.gro...@gmail.com on 19 Apr 2011 at 10:09

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1318.

Original comment by mark.gro...@gmail.com on 19 Apr 2011 at 10:15

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r1319.

Original comment by mark.gro...@gmail.com on 19 Apr 2011 at 10:15

GoogleCodeExporter commented 9 years ago 
You might want to checkout the latest pdsh trunk and verify that scripts/dshbak 
now
works for your case. The fix will be in the next (imminent) release of pdsh.

Thanks again for catching this bug.

mark

Original comment by mark.gro...@gmail.com on 19 Apr 2011 at 10:17

GoogleCodeExporter commented 9 years ago 
Issue 27 has been merged into this issue.

Original comment by mark.gro...@gmail.com on 28 Apr 2011 at 1:38

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r1335.

Better fix for print_header() from Scott M. McDermott
via Issue 27.

Original comment by mark.gro...@gmail.com on 28 Apr 2011 at 2:35