[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-27 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-222195257 @luhaijiao I'd recommend trying e7a63be Port forwarding works for me. I've not tried the VPN functionality. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-27 Thread luhaijiao
Github user luhaijiao commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-222182040 @dsclose we installed your updated commits d582358 and bf285e1 ONLY in our environment to solve the RvR issue (VR network services intermittent hang due to

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-27 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-222085574 @swill agreed; in particular, my note concerning the merge of lines 299 and 300 - i've not even tested that locally - not had a chance to even figure out what it's

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221904409 I actually had a test running overnight for this one and it had similar results. ``` Tests Run: 85 Skipped: 0 Failed: 14 Errors: 6

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221901259 I will retest this one. Thanks for kicking it off again, we are green now. :) --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221867807 @swill no worries. Happy to do that - the CI output was vast and I wasn't looking forward to combing through it. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread dsclose
GitHub user dsclose reopened a pull request: https://github.com/apache/cloudstack/pull/1519 Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces This PR addresses CLOUDSTACK-9339 and may need a code review from someone familiar with the System VM scripts. In

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread dsclose
Github user dsclose closed the pull request at: https://github.com/apache/cloudstack/pull/1519 --- 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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221862724 @dsclose sorry to do this to you. Can you close and reopen again to kick off the jobs again? Thanks... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread dsclose
GitHub user dsclose reopened a pull request: https://github.com/apache/cloudstack/pull/1519 Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces This PR addresses CLOUDSTACK-9339 and may need a code review from someone familiar with the System VM scripts. In

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-26 Thread dsclose
Github user dsclose closed the pull request at: https://github.com/apache/cloudstack/pull/1519 --- 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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221637516 @dsclose can you kick it off again. It is just a timeout which caused it to fail, so kicking it off again could fix it. `No output has been received in the last

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221611699 Same issue. I don't think this is related to the PR. I'll leave this be for now. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221600590 > No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself. > >The build has been

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
GitHub user dsclose reopened a pull request: https://github.com/apache/cloudstack/pull/1519 Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces This PR addresses CLOUDSTACK-9339 and may need a code review from someone familiar with the System VM scripts. In

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
Github user dsclose closed the pull request at: https://github.com/apache/cloudstack/pull/1519 --- 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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
Github user dsclose closed the pull request at: https://github.com/apache/cloudstack/pull/1519 --- 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

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
GitHub user dsclose reopened a pull request: https://github.com/apache/cloudstack/pull/1519 Cloudstack 9339: Virtual Routers do not handle Multiple Public Interfaces This PR addresses CLOUDSTACK-9339 and may need a code review from someone familiar with the System VM scripts. In

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
Github user dsclose commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1519#discussion_r64537215 --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py --- @@ -288,14 +290,16 @@ def post_configure(self, address): """ The

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-25 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221501553 @swill taking a look 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 not

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221278559 @dsclose if you can fix the merge conflicts I can run this again and see what is outstanding. Thanks... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-24 Thread luhaijiao
Github user luhaijiao commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-221202635 @dclose I think this PR is important for RvR environment too, particularly to the commits d582358 and bf285e1. Would be very nice to have it in 4.9 if you can

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-12 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-218842740 @dsclose we have merge conflicts on this one now. Also, prior to merging the PRs that caused the conflicts, I ran the above CI. You will probably want to review

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-12 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-218842214 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 9 Errors: 10 Duration: 10h 09m 15s ``` **Summary of

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-06 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-217495575 @swill ok, thanks, good luck with the master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-06 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-217491038 There is an issue currently on master which I am trying to get sorted out, but there should not be a problem on `4.7`. It looks like Jenkins may be out of diskspace

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-06 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-217490305 Same error over at #1519 - something introduced in an earlier commit? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-06 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-217489945 I rebased against the latest 4.7 before force pushing. Has an error been introduced along the way? --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers do not h...

2016-05-06 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-217397801 Looks like the build environment isn't sufficient on jenkins-test-a20: ``` [ERROR] Java HotSpot(TM) 64-Bit Server VM warning: Insufficient space for