dredknight opened a new issue, #8847:
URL: https://github.com/apache/cloudstack/issues/8847

   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and main branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete 
the comments.
   -->
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Bug Report
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   API
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on main 
branch.
   -->
   
   ~~~
   4.17.2
   ~~~
   
   ##### CONFIGURATION
   <!--
   Information about the configuration if relevant, e.g. basic network, 
advanced networking, etc.  N/A otherwise
   -->
   Using XCP-NG 8.2 hypervisor
   CloudMonkey 6.2
   
   ##### OS / ENVIRONMENT
   <!--
   Information about the environment if relevant, N/A otherwise
   -->
   
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   Using parameter extraconfig in deployVirtualMachine and updateVirtualMachine 
does not update 'platform' field
   
   ##### STEPS TO REPRODUCE
   <!--
   For bugs, show exactly how to reproduce the problem, using a minimal 
test-case. Use Screenshots if accurate.
   
   For new features, show how the feature would be used.
   -->
   
   <!-- Paste example playbooks or commands between quotes below -->
   ~~~
   Using cloudmonkey create new instance on the XCP-NG hypervisor with added 
extra argument platform:exp-nested=true which enables nested virtualization. 
The value is UTF8 encoded as required by the command.
   ```
   deploy virtualmachine templateid=b2b4c2ac-60a4-4bb0-9bb6-59cb28e793de 
name=test_nested networkids=1dfcc69a-1072-4410-881d-3818a75bcce8 
displayname=test_nested domainid=05b38e5c-8041-11ed-a10f-d214862adf28 
serviceofferingid=67e9d5bb-ed5d-4d97-9766-cc488de233a3 
zoneid=6130f25a-8dd4-469c-8964-45b2d401884d  details[0].cpuNumber=2 
details[0].memory=2048 extraconfig=platform%3Aexp-nested-hvm%3Dtrue
   ```
   The issue is also valid for using update command on existing vm instance.
   ```
   update virtualmachine id=4d188b3b-e3cd-4920-b33c-cdb502d6478d 
extraconfig=platform%3Aexp-nested-hvm%3Dtrue
   ```
   
   The current workaround is using **details** argument. The problem is that 
the value overwrites the existing settings instead of appending. Unfortunately 
I have not found a better solution.
   ```
   deploy virtualmachine templateid=b2b4c2ac-60a4-4bb0-9bb6-59cb28e793de 
name=test_nested networkids=1dfcc69a-1072-4410-881d-3818a75bcce8 
displayname=test_nested domainid=05b38e5c-8041-11ed-a10f-d214862adf28 
serviceofferingid=67e9d5bb-ed5d-4d97-9766-cc488de233a3 
zoneid=6130f25a-8dd4-469c-8964-45b2d401884d  details[0].cpuNumber=2 
details[0].memory=2048 
details[0].platform="device-model:qemu-upstream-compat;vga:std;videoram:8;apic:true;viridian:false;timeoffset:0;pae:true;acpi:1;hpet:true;secureboot:false;nx:true;exp-nested-hvm:true"
 
   
   update virtualmachine id=4d188b3b-e3cd-4920-b33c-cdb502d6478d  
details[0].cpuNumber=2 details[0].memory=2048  
details[0].platform="device-model:qemu-upstream-compat;vga:std;videoram:8;apic:true;viridian:false;timeoffset:0;pae:true;acpi:1;hpet:true;secureboot:false;nx:true;exp-nested-hvm:true"
   ```
   
   ~~~
   
   <!-- You can also paste gist.github.com links for larger files -->
   
   ##### EXPECTED RESULTS
   <!-- What did you expect to happen when running the steps above? -->
   
   ~~~
   Expected result is to append the new keypair into the existing "platform" 
variable.
   ```
   "details": {
           "cpuNumber": "2",
           "memory": "2048",
           "platform": 
"device-model:qemu-upstream-compat;vga:std;videoram:8;apic:true;viridian:false;timeoffset:0;pae:true;acpi:1;hpet:true;secureboot:false;nx:true;exp-nested-hvm:true"
         },
   ```
   ~~~
   
   ##### ACTUAL RESULTS
   <!-- What actually happened? -->
   
   <!-- Paste verbatim command output between quotes below -->
   ~~~
   The output of this command shows that platform field is appended the new 
parameter as it should per documentation, instead a new field called 
extraconfig-1 is added.
   "details": {      "Message.ReservedCapacityFreed.Flag": "true",
         "cpuNumber": "2",      
         "extraconfig-1": "platform:exp-nested-hvm=true",
         "memory": "2048",
         "platform": 
"device-model:qemu-upstream-compat;vga:std;videoram:8;apic:true;viridian:false;timeoffset:0;pae:true;acpi:1;hpet:true;secureboot:false;nx:true"
       },
   ~~~
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to