NuKVM/xen-orchestra
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(xo-server/vm.set): cast logical numbers to XAPI string values

Browse Source
This commit is contained in:
Julien Fontanet
2019-05-27 11:10:19 +02:00
parent 453c329f14
commit 49e1b0ba7e
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/xo-server/src/xapi/index.js
View File

@@ -1577,7 +1577,8 @@ export default class Xapi extends XapiBase {
async setVcpuWeight(vmId, weight) {
await this.getObject(vmId).update_VCPUs_params(
'weight',
weight || null // Take all falsy values as a removal (0 included)
// Take all falsy values as a removal (0 included)
weight ? String(weight) : null
)
}

6
packages/xo-server/src/xapi/mixins/vm.js
View File

@@ -294,7 +294,7 @@ export default {
coresPerSocket: {
set: (coresPerSocket, vm) =>
vm.update_platform('cores-per-socket', coresPerSocket),
vm.update_platform('cores-per-socket', String(coresPerSocket)),
},
CPUs: 'cpus',
@@ -318,7 +318,7 @@ export default {
cpuCap: {
get: vm => vm.VCPUs_params.cap && +vm.VCPUs_params.cap,
set: (cap, vm) => vm.update_VCPUs_params('cap', cap),
set: (cap, vm) => vm.update_VCPUs_params('cap', String(cap)),
},
cpuMask: {
@@ -443,7 +443,7 @@ export default {
`The different values that the video RAM can take are: ${XEN_VIDEORAM_VALUES}`
)
}
return vm.update_platform('videoram', videoram)
return vm.update_platform('videoram', String(videoram))
},
},