Michal Skrivanek has posted comments on this change.
Change subject: vm: sync device update during creation of a vm
......................................................................
Patch Set 2: Code-Review-1
we can't merge it as is; the path preparation may take ages (the timeout for migration
destination is 42s IIRC)
what has changed here? It should not be a problem to change the values as long as the
length of lists remain. Or at least that's my understanding
I think we still need the answer why the heck this was ever working without problems. What
has changed?
--
To view, visit
http://gerrit.ovirt.org/34599
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I477551500ccc2297eb0c05d6562710bc420363a5
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No