Github user glennwagner commented on the issue:
https://github.com/apache/cloudstack/pull/1601
LGTM
Testing 4.9 master with pr 1601
1. Original results without PR KVM hosts failing to add , error in logs
were NIO connection errors
2. After the PR
Github user glennwagner commented on the pull request:
https://github.com/apache/cloudstack/pull/1422#issuecomment-193666093
Made Simple - LGTM - Can we look at making these Fields configurable?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user glennwagner commented on the pull request:
https://github.com/apache/cloudstack/pull/1398#issuecomment-193325954
LGTM no issues
---
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
Github user glennwagner commented on the pull request:
https://github.com/apache/cloudstack/pull/1414#issuecomment-193156160
Had a look ta your changes, PR 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
Github user glennwagner commented on the pull request:
https://github.com/apache/cloudstack/pull/1399#issuecomment-192446723
LGTM - HTTP added to GSLB
---
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
Github user glennwagner commented on the pull request:
https://github.com/apache/cloudstack/pull/1394#issuecomment-192290945
LGTM - Tested and working correctly
---
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 glennwagner commented on the pull request:
https://github.com/apache/cloudstack/pull/1363#issuecomment-174850592
This looks good , Will test it in the LAB today
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well