Bug 1911610 - [RFE] VM without ballooning won't have memory stats
Summary: [RFE] VM without ballooning won't have memory stats
Keywords:
Status: CLOSED DUPLICATE of bug 1917718
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: 4.4.4.5
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.4.6
: ---
Assignee: Liran Rotenberg
QA Contact: meital avital
URL:
Whiteboard:
Depends On:
Blocks: 1917718
TreeView+ depends on / blocked
 
Reported: 2020-12-30 10:49 UTC by Liran Rotenberg
Modified: 2021-04-11 15:05 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-01-31 14:03:00 UTC
oVirt Team: Virt
Embargoed:
ahadas: ovirt-4.4?
ahadas: planning_ack?
ahadas: devel_ack+
ahadas: testing_ack?


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 114207 0 master MERGED core: change balloon behvaior 2021-04-19 07:05:25 UTC

Description Liran Rotenberg 2020-12-30 10:49:59 UTC
Description of problem:
When the ballooning is disabled we have no memory data on the guest.
Ovirt GA isn't part of RHCOS/EL8 (and future) OSes.
In the VDSM code when we have no ballooning available we fallback to OGA which doesn't exist in those.

Version-Release number of selected component (if applicable):
Any VDSM with OS that doesn't support OGA.

How reproducible:
100%

Steps to Reproduce:
1. Create a VM without OGA (el8).
2. Disable ballooning on the VM.
3. Run the VM.
4. Check the VM memory.

Actual results:
Memory is always on 0%, no stats on memory.

Expected results:
Having Memory provided from GA to VDSM and reported back to the engine.

Additional info:
https://github.com/oVirt/vdsm/blob/master/lib/vdsm/virt/vmstats.py#L436-L440

Comment 1 Liran Rotenberg 2021-01-04 13:34:51 UTC
Ballooning device is used for memory stats gathering of the guest when qemu-guest-agent is in use.
There is no planning to change it.

We should enable the ballooning device on high performance profile, currently we disable it.
Users who want only the memory stats gathering without further ballooning functions should disable it on cluster level, like the way it's recommended to do so for KSM with high performance VMs.

Comment 2 Arik 2021-01-04 13:59:06 UTC
(In reply to Liran Rotenberg from comment #1)
> We should enable the ballooning device on high performance profile,
> currently we disable it.

This can happen regardless of the VM profile, right?
Let's say I disable ballooning for a server VM - it would lead to the same issue.

> Users who want only the memory stats gathering without further ballooning
> functions should disable it on cluster level, like the way it's recommended
> to do so for KSM with high performance VMs.

Yes, that can be used as a workaround when ballooning/ksm can be disabled for all VMs in the cluster.

Would you agree that it becomes more of an RFE to change what happens when disabling (or not-enabling) ballooning on a VM so it would define the balloon device but won't trigger balloon inflation/deflation?

Comment 3 Liran Rotenberg 2021-01-04 15:58:21 UTC
(In reply to Arik from comment #2)
> (In reply to Liran Rotenberg from comment #1)
> > We should enable the ballooning device on high performance profile,
> > currently we disable it.
> 
> This can happen regardless of the VM profile, right?
> Let's say I disable ballooning for a server VM - it would lead to the same
> issue.
Yes.
> 
> > Users who want only the memory stats gathering without further ballooning
> > functions should disable it on cluster level, like the way it's recommended
> > to do so for KSM with high performance VMs.
> 
> Yes, that can be used as a workaround when ballooning/ksm can be disabled
> for all VMs in the cluster.
> 
> Would you agree that it becomes more of an RFE to change what happens when
> disabling (or not-enabling) ballooning on a VM so it would define the
> balloon device but won't trigger balloon inflation/deflation?
I will need to look into it further.
But we should check it and for high performance VMs (or even have another checkbox on the VM for it).
This direction in general is an RFE from my point of view.

Comment 4 Michal Skrivanek 2021-01-21 09:21:24 UTC
I think it would be fine to always include ballooning device and use different means to communicate to mom that ballooning should be skipped for this VM. Any extra property would work I guess, probably even just setting guaranteed mem to current.  In that case it really just means to enable balloon unconditionally...

Comment 5 Arik 2021-01-31 14:03:00 UTC

*** This bug has been marked as a duplicate of bug 1917718 ***


Note You need to log in before you can comment on or make changes to this bug.