Change in vdsm[master]: vm: make _set_lastStatus safer

fromani at redhat.com fromani at redhat.com
Fri Dec 5 12:30:15 UTC 2014


Francesco Romani has posted comments on this change.

Change subject: vm: make _set_lastStatus safer
......................................................................


Patch Set 1: Verified+1

Verification: addressing a race, there is still room for uncertainity. However:
- basic sanity check: booted VM, recovered VM, migration
- verification as per BZ entry: tried mass migration of VMs (10 at the same time) an handful of time (~10 times) with no error spotted besides the expected noise when QEMU dies.

NOTE:
- verified on a patched 3.5 environment, but relevant code is identical to master
- on the same environment VDSM consistently broke before.

-- 
To view, visit http://gerrit.ovirt.org/35896
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I172c6a4d4bae156294f7f8944912c877f55ab5a0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Vitor de Lima <vdelima at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list