mike-tutkowski commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-355701751 @rafaelweingartner Oh, it's no problem. There is so much code here to implement all of these new features and it all has to work alongside the old code, so sometimes it's easier to change as little as possible from the old code to "assure" yourself you haven't broken any of it along the way. Of course, as you do this, the new code placed on top of the old code ends up adding complexity. At some point, it's best to "suck it up," sit down, and rewrite a bunch of it to support the entire feature set, but be less complicated. To do this, though, it's nice to have a large set of automated tests to catch anything you might have broken in the code during the rewrite. I do have a lot of new tests to catch issues in the new code, but I've never been super confident in CloudStack's automated tests related to its legacy features.
---------------------------------------------------------------- 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
