Adam Litke has uploaded a new change for review.

Change subject: schema: New type VmParameters
......................................................................

schema: New type VmParameters

Although the format of the data passed via VM.create is very similar to a
VmDefinition, some of the parameters don't make sense to pass in the context of
creating a new VM.  Solve this by creating a new type for VM parameters.

Change-Id: I00d1b9aed55cbfc2210c1a4091bce17d45b90e67
Signed-off-by: Adam Litke <[email protected]>
---
M vdsm_api/vdsmapi-schema.json
1 file changed, 48 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/39/7839/1

diff --git a/vdsm_api/vdsmapi-schema.json b/vdsm_api/vdsmapi-schema.json
index f6d3c77..b1dc60c 100644
--- a/vdsm_api/vdsmapi-schema.json
+++ b/vdsm_api/vdsmapi-schema.json
@@ -2346,6 +2346,53 @@
           'vmId': 'UUID', 'vmName': 'str', 'vmType': 'VmType'}}
 
 ##
+# @VmDefinition:
+#
+# Full virtual machine status and properties.
+#
+# @acpiEnable:            Indicates if ACPI is enabled inside the VM
+#
+# @custom:                #opional A dictionary of custom, free-form properties
+#
+# @devices:               #optional An array of VM devices requested
+#
+# @display:               The type of display
+#
+# @kvmEnable:             Indicates if KVM hardware acceleration is enabled
+#
+# @memSize:               The amount of memory assigned to the VM in MB
+#
+# @nice:                  The host scheduling priority
+#
+# @smp:                   The number of CPUs presented to the VM
+#
+# @smpCoresPerSocket:     #optional Indicates the number of CPU cores per 
socket
+#
+# @smpThreadsPerCore:     #optional Indicates the number of CPU threads per 
core
+#
+# @timeOffset:            The time difference from host to the VM in seconds
+#
+# @transparentHugePages:  Indicates if the Transparent Huge Pages feature is
+#                         enabled for this virtual machine
+#
+# @vmId:                  The VM UUID
+#
+# @vmName:                The VM name
+#
+# @vmType:                The type of VM
+#
+# Since: 4.10.0
+##
+{'type': 'VmParameters',
+ 'data': {'acpiEnable': 'bool',
+          '*custom': 'StringMap', '*devices': ['VmDevice'],
+          'display': 'VmDisplayType', 'kvmEnable': 'bool', 'memSize': 'uint',
+          'nice': 'int', 'smp': 'uint', 'smpCoresPerSocket': 'uint',
+          'smpThreadsPerCore': 'uint', 'timeOffset': 'uint',
+          'transparentHugePages': 'bool', 'vmId': 'UUID', 'vmName': 'str',
+          'vmType': 'VmType'}}
+
+##
 # @VmFullStatus:
 #
 # A VmDefinition uses the same format as a VmFullStatus.
@@ -4278,7 +4325,7 @@
 # Since: 4.10.0
 ##
 {'command': {'class': 'VM', 'name': 'create'},
- 'data': {'vmParams': 'VmDefinition'},
+ 'data': {'vmParams': 'VmParameters'},
  'returns': 'VmDefinition'}
 
 ##


--
To view, visit http://gerrit.ovirt.org/7839
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I00d1b9aed55cbfc2210c1a4091bce17d45b90e67
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to