[libvirt] [PATCH] Revert "LXC: show used memory as 0 when domain is not active"
Daniel P. Berrange
berrange at redhat.com
Fri Aug 28 08:34:17 UTC 2015
On Thu, Aug 27, 2015 at 04:47:24PM -0600, Eric Blake wrote:
> On 08/27/2015 04:38 PM, Jim Fehlig wrote:
>
> >>>>> This reverts commit 1ce7c1d20cfd5afb26d2dbc88201085d52415d0e,
> >>>>> which introduced a significant semantic change to the
> >>>>> virDomainGetInfo() API. Additionally, the change was only
> >>>>> made to 2 of the 15 virt drivers.
> >>>>>
>
> >>>> I guess we should probably do this in the stable branch too, since I
> >>>> think it made it into the most recent release.
> >>>>
> >>> Erm, the change is out for a while now:
> >>>
> >>> git desc 1ce7c1d20cfd5afb26d2dbc88201085d52415d0e
> >>> v1.2.6-225-g1ce7c1d
> >>>
> >>
> >> Oh yes, so it is. I'm still inclined to say we should be reverting it as
> >> I think it is wrong. The change was based on the misleading field name
> >> shown by virsh. The info->memory field shows the current balloon target,
> >> and conceptually this should be equal to the max memory if the ballooon
> >> driver is not active. As such I think it should be equal to max memory
> >> if shutoff too.
> >>
> >
> > Yikes, I forgot to commit this before leaving for travel/vacation. Is it
> > still ok to commit to master? And just to be clear, then cherry pick to
> > 1.2.7-1.2.18 maint branches?
>
> Yes, that sounds like the best way to handle it.
Agreed
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list