Trouble Creating VM via API

bryonadams

It looks like if I exclude those three values it makes the VM. So maybe the better question is; if I'm taking these values from the template, am I not meant to override them? The description seems like an odd one to not be able to override if that's how it works.

lsouai-vates

@bryonadams Hello! I'm glad you finally managed to create your VM.
This message isn't very clear, I agree... In my opinion, it seems like you entered too many parameters for these endpoints, but I'll contact the XO team to improve this type of error message.
I'll also ask for information about the templates.

If you encounter any other issues with the REST API related to VM creation, please feel free to report them in this thread.

lsouai-vates

@bryonadams it seems like you are not on the most recent version of Xen Orchestra. Could you update your XO?

"excess property" errors are often wrong parameter errors.

name_description and memory are accepted but the auto_poweron parameter is not allowed because it is misspelled... the API expects autoPoweron.

bryonadams

@lsouai-vates
Xen Orchestra is telling me I'm up to date, nothing available according to the UI.

- node: 20.18.3
- npm: 10.8.3
- xen-orchestra-upload-ova: 0.1.6
- xen-orchestra-web: 0.21.1
- xo-server: 5.181.0
- xo-server-telemetry: 0.7.0
- xo-server-xoa: 0.31.2
- xo-web-free: 5.178.0
- xoa-cli: 0.40.3
- xoa-updater: 0.50.10

Also, auto_poweron is what I used because that's what the API returns as the method. That should return a key that I can functionally copy/paste into a POST. See the below GET for the actions on create_vm. The error handling is not returning helpful messages if the three keys I used are invalid for different reasons.

lsouai-vates

@bryonadams, have you checked the API Swagger?

{your URL xo}/docs/#/vms/CreateVm

As I said before, the three values you entered cannot be sent during VM creation. That's why you received this error message.

But I agree, this message is not very readable and we should improve it.We will investigate how to improve their readability in our future roadmap.

bryonadams

@lsouai-vates said in Trouble Creating VM via API:

name_description and memory are accepted but the auto_poweron parameter is not allowed because it is misspelled... the API expects autoPoweron.

Sorry, I was responding to this in particular. I was taking auto_poweron straight from the API GET endpoint. Keys or values from a GET should be usable in a POST. If these values aren't accepted during creation then it's moot, though I think description should be allowed to override so that a VM can have a meaningful description.

@lsouai-vates said in Trouble Creating VM via API:

@bryonadams, have you checked the API Swagger?

I'm not sure how to get this, I tried via web browser and just get a "Cannot GET /docs/" message.

olivierlambert

The right URL is:
https://<XO URL>/rest/v0/docs/

bryonadams

@olivierlambert That worked, and is exactly what I was looking for in the API docs but overlooked. Are there plans to put this type of API explorer/documentation on the documentation website or is the expectation that we'll interact with XO directly for documentation of API endpoints?

olivierlambert

Swagger doc is auto generated, and yes it's planned to have a button in XO 6 to access it directly. Note we communicate often (almost each release announcement) of our swagger doc.

lsouai-vates

Keys or values from a GET should be usable in a POST.

@bryonadams I understand your point of view and the reasoning behind it. However, it’s not always an absolute rule that every field returned by a GET endpoint can be passed back to a POST (or PUT).

In this case, the API error message you received clearly indicates that these parameters are “excess properties not allowed”, meaning they cannot be used as input parameters.

Indeed, it also seems you’re not running the latest version of XO. Could you please check again? You should be getting the recent REST API updates:
https://github.com/vatesfr/xen-orchestra/pull/8798

We’re currently in the process of migrating from the old REST API to the new one, so for now Swagger remains the single source of truth. As @olivierlambert mentioned, we plan to provide direct access to it from XO-6, and we’ve been communicating about this in our blog posts.

MathieuRA opened this pull request in vatesfr/xen-orchestra

Trouble Creating VM via API

closed feat(@vates/types): enhance createVm type #8798