suryag1201 commented on code in PR #14262:
URL: https://github.com/apache/cloudstack/pull/14262#discussion_r4274244027


##########
ui/src/views/storage/ResizeVolume.vue:
##########
@@ -105,14 +103,23 @@ export default {
     },
     fetchData () {
       this.loading = true
+      if (this.resource.size != null) {
+        this.form.size = this.resource.size / (1024 * 1024 * 1024)

Review Comment:
   Thanks @Damans227 for reviewing this PR
   Root and data disk sizes are entered in GB, and both parameters are LONG, so 
a fractional value is rejected before the VM or volume is created. For example:
   deployVirtualMachine with rootdisksize=3.5 returns: Unable to execute API 
command deployvirtualmachine due to invalid value 3.5 for parameter rootdisksize
   createVolume with size=5.6 returns: Unable to execute API command 
createvolume due to invalid value 5.6 for parameter size
   resizeVolume.size is also a LONG in GB. A volume created through these calls 
therefore has a whole-GB size, and that is the value prefilled on Resize.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to