winterhazel commented on PR #13897: URL: https://github.com/apache/cloudstack/pull/13897#issuecomment-6458325466
> Hi @winterhazel, if testing has been done for the changes that you've committed, we are good to proceed. @sandeeplocharla ok, these commits were indeed manually tested. - For 95bbde4, I validated that changing the scope of a NFS primary storage from zone to cluster (which uses the no-detail path) makes libvirt unmount and undefine the pool on hosts outside the destination cluster. - For 04bba56 and 30ae33f, I validated that a name-only pool update does not affect its details, and that updating the capacity IOPS of a SolidFireShared pool correctly persists the recalculated min_iops/max_iops/burst_iops values (PS: I do not have access to a SolidFire storage, so I emulated one by updating the entry of a NFS storage so that the code believed it was a SolidFire storage, and skipped the calls to the appliance itself). -- 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]
