Adam Litke has posted comments on this change. Change subject: schema: New type VmParameters ......................................................................
Patch Set 7: (1 inline comment) .................................................... Commit Message Line 7: schema: New type VmParameters Line 8: Line 9: Although the format of the data passed via VM.create is very similar to a Line 10: VmDefinition, some of the parameters don't make sense to pass in the context of Line 11: creating a new VM. Solve this by creating a new type for VM parameters. For example: displayIP, displayPort, pid, etc. These are not selectable by and end user when creating a vm, but rather, are assigned by vdsm and represent run-time properties of a VM. Would you be happier with 'VmCreateParameters' to be the set of parameters for creating a VM and 'VmProperties' to represent the run-time state of the VM? Line 12: Line 13: Change-Id: I00d1b9aed55cbfc2210c1a4091bce17d45b90e67 -- To view, visit http://gerrit.ovirt.org/7839 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I00d1b9aed55cbfc2210c1a4091bce17d45b90e67 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Royce Lv <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: ShaoHe Feng <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
