Change in ovirt-engine[master]: core: move memory related utility method to utils class

ahadas at redhat.com ahadas at redhat.com
Wed Oct 7 09:32:36 UTC 2015


Arik Hadas has submitted this change and it was merged.

Change subject: core: move memory related utility method to utils class
......................................................................


core: move memory related utility method to utils class

Move the mapping of storage domain type to memory volume type to
the MemoryUtils class.
1. It is the approapriate place for it to reside it as it is
   a utility method which is related to memory.
2. It removes wrong dependency between the memory volume builder
   for memory in live snapshots and the hiberante command.

Change-Id: I49176df3d416448e2063438ccec6fbd50b251c64
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HibernateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/LiveSnapshotMemoryImageBuilder.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryUtils.java
3 files changed, 13 insertions(+), 18 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Arik Hadas: Verified



-- 
To view, visit https://gerrit.ovirt.org/46510
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I49176df3d416448e2063438ccec6fbd50b251c64
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list