Liron Aravot has posted comments on this change. Change subject: core: End VM export gracefully upon failure in updating OVF ......................................................................
Patch Set 1: Tal, i'm totally sure what is preferred here. It's very rare that under normal circumstances an export would fail on the ovf upload phase - the scenario described in the bug is caused by a "broken" export domain and maybe that's something we need to fix regardless (better export domain monitoring). My first instinct is toward just leaving the situation as is (if export fails on that, it's better to keep retrying or to check what is wrong). If you want to revert on that case it's also fine by me, but i'm not 100% sure about that. Allon - yes, we would use coco for that on 3.6. -- To view, visit https://gerrit.ovirt.org/37583 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iefa719e8872a69de5b3d3fa79758e12e7b63f80f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Amit Aviram <[email protected]> Gerrit-Reviewer: Candace Sheremeta <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Fred Rolland <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Idan Shaby <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
