SadiJr opened a new pull request, #6386:
URL: https://github.com/apache/cloudstack/pull/6386

   Using the VMWare hypervisor, when trying to create IOPS-limited VMs, either 
using Compute or Disk offerings, the IOPS limits were not respected in the 
hypervisor. This behavior has been corrected, in order to be able to limit, in 
VMWare, the IOPS limit of a VM, as long as the option to limit IOPS via 
hypervisor is used. Also, I added a new method to LogUtils class to enable to 
log objects using Gson without exception, to improve logs information.
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [x] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [x] Minor
   
   
   ### Screenshots (if appropriate):
   
   
![root-and-data-disks](https://user-images.githubusercontent.com/31869303/168104602-6472fb2c-12da-4080-83bc-ad2cb47b4000.png)
  
   
   
   
![root-and-data-disks-vmware](https://user-images.githubusercontent.com/31869303/168104660-3c1619dd-40c8-4ee1-af93-dbdc42abe325.png)
   
   
   ### How Has This Been Tested?
   It was tested in a local lab:
   
   1. I created three new offerings: one Compute Offering with IOPS limit,  one 
disk offering with IOPS limit and one disk offering without IOPS limit;
   2. I created a new VM using this compute and disks offerings;
   4. Before, the IOPS limitation was not applied;
   5. Now, the IOPS limitations are being applied.
   Also, I added new unit tests.


-- 
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