rhtyd commented on issue #2248: CLOUDSTACK-10056
URL: https://github.com/apache/cloudstack/pull/2248#issuecomment-324924227
 
 
   @nathanejohnson @bvbharatk @karuturi I further debugged the issue, the 
original change from #1967 should be reverted or restricted to only parameters 
that are on integer type, by performing a deploy vm test, I think we should 
restrict the code that does string->int->string conversion to only these 
details: `cpuNumber`, `cpuSpeed` and `memory`, or better yet fix the issue in 
UI that is causing double-byte issue.
   
   I deployed a VM with custom compute offering:
   ![screenshot from 2017-08-25 
15-40-32](https://user-images.githubusercontent.com/95203/29716509-07ffa75a-89ac-11e7-8244-0255a68292cd.png)
   
   And it sent the following params as details:
   ![screenshot from 2017-08-25 
15-41-18](https://user-images.githubusercontent.com/95203/29716560-15f5f6b6-89ac-11e7-8bfd-17f6f9f4aa45.png)
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to