Allon Mureinik has posted comments on this change. Change subject: [wip] core: OvfReader - set IsManaged to true for managed devices ......................................................................
Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) @Liron - why does this fail? .................................................... Commit Message Line 9: When reading ovf data, managed devices were added to the managed devices Line 10: map - but the isManaged flag wasn't set to true, causing that possibly Line 11: wrong data would be persisted/used. Line 12: Line 13: Related to Bug-Url: https://bugzilla.redhat.com/882644 Please replace this with a standard footer: Related-To: https://bugzilla.redhat.com/882644 Line 14: Line 15: Change-Id: Ifa51f5aa6655b5b9ae6fd1a8c5e0c206fa9630e6 -- To view, visit http://gerrit.ovirt.org/13370 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifa51f5aa6655b5b9ae6fd1a8c5e0c206fa9630e6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Liron Ar <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
