[Orca-dev] [PATCH] Updates to include additional Solaris performancemetrics for graphing

[Sean O'Neill]

At 01:37 PM 12/19/2002 -0800, Blair Zajac wrote:

>Hi Sean,
>
>Thanks.  This one applied cleanly.
>
>Do you have sample plots for these?  I'd like to add them to the
>
>     http://www.orcaware.com/orca/docs/orcallator.html

Yeah, just goto http://chivas.oneill.dhs.org/orca

>page.  This will then let us use href elements for the new plots so
>people can get more information on it.
>
>Rather lengthy review below.
>
>Also, please include the log message with any diffs.  It makes it
>easier to have the entire patch and log message in one email,
>rather than finding the old email with the log message.

Not sure what you mean here.  What "log message" ?  You mean a log entry 
from svn ?  Assuming so, I thought log entries were only done during a 
commit action which I can't do - maybe time for me to read the SVN doco 
again ? :)

>Regarding the new measurements, let's have all measurements that
>are something per second have a "/s" in the name.  I haven't looked
>closely at which ones are, but I may note some below.

K, I take a look and make changes.


>I definitely want to get these new column names right so we don't
>have to change them later and mess up plots.
>
>In orcallator.cfg.in:

I started to reply back on this section .. NEVERMIND ... LOL I'll reply 
back with the changes your requested.

>I noticed in orcallator.se that ufsinopage is measured, but there's
>no plot for it.

Hmmmm, I'll take a look.


>Now in orcallator.se:
>
>+// Version 1.37:   Dec 20, 2002 Additions by Sean O'Neill - Happy Holidays !!
>+//                              Added several new variables for Orca to 
>collect
>+//                              and graph including:
>+// pgrec - page reclaims (include pageout)
>+// pgfrec - page reclaims from free list
>
>Anal, but can the list of names be justified as all the other comments are?

K np

>+// ufsinopage - inodes taked with no attached pages
>
>Typo on taked?

Yeah typo


>In lib/Makefile.in and ./orcallator/start_orcallator.sh.in, how
>about we put the 3.2.1 and 3.3 directories in a SE directory,
>so it looks like
>
>lib/SE/3.2.1
>lib/SE/3.3
>
>and that it installs this way also.

k


>Regarding ./lib/Orca/Constants.pm, I'd like to keep this change out of
>this patch.  How do the output PNG files look with this change?

http://chivas.oneill.dhs.org/orca/o_gauge_usr_pct,o_gauge_sys_pct,o_gauge_wio_pct,o_gauge_idle_pct-hourly.html

The data elements in the graph are still obvious enough that you can 
differentiate between the data elements for whatever interval you have Orca 
setup for.   Note the data in the graph example is 1 minute interval data - 
not 5 minutes.  I don't have any graph examples show this new data using 5 
minute intervals - sorry.

>One of the things that these numbers are designed for is that the number
>of data points in a plot is less than the number of pixels in a plot,
>otherwise RRD needs to resample the data, and this change from 1.5
>days to 3 days bumps the number of data points from 432 to 864, which
>is greater than the default width of 500.

Three days ??? The change I made to Constants.pm makes the Hourly graph 
chart 3 hours worth of data instead of the usual 1.5 hours - not 3 days - 
unless there is something I missed when changing this - which apparently I 
did.  You've lost me on the math you are using for the number data elements 
vs pixels.  Following the comments in Constants.pm:

(1.5*60*60)/300 = 18 data points

My changes double this to 36 data points but I'm not sure how this relates 
to what you are referring to above.

How does this math work out when the collection interval is changed to 60 
seconds ?  Actually, if this does something weird like forcing Orca to 
"resample" all the time, can you write something up and put this into the 
Orca FAQ to explain the "cost" of making the interval shorter - assuming 
this is a cost that is ?


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

Sean O'Neill