Allon Mureinik has posted comments on this change. Change subject: core: Allow creating ISO domain on other file-based storages except nfs ......................................................................
Patch Set 4: Mark Regarding my comment in RemoveStoragePoolCommand.removeDomainFromPool(StrorageDomain, VDS): The current code does the following (excuse the pseudocode): if !localfs: detachDoamin else: // Silently assumes this is a data domain removeDomain After your change, when destroying a storage pool, all attached localfs ISO domains will be REMOVED, and not DETACHED. Is this the behaviour we want? -- 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: 4 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: Daniel Erez <[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
