Dan Kenigsberg has posted comments on this change.
Change subject: check disk shared flag exist before use it ......................................................................
Patch Set 1: Looks good to me, approved
Unfortunately, we cannot re-spin ovirt-3.1 so that Engine avoids its buggy behavior.
I know that there are Vdsm developers that would not like Vdsm to "cave in". I am afraid that without them stating their opinion here, I cannot take their sentiments too deeply.
Thanks, Royce.
-- To view, visit http://gerrit.ovirt.org/7051 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Ib0d9378de9ea8affd2f5d071839ee18b53a8b2ef Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv lvroyce@linux.vnet.ibm.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Eduardo ewarszaw@redhat.com Gerrit-Reviewer: Royce Lv lvroyce@linux.vnet.ibm.com Gerrit-Reviewer: oVirt Jenkins CI Server