Plugins » Xen

When deploying a server from Satellite, we ran into an error that VCPUS_at_startup wasn't defined, and the deployment was rolled back. Looking through the source code, it appears that the commit that fixed issue 28003, changed the mixed case to lower case (snake case). In our own experiments we changed the casing back to mixed casing, and we could successfully deploy the server.

https://github.com/theforeman/foreman-xen/commit/26e9206804ae7db206167d61987caa85e1921c1c

We are running xcp-ng version: 8.2
Foreman-xen is 1.0.1

As example in xenadmin, the windows admin panel, the xenapi uses mixed case:

https://github.com/xenserver/xenadmin/blob/c506c5e47c110760d62a03f72775217943e3da54/XenModel/XenAPI/VM.cs

I can make a pull request, that changes the attributes, but I assume it was changed for a reason.

Best regards,
Jonas