From Martin Sivák <msivak(a)redhat.com>:
Martin Sivák has submitted this change and it was merged. (
https://gerrit.ovirt.org/79504
)
Change subject: common: Fix HugePagesUtils units (KiB vs MiB) and make it static
......................................................................
common: Fix HugePagesUtils units (KiB vs MiB) and make it static
This fixes an unit mismatch as hugepage size is reported in
kilobytes and the memory size in megabytes.
The unnecesary injection is also removed to make this available
from non-CDI enabled contexts.
Bug-Url:
https://bugzilla.redhat.com/1461476
Change-Id: Ibab4e0225c3de1fe04d145ae58887285b94a41f5
Signed-off-by: Martin Sivak <msivak(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/HugePageUtils.java
M
backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/HugePageUtilsTest.java
2 files changed, 32 insertions(+), 24 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, but someone else must approve
Martin Sivák: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
Andrej Krejcir: Looks good to me, but someone else must approve
--
To view, visit
https://gerrit.ovirt.org/79504
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibab4e0225c3de1fe04d145ae58887285b94a41f5
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>