[GitHub] [cloudstack] ravening commented on pull request #4220: Fix cpuallocated value in findHostsForMIgration api

2020-08-05 Thread GitBox
ravening commented on pull request #4220: URL: https://github.com/apache/cloudstack/pull/4220#issuecomment-669061434 > @ravening sorry I missed your comment, and merged based on @GabrielBrascher 's `I am good with the code as it is`. Do you need to do more work on this PR, should we

[GitHub] [cloudstack] ravening commented on pull request #4220: Fix cpuallocated value in findHostsForMIgration api

2020-08-04 Thread GitBox
ravening commented on pull request #4220: URL: https://github.com/apache/cloudstack/pull/4220#issuecomment-668505963 > LGTM, did not manually test it @rhtyd @DaanHoogland its not yet ready for merge. Im waiting for @GabrielBrascher reply for my question. If you can also reply it