[Ovirt-devel] Re: [Patch] Experimental Graphs Backend Code

[Hugh O. Brock]

On Wed, May 28, 2008 at 03:46:14PM -0400, Hugh O. Brock wrote:
> On Tue, May 27, 2008 at 05:23:40PM -0400, Mohammed Morsi wrote:
> > This patch is for review only, and is not meant to be committed (partly 
> > because I didn't update before generating it and partly because I know some 
> > parts are wrong, but need feedback on how to fix them). The major changes 
> > to this patch are in the hardware controller and consist of wiring up the 
> > backend calls to generate the graphs on the hardware summary page, namely 
> > the calls to the database to generate the availability graphs and the calls 
> > to Mark's API to generate the history and snapshot graphs. Various other 
> > small interface and styling tweaks are also included.
> 
> 
> Right now we *can't* overcommit RAM, so I don't know if we want the
> same logic for this graph as for the cpu graph. Also, I assume that
> memory_available is "the sum of all memory on all machines in the
> pool," where "memory used" is "the sum of all memory currently in use
> by active VMs?" (Scott, you can probably comment on this too I'm sure)

Correction: Dan and Brian tell me that KVM will actually overcommit
RAM, so yeah, I guess we had best leave the "flippable" stacked bar in for RAM as well.

--Hugh