nova-compute does not respect reserved memory for dom0 when using libvirt/XEN

Bug #1187330 reported by Thomas Kaergel
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Expired
Undecided
Unassigned

Bug Description

When using libvirt/XEN nova-compute does not respect the reserved memory for dom0. Dom0 is skipped in memory calculation in nova/virt/libvirt/driver.py. I suggest introducing of a new config-option named "libvirt_xen_ballooning=bool" which represents the usage of XEN ballooning defaulting to "true" (as this is xen default). If this value is set to false then dom0-memory is included in calculation of used memory.

Tags: xen
Julien Danjou (jdanjou)
affects: ceilometer → nova
Changed in nova:
assignee: nobody → Thomas Kaergel (thomas-kaergel)
melanie witt (melwitt)
tags: added: libvirt
Michael Still (mikal)
Changed in nova:
status: New → Triaged
importance: Undecided → Low
tags: added: low-hanging-fruit
Mathew Odden (locke105)
Changed in nova:
assignee: Thomas Kaergel (thomas-kaergel) → nobody
Rohan (kanaderohan)
Changed in nova:
assignee: nobody → Rohan (kanaderohan)
Rohan (kanaderohan)
Changed in nova:
assignee: Rohan (kanaderohan) → nobody
Revision history for this message
Sumanth Nagadavalli (sumanth-nagadavalli) wrote :

Right now, the method, get_memory_mb_used in nova/virt/libvirt/driver.py does the following things for dom0,
m = open('/proc/meminfo').read().split()

idx1 = m.index('MemFree:')
idx2 = m.index('Buffers:')
idx3 = m.index('Cached:')

used += (dom_mem -
                             (int(m[idx1 + 1]) +
                              int(m[idx2 + 1]) +
                              int(m[idx3 + 1])))

Looks like, it is assuming that the ballooning is enabled by default.

Is the expectation to hide it behind a config?

Changed in nova:
assignee: nobody → Sumanth Nagadavalli (sumanth-nagadavalli)
Revision history for this message
Geza Gemes (geza-gemes) wrote :

Seems to be a duplicate of bug #1266415, for which fix was commited: https://review.openstack.org/#/c/70835/

Alvaro Lopez (aloga)
tags: added: xen
Sean Dague (sdague)
Changed in nova:
assignee: Sumanth Nagadavalli (sumanth-nagadavalli) → nobody
Revision history for this message
shailendra acharya (acharyashailendra1) wrote :

hello i want to work over this bug, could u help me to reproduce this bug and plz also share useful links

Sean Dague (sdague)
Changed in nova:
status: Triaged → Confirmed
Changed in nova:
assignee: nobody → Sreeram Vancheeswaran (sreeram-vancheeswaran)
Changed in nova:
assignee: Sreeram Vancheeswaran (sreeram-vancheeswaran) → nobody
Revision history for this message
Eduardo Otubo (otubo) wrote :

Can anyone confirm it is a duplicate of bug #1266415 (with corresponding commit https://review.openstack.org/#/c/70835/)? I want to work on this issue otherwise.

Revision history for this message
Markus Zoeller (markus_z) (mzoeller) wrote :

The age and number of assignee changes suggest that this isn't really
a low-hanging-fruit. I'm removing the tag to not confuse new folks who
are searching for that tag.

tags: removed: libvirt low-hanging-fruit
Revision history for this message
Markus Zoeller (markus_z) (mzoeller) wrote : Cleanup EOL bug report

This is an automated cleanup. This bug report has been closed because it
is older than 18 months and there is no open code change to fix this.
After this time it is unlikely that the circumstances which lead to
the observed issue can be reproduced.

If you can reproduce the bug, please:
* reopen the bug report (set to status "New")
* AND add the detailed steps to reproduce the issue (if applicable)
* AND leave a comment "CONFIRMED FOR: <RELEASE_NAME>"
  Only still supported release names are valid (LIBERTY, MITAKA, OCATA, NEWTON).
  Valid example: CONFIRMED FOR: LIBERTY

Changed in nova:
importance: Low → Undecided
status: Confirmed → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.