Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/772#issuecomment-138505642
Hi @jayapalu
The change LGTM regarding the tests I have executed, which covers almost
the whole thing. If you tested it manually, that's fine by me.
I can help writing a test to cover it, but won't have time to do so in the
coming weeks. I would say go on with the merge as soon as you get a 2nd LGTM.
Cheers,
Wilder
---
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.
---
