Bug 1481246 - VM does not consume any hugepages on a host
VM does not consume any hugepages on a host
Status: CLOSED CURRENTRELEASE
Product: ovirt-engine
Classification: oVirt
Component: BLL.Virt (Show other bugs)
4.2.0
x86_64 Linux
unspecified Severity high (vote)
: ovirt-4.2.0
: ---
Assigned To: Francesco Romani
Artyom
: Triaged
Depends On:
Blocks: 1457239 1461476
  Show dependency treegraph
 
Reported: 2017-08-14 08:32 EDT by Artyom
Modified: 2017-12-20 06:13 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-12-20 06:13:10 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: Virt
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rule-engine: ovirt‑4.2+
alukiano: testing_plan_complete-
rule-engine: planning_ack+
tjelinek: devel_ack+
mavital: testing_ack+


Attachments (Terms of Use)
engine log (9.69 MB, text/plain)
2017-08-14 08:32 EDT, Artyom
no flags Details
vdsm log (16.98 MB, text/plain)
2017-08-14 09:04 EDT, Artyom
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
oVirt gerrit 81610 master MERGED vm: prepare hugepages outside _buildDomainXML 2017-09-22 12:24 EDT
oVirt gerrit 81860 master MERGED core: enable hugepages when using engine xml 2017-09-19 12:39 EDT

  None (edit)
Description Artyom 2017-08-14 08:32:35 EDT
Created attachment 1313093 [details]
engine log

Description of problem:
VM does not consume any hugepages on a host

Version-Release number of selected component (if applicable):
ovirt-engine-4.2.0-0.0.master.20170811144920.gita423008.el7.centos.noarch

How reproducible:
Always

Steps to Reproduce:
1. Configure hugepages on the host
HugePages_Total:    2048
HugePages_Free:     2048
HugePages_Rsvd:        0
HugePages_Surp:        0
Hugepagesize:       2048 kB
2. Create VM with 2 GB of memory and add custom property hugepages with value 2048 to it
3. Start VM

Actual results:
VM does not have any hugepage property under dumpxml
# virsh -r dumpxml 2 | grep -i huge
and so it does not consume any hugepages on the host 

Expected results:
VM has hugepages property and it consumes hugepages on the host

Additional info:
Comment 1 Michal Skrivanek 2017-08-14 08:50:43 EDT
please attach vdsm log as well, specify time frame and name of the VM as well
Comment 2 Artyom 2017-08-14 09:04 EDT
Created attachment 1313103 [details]
vdsm log

You can start looking from 
2017-08-14 15:30:22,607+03 INFO  [org.ovirt.engine.core.vdsbroker.CreateVDSCommand] (org.ovirt.thread.EE-ManagedThreadFactory-engine-Thread-18) [bef6940e-242f-4588-ac22-5125535808ca] START, CreateVDSCommand( CreateVDSCommandParameters:{hostId='ed843075-f8d0-4f1b-86f4-326f8de72275', vmId='0dc2a097-0347-4cbc-af10-a6b72ef69018', vm='VM [golden_env_mixed_virtio_0]'}), log id: 5714e3a2
Comment 3 Tomas Jelinek 2017-09-04 03:34:11 EDT
seems like another regression of libvitxml
Comment 4 Francesco Romani 2017-09-11 04:46:39 EDT
(In reply to Tomas Jelinek from comment #3)
> seems like another regression of libvitxml

Yes.
The Vdsm side should be fixed now with https://gerrit.ovirt.org/#/c/81610/ .
The bug is that Vdsm doesn't do the hugepages preparation in the new engine XML flow; the cleanup stage should be OK.

To fix this BZ, we need to make sure that Engine works well: it should
1. send the custom properties encoded in the domain metadata (same values as before, let's just put them in the metadata)
2. do the same additions to the domain XML that Vdsm used to.

CC Arik to make sure he's aware
Comment 5 Francesco Romani 2017-09-22 12:27:24 EDT
Vdsm patch merged, but Engine needs to send the custom properties in the domain XML metadata, once that is done we can move to MODIFIED.
Comment 6 Arik 2017-09-22 13:58:20 EDT
Engine side is merged as well
Comment 7 Francesco Romani 2017-09-25 07:53:08 EDT
regression caused by domain xml. Doesn't need doc string.
Comment 8 Artyom 2017-09-26 04:37:33 EDT
Verified on ovirt-engine-4.2.0-0.0.master.20170921184504.gitfcfc9a7.el7.centos.noarch
Comment 9 Sandro Bonazzola 2017-12-20 06:13:10 EST
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.

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