[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/952 --- 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: Improve marvin test site2site VPN

2015-10-22 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150173017 The exceptions: error: [Errno 113] No route to host error: [Errno 111] Connection refused until ===SSH to Host 192.168.23.12 port : 22 SUCCESS

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150171835 New test run, same result: ``` [mct-run-marvin-tests@3] Running shell script + nosetests --with-xunit --xunit-file=integration-test-results/smoke/

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150162235 @wilderrodrigues I think it was, in the end I ran only the smoke test suite and even that got errors so please ignore for this PR --- If your project is set up

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150161422 That's what I thought. The JUnit report plugin in Jenkins sais the test passed. (See last line of table in picture) ![screen shot 2015-10-22 at 11 13

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150160922 Hi @miguelaferreira The system-err is because the Marvin code prints all the ssh commands to the terminal. It doesn't mean it failed. All the SSH re

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150041972 did the check-pr test on this one but faied rerunning to further investigate --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-21 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150027934 thanks for testing @miguelaferreira and @wilderrodrigues for chasing this pr haven't seen this type of report before but looking at the combination of:

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-21 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149990698 I ran the test against master with additional PRs (**#952**, #958, #951, #957, #940) and the result was `SUCCESSFUL`: ```xml

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-21 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149961213 @wilderrodrigues running it now (together with a bunch of other tests). I'll report the results tomorrow. --- If your project is set up for it, you can repl

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149958354 Ping @remibergsma @karuturi @DaanHoogland @borisroman @wido @bhaisaab @miguelaferreira @michaelandersen @bvbharat @runseb @terbolous This is a test:

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-20 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149694171 @remibergsma: why another run? Jenkins (and travis) is green --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-20 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149508039 Please have a look at Jenkins and/or force push to trigger another run. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-20 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149487696 Run the tests and it LGTM :+1: ``` Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS === ok

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-20 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149457564 Testing this one now! --- 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 no

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-19 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149443237 @michaelandersen Thank you! Can you please rebase with current master? Will test it afterwards :-) --- If your project is set up for it, you can reply to this e

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-19 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149249379 Thanks, @michaelandersen I will have a look at this one today! Cheers, Wilder --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-19 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-149238771 run with: nosetests --with-marvin --log-folder-path=/tmp/marvin/ --marvin-config=/data/shared/marvin/mct-zone1-kvm1.cfg -a tags=advanced test/integratio

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-19 Thread michaelandersen
GitHub user michaelandersen opened a pull request: https://github.com/apache/cloudstack/pull/952 Improve marvin test site2site VPN Add optional fields: iprange and fordisplay to Marvin base.py class method Vpn.create Add optional field: passive to Marvin base.py class method Vp