mike-tutkowski commented on issue #2761: Add managed storage pool constraints to MigrateWithVolume API method URL: https://github.com/apache/cloudstack/pull/2761#issuecomment-409767259 I have to let this test suite run and come back to it later. Just an FYI, though: The test that failed before (and lead to this PR) has now passed. I'm running the entire relevant test suite, though, to see if anything else pops up. Now, this is not at all necessary, but what do you think about these suggested changes: https://github.com/mike-tutkowski/cloudstack/commit/f12e9af1bb4ab384298ffdced8f30b22e484ab13 They are not critical...just some changes I thought simplified the code a bit.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
