[GitHub] cloudstack pull request: Fix error message in test_isolate_network...

2015-10-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/972 --- 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] cloudstack pull request: Fix error message in test_isolate_network...

2015-10-23 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/972#issuecomment-150668944 Ok, here they are: ``` === TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS === ok === TestName: test_RVR_Network_FW_PF_

[GitHub] cloudstack pull request: Fix error message in test_isolate_network...

2015-10-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/972#issuecomment-150635269 Thanks @miguelaferreira, run the tests and it works fine! LGTM ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_

[GitHub] cloudstack pull request: Fix error message in test_isolate_network...

2015-10-23 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/972#issuecomment-150602167 I've tested this in a 2 KVM hosts setup (a.k.a [mct-zone1-kvm1-kvm2.cfg](https://github.com/schubergphilis/MCT-shared/blob/master/marvin/mct-zone1-kvm1-kvm2.c

[GitHub] cloudstack pull request: Fix error message in test_isolate_network...

2015-10-23 Thread miguelaferreira
GitHub user miguelaferreira opened a pull request: https://github.com/apache/cloudstack/pull/972 Fix error message in test_isolate_network_FW_PF_default_routes While running test_isolate_network_FW_PF_default_routes it is expected that SSH'ing into a VM does not work immediately.