[rancid] paloalto feedback in current alpha [was: Re: clogin commenting script commands following multiple blanks lines]

[Erik Muller]

On 11/5/18 14:29 , heasley wrote:
> Fri, Oct 26, 2018 at 06:24:10PM -0400, Erik Muller:
>>
>> +# This routine parses "show chassis inventory"
>> +sub ShowInventory {
...
> 
> Is this a new command or specific to larger platforms?  What is the error
> if the command is unknown?

I assume it's universal across all PA (looks like it was introduced to the 
CLI reference docs in panos 6.0), but I only have one model to test 
against, on which it does:
admin at fw1.ams(active)> show chassis inventory

Slot       Component            Serial Number   Ports  Revision
            PA-3250              0xxxxxxxxxx1    20     1.1

FANTRAY1   PA-3200-FANTRAY      0xxxxxxxxxx9           1.0

PS1 (left) D1U54P-W-650-12-HB4C-PAN Mxxxxxxxxxx7           RB
PS2 (right) D1U54P-W-650-12-HB4C-PAN Mxxxxxxxxxx1           RB

Presumably on a box that doesn't support it, you'd get:
admin at fw1.ams(active)> show badcommand

Invalid syntax.
admin at fw1.ams(active)>


>> diff -ur rancid-3.99.99/lib/nxos.pm.in rancid-3.99.99-em/lib/nxos.pm.in
>> --- rancid-3.99.99/lib/nxos.pm.in	2018-09-19 18:02:44.000000000 -0400
>> +++ rancid-3.99.99-em/lib/nxos.pm.in	2018-10-26 17:58:51.611639817 -0400
>> +	# Drop bcm_mem_locl_trace.log
>> +	/\s+bcm_mem_lock_trace\.log$/ && next;
>> +
>> -	if (/( debug_logs| log)\/$/) {
>> +	if (/( debug_logs| log| vdc_\d+)\/$/) {
 >
> did I miss an explaination of this patch?  I see what it does, but ...

Whoops, meant to break that out as a separate patch.  Nexus 9300s on 
7.0.3.I7.5 seem to fairly regularly append to debug:/bcm_mem_lock_trace.log 
resulting in a lot of noisy diffs.
The vdc_1 bit I was testing, but may not be necessary - I think that may 
only update when a config change is saved, but doesn't seem to generate any 
spurious changes in normal operation.