[Orca-users] RE: Problem with orcallator.se in SAN and/or T3-environments

[Sean O'Neill]

At 04:08 PM 2/18/2003 -0800, Blair Zajac wrote:
>B.Schopp at gmx.de wrote:
> >
> > Hi all,
> >
> > i fixed the problem with the changes shown below.
> > For better understanding:
> > The problem resided in orcallator.se: a field length
> > description which has been to short made orcallator
> > to recognize the T3-disks every $INTERVALL as new
> > disks.
> > Also the regexp in orcallator.cfg did not match the T3-
> > disks, so orca didn't generate graphs for them.
> >
> > I patched both orcallator.se and orcallator.cfg last
> > Tuesday and is working stable till now.
> >
> > Greetings
> >
> > Burkhardt Schopp
> >
> > orcallator.se:
> > OLD:
> > struct RawDisk {
> >   // Exposed interface that matches kstat.
> >   char          long_name[16];
> >   char          short_name[8];
> >   double        reads;
> >
> > NEW:
> > struct RawDisk {
> >   // Exposed interface that matches kstat.
> >   char          long_name[45]; // 45 should be enough for WWNs
> >   char          short_name[8];
> >   double        reads;
> >
> > orcallator.cfg:
> > OLD:
> > plot {
> > title                   %g Disk Run Percent
> > source                  orcallator
> > data
> > disk_runp_((?:c\d+t\d+d\d+)|(?:c\d+d\d+)|(?:[ms]d\d+))
> > line_type               line2
> >
> > NEW:
> > plot {
> > title                   %g Disk Run Percent
> > source                  orcallator
> > data
> > 
> disk_runp_((?:c\d+t\d+d\d+)|(?:c\d+d\d+)|(?:[ms]d\d+)|(?:c\d+t([A-Z0-9]{32})+d\d+))
> > line_type               line2
> >
>
>Burkhardt,
>
>Thanks for the patch.  Sean is working on integrating this into
>the Orca source tree.
>
>For the disk run percent change, I think there's a extra + in this line
>near the end that can be removed.  Otherwise, it can match any number of
>blocks of 32 characters of [A-Z0-9].
>
>Your change:
>disk_runp_((?:c\d+t\d+d\d+)|(?:c\d+d\d+)|(?:[ms]d\d+)|(?:c\d+t([A-Z0-9]{32})+d\d+))
>
>New change:
>disk_runp_((?:c\d+t\d+d\d+)|(?:c\d+d\d+)|(?:[ms]d\d+)|(?:c\d+t([A-Z0-9]{32})d\d+))
>
>Does this look ok?

Burkhardt,

Please reply to Blair.  I'm not going to commit the updates into Subversion 
until you comment on Blair's posting above.  Blair's comment looks 
correct.  If you already have commented back and I missed your posting, my 
apologies - simply resend it if so.

Is anyone else out there using T3 disks that has tried Burkhardt's update 
who can comment back on this ?


--
........................................................
......... ..- -. .. -..- .-. ..- .-.. . ... ............
.-- .. -. -... .-.. --- .-- ... -.. .-. --- --- .-.. ...

Sean O'Neill