Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1470
@remibergsma can you address comments from @jburwell, especially remove the
`new String()`. I can help with further reviewing and testing once code changes
are finalized, thanks.
---
If your
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1470
merge conflicts on 4.9. :(
@remibergsma Can you merge this please?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1470
since the last CI run, there are no new commits and it has the required
reviews. I am merging this now to 4.8+
---
If your project is set up for it, you can reply to this email and have your
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1470
### ACS CI BVT Run
**Sumarry:**
Build Number 164
Hypervisor xenserver
NetworkType Advanced
Passed=73
Failed=0
Skipped=3
_Link to logs Folder (search