Github user harikrishna-patnala commented on the issue:
https://github.com/apache/cloudstack/pull/874
tag:mergeready
---
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
Github user kishankavala commented on the issue:
https://github.com/apache/cloudstack/pull/874
LGTM
---
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
Github user harikrishna-patnala commented on the issue:
https://github.com/apache/cloudstack/pull/874
The are 2 failed test cases which are failing on all other PRs
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/874
### ACS CI BVT Run
**Sumarry:**
Build Number 411
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=2
Skipped=7
_Link to logs Folder
Github user harikrishna-patnala commented on the issue:
https://github.com/apache/cloudstack/pull/874
I have updated the PR based on the comments. 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
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/874
### ACS CI BVT Run
**Sumarry:**
Build Number 105
Hypervisor xenserver
NetworkType Advanced
Passed=72
Failed=1
Skipped=3
_Link to logs Folder