VCPUs_at_startup attribute not found, when deploying from Satellite 6
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.
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:
I can make a pull request, that changes the attributes, but I assume it was changed for a reason.
#2 Updated by Martin Juhl 5 months ago
I think that this bug has been solved incorrectly..
If you look at: https://github.com/xcp-ng/xen-api/blob/v21.4.0/ocaml/xapi/xapi_xenops.ml:1193 the variable is actually uses the uppercase variant..
Using the upstream 1.0.1 version fails for us, when trying to deploy, telling us that the uppercase variant of the variable can't be found...
I don't know how this has been working for @Nikola, but as far as I can tell, the variable has never been using the lower case variant..
Can we please reverse this commit?