[rancid] Re: Adding "show interfaces status" to rancid

[Per Carlson]

Hi.

> I would like to add the “show interfaces status” command to rancid. From
> what I can gather from previous postings, I will need to:
>
> -        Modify %commands :
> %commands=(
>
>         'show interfaces status'          => "ShowIntStatus",  …
>
> -        Modify @commands :
> @commands=(
>
>         "show interfaces status", …

Are you looking at an old RNACID version? In 2.3.2 there is only one
list of hashes to modify:

# Main
@commandtable = (
        {'admin show version'           => 'ShowVersion'},
        {'show version'                 => 'ShowVersion'},
        ...
        {'write term'                   => 'WriteTerm'},
);

The actual %commands and @commands are automatically built based on
@commandtable.

To answer the question: yes just add {'show interfaces status' =>
"ShowIntStatus"}, to @commandtable.

> -        Develop a new sub (“ShowIntStatus”) to process/format the output

Correct. The hard part is to figure out how to use ProcessHistory.


> Is there any other (easier) way to accomplish this, or is this the
> recommended approach?

AFAIK it's the recommended and only way.

I would also recommend to put the code in a separate patch. That way
it's easier to re-apply it whenever a new upstream is released.

-- 
Pelle

A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?