Allon Mureinik has posted comments on this change. Change subject: Allow creating ISO domain on localfs ......................................................................
Patch Set 3: Hi Mark, Sorry for the clattered review, but a few more issues I've noticed: In org.ovirt.engine.core.bll.storage.RemoveStoragePoolCommand.removeDomainFromPool(StorageDomain, VDS), we should not delete a local ISO domain just because it's attached to the pool we're removing. -- To view, visit http://gerrit.ovirt.org/12687 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2a8d3ea8ab4ac10353ec8574287458e8eb63e882 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mark Wu <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Sharad Mishra <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
