[rancid] clogin goes out of phase with extreme switches

[Nicolai Langfeldt]

Hi,

Our rancid version is 2.3.8.

We have a large number of extreme switches. xrancid has a tendency to
fail, quite reliably for a few of them (and then suddenly it works again
on a switch to switch basis).

The problem lies in clogin which fires off commands prematurely. Here
follows some clogin -d output with some annotations to make it clearer
to me what happens.

clogin has logged in and sent the clipaging commands without problem.
The first commands have been sent also no problem. In this instance, at
the fifth prompt everything goes south. It appears that it's always on
the fifth prompt that stuff breaks (*mumble* at least with this switch).

expect: does "fs1.skatt.5 # show diag\r\n\r" (spawn_id exp6) match
regular expression "\u0008+"? (No Gate, R\
E only) gate=yes re=no
"^[^\n\r *]*.? ?fs1.skatt.[0-9]+ #"? Gate "*fs1?skatt* #"? gate=yes re=yes

******* The prompt remains in buffer, a prompt match is found at once
and the next command is send prematurely. Only how is the match buffer
advanced the right way:

expect: set expect_out(0,string) "fs1.skatt.5 #"
expect: set expect_out(spawn_id) "exp6"
expect: set expect_out(buffer) "fs1.skatt.5 #"
fs1.skatt.5 #

***** send: sending "show switch\r" to { exp6 }

Gate keeper glob pattern for '^H+' is ''. Not usable, disabling the
performance booster.
Gate keeper glob pattern for '^[^
*]*.? ?fs1.skatt.[0-9]+ #' is '*fs1?skatt* #'. Activating booster.
Gate keeper glob pattern for '^[^
]*.? ?fs1.skatt.[0-9]+ #.' is '*fs1?skatt* #?'. Activating booster.
Gate keeper glob pattern for '^--More--[
]+' is '--More--*'. Activating booster.
Gate keeper glob pattern for '[^
]*[
]+' is ''. Not usable, disabling the performance booster.
Gate keeper glob pattern for '[^
]*Press <SPACE> to cont[^
]*' is '*Press <SPACE> to cont*'. Activating booster.
Gate keeper glob pattern for '^ *--More--[^
]*' is '*--More--*'. Activating booster.
Gate keeper glob pattern for '^<-+ More -+>[^
]*' is ''. Not usable, disabling the performance booster.

******* Here we're out of phase. We start seeing the output of the "show
diag" command sent before "show switch"

expect: does " show diag\r\n\r" (spawn_id exp6) match regular expression
"\u0008+"? (No Gate, RE only) gate=\
yes re=no