Github user kishankavala commented on the issue:
https://github.com/apache/cloudstack/pull/1379
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 cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1379
### ACS CI BVT Run
**Sumarry:**
Build Number 362
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=7
_Link to logs Folder
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1379
Code changes look ok to me 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
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1379
These scripts CI tests are not needed
---
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
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1379
These scripts are for config drive feature. Deploy vm in shared network
without services then vm with config drive will be deployed.
Place these scripts in vm template to set VM password
