Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1487#issuecomment-209099808
just to finalize my report, @swill ; I isolated the failing test:
```
SSH into VM: 192.168.23.7
Trying SSH Connection: Host:192.168.23.7 User:roo
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1487
---
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 the feature is
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1487#issuecomment-208924288
Ok, perfect. I will merge this now. Thx for confirming.. :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1487#issuecomment-208917458
fine I had a failure but the re-run worked. Not very reassuring but mainly
about my test-env not the pr.
---
If your project is set up for it, you can reply t
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1487#issuecomment-208907818
@DaanHoogland I already ran tests against this one as #1449. I am planning
to merge this today...
---
If your project is set up for it, you can reply to this email
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1487#issuecomment-208819610
code LGTM running tests as we write
---
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 pro
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/1487#issuecomment-208812713
Seems like a sane Python change. Looking at the code it's OK. I can't find
an issue.
Testing it is a bit harder to do for me now.
Based on the code: L
GitHub user kollyma opened a pull request:
https://github.com/apache/cloudstack/pull/1487
speedup iptables by prefetching the variables
-- This PR is replacing speedup iptables setup #1449
-- Squashing commits and cleanup
PR against 4.7 as discussed with Remi Bergsma. T
