Github user niteshsarda commented on the issue:
https://github.com/apache/cloudstack/pull/1850
I have tested this and LGTM for test.
---
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 featu
Github user sudhansu7 commented on the issue:
https://github.com/apache/cloudstack/pull/1850
@koushik-das
Added marvin test. Please review.
---
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 hav
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1850
Code changes LGTM. Can you add some unit/marvin tests for the scenario that
you have mentioned?
---
If your project is set up for it, you can reply to this email and have your
reply appear