[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues closed the pull request at: https://github.com/apache/cloudstack/pull/1259 --- 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 f

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-12971 This PR is no longer needed as the PR #1277 contains the same fixes, but has been created against branch 4.7. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166619798 Ping @remibergsma @miguelaferreira @borisroman @michaelandersen Test are fine. It seems the the hiccups on network were caused by my environment. It

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166605170 wget google.nl works fine as well. ``` # wget google.com --2015-12-22 12:20:47-- http://google.com/ Resolving google.com... 2a00:1450:4

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166560296 Ping @remibergsma @miguelaferreira @borisroman @michaelandersen One test failed: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false. This tes

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166401238 Hey @wilderrodrigues that's awesome! Would you mind pointing this PR to the 4.7 branch? I created it today and it finally got mirrored :-) --- If your project

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166383593 Hi @remibergsma @miguelaferreira @DaanHoogland @borisroman False alarm! After the crash, when I rebooted my Test Environment the virbr0.50 was not

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166357140 Ping @remibergsma @miguelaferreira @DaanHoogland @borisroman The changes in this PR affected a couple of of RVRs. I got tests failing (default allo

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166244325 I had problems when running the tests: my test environment crashed. Could not test it before because I was on my way to France. Will restart the tests now.

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166014008 Thanks, @DaanHoogland ! :) My other tests are still running. Will post results here once they are done. --- If your project is set up for it, you c

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166010459 @wilderrodrigues my bad: ..-pr-.. means cloudstack-pr-analysis, ..-pull-.. means cloudstack-pull-analysis. The new job is a generic job does a rats tes

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166010092 Sorry @DaanHoogland, I don't understand what you meant. :) pr/pull jobs related to my previous comment? Or is it something preventing you to test the PR? So

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166009331 @wilderrodrigues the ..-pr-.. jobs are an attempt from me to deal with all the errors in the ..-pull-.. jobs and the lack of rats test. By now it seems the ..-

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-166005919 From here I see that the build was successful: https://builds.apache.org/job/cloudstack-pr-analysis/162/console ``` [INFO]

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1259#issuecomment-165997403 Ping @remibergsma @DaanHoogland @borisroman @miguelaferreira @bhaisaab @karuturi * Environment ``` Management Server on CentOS 7.1

[GitHub] cloudstack pull request: Critical VPCVR issues fixed: CLOUDSTACK-9...

2015-12-19 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/1259 Critical VPCVR issues fixed: CLOUDSTACK-9154; CLOUDSTACK-9187; and CLOUDSTACK-9188 This PR fixes: * CLOUDSTACK-9154: rVPC doesn't recover from cleaning up of network garbage c