Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1132#issuecomment-164150715 code LGTM and @remibergsma his test succeed, as this is in 4.5 I think it must go in 4.6 as well, however it contains way to little tests and can not be guaranteed to continue working. @bhaisaab please consider continued maintenance by automation, i.e. both unit - and integration tests. merging
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---