[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-05 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1802 LGTM. Merging this now, @murali-reddy in case we hit the issue, we'll need to rework the fix. Thank you for the PR. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-04 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 This is good to be merged now. test_privategw_acl.py is passing mostly. there are still intermittent failures (not addressed by this PR) in the test_privategw_acl.py like create and delete

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Trillian test result (tid-573) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33703 seconds Marvin logs:

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-04 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test centos7 vmware-55u3 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-02 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-320 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-02 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-02 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy please push -f on the PR to re-kick travis, or close/open the PR @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy can you have a look at why private_gw failed again for vmware? --- 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 issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Trillian test result (tid-537) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34106 seconds Marvin logs:

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Trillian test result (tid-536) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 25203 seconds Marvin logs:

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1802 Coupled with @rhtyd's explanation, we can merge this PR if the current blueorganutan run comes up clean. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1802 @jburwell the issue was with an intermittent test that sometimes fail when clocks get skewed in virtualized environment: ` HypervisorUtilsTest.checkVolumeFileForActivityTest:68 Didn't block long

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @jburwell a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test matrix --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy @rhtyd can you investigate the Travis failures? --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-300 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Packaging result: ✔centos6 ✖centos7 ✖debian. JID-299 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan package --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy agreed that it is extremely unlikely that the code change impacts that test case. However, we know that a failure to cleanup between tests can cause failures when they are run

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @jburwell This PR is not rebased to fix from #1801. This PR is confined to test_privategw_acl.py, so no scope of any regression out side of it. --- If your project is set up for it, you

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy I see failures in the `test_router_dhcp_opts` test case. Was this PR rebased to get the fix from #1801? If not, I would like to rebase and re-run the tests. Otherwise, we may

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Trillian test result (tid-517) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31158 seconds Marvin logs:

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @jburwell this PR is ready for merge. PR fixes intermittent failures of suite of tests in test_privategw_acl.py. test_04_rvpc_privategw_static_routesSuccess 553.97

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Trillian test result (tid-515) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 26871 seconds Marvin logs:

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-12-01 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1802 LGTM based on code review. --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1802 LGTM, let's wait for the result. @blueorangutan test centos7 vmware-55u3 --- 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 issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-291 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan package --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy a Trillian-Jenkins test job (centos6 mgmt + kvm-centos6) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test centos6 kvm-centos6 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan test centos7 kvm-centos6 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 Packaging result: ✔centos6 ✖centos7 ✔debian. JID-290 --- 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

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1802 @murali-reddy a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1802: CLOUDSTACK-9635: fix test_privategw_acl.py

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1802 @blueorangutan package --- 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