DaanHoogland commented on PR #6540: URL: https://github.com/apache/cloudstack/pull/6540#issuecomment-1178818421
> > clgtm we should go through the test files and assess whether all tests make sense/are valid. `test_deploy_vgpu_vm` seems excesive in comparison to its niche. The minimal change is certainly in line with the scope of this PR though! > > Yes @DaanHoogland, I've gone through them and made the changes in the test files. By code and usage both APIs look similar. Let's wait for the smoke test results. Don't worry @harikrishna-patnala I do not mean for this PR to change but just to put a mental note to us all. -- 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]
