Change in ovirt-engine[ovirt-engine-4.0]: core: convert memory volumes only on active snapshots to dis...

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Tue Oct 18 09:43:47 UTC 2016


Piotr Kliczewski has submitted this change and it was merged.

Change subject: core: convert memory volumes only on active snapshots to disks properly
......................................................................


core: convert memory volumes only on active snapshots to disks properly

We have no vm_configuration on snapshots of type ACTIVE, thus we need
special treatment for this case. The solution is:
- If the same memory resides on both ACTIVE and other than ACTIVE
  snapshots, process one of the snapshots that is not ACTIVE.
- If the memory resides only on ACTIVE snapshot, compute its size
  according to the values in vm_static rather than in vm_configuration.

Bug-Url: https://bugzilla.redhat.com/1382746
Signed-off-by: Arik Hadas <ahadas at redhat.com>
Signed-off-by: Marek Libra <mlibra at redhat.com>
Change-Id: I715082f267b5eaa325cc2bb8988b25ac89e3b043
---
M packaging/dbscripts/upgrade/04_00_0140_convert_memory_snapshots_to_disks.sql
1 file changed, 34 insertions(+), 3 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Michal Skrivanek: Verified; Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I715082f267b5eaa325cc2bb8988b25ac89e3b043
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Marek Libra <mlibra at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list