Re: PATCH: Custom table support

Simon Leinen <[email protected]> Fri, 21 Jul 2006 14:52:17 +0200
Newsgroups gmane.network.mrtg.devel
Message-ID <[email protected]>
Phil,

your patch looks like a very useful generalization of an important
feature in MRTG.

> The syntax is somewhat ugly, but works:
>  %{TableName}{Header=Value}{RetrieveField}

> Eg:
>  %{dfTable}{dfFileSys=/vol/vol0/}{dfPerCentInodeCapacity}&%{dfTable}{dfFileSys=/vol/web/}{dfPerCentKBytesCapacity}:ro-community@netapp1b-mgmt

> A bit longer than I'd like, but given what it's achieving, I don't
> see a way to make it shorter.

What I don't understand is why you even need the `TableName' - it
should be sufficient that the `Header' and `RetrieveField' have the
same index structure, right?

In some cases there may not even be a common table, e.g. you may want
to refer to ifDescr (from ifTable) and retrieve ifHCInOctets (from
ifXTable).

Regards,
-- 
Simon.

--
Unsubscribe mailto:[email protected]?subject=unsubscribe
Help        mailto:[email protected]?subject=help
Archive     http://lists.ee.ethz.ch/mrtg-developers