[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 Trillian test result (tid-90) Environment: xenserver-65sp1 (x2), Advanced Networking Total time taken: 29144 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releas

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread NuxRo
Github user NuxRo commented on the issue: https://github.com/apache/cloudstack/pull/1692 Hi guys, sorry this has caught me in my holidays so no ability to do anything about it. In the following week I'll work on a new minimal kernel with improved support for VMware stuff.

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-30 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 @rhtyd sure, will do and post results --- 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 pull request #1542: CLOUDSTACK-9379: Support nested virtualizatio...

2016-09-30 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1542#discussion_r81407458 --- Diff: test/integration/smoke/test_nested_virtualization.py --- @@ -0,0 +1,152 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1542 @nvazquez you're welcome. If you've some bandwidth, can you run your test against this template which will also help validate the new vmware test appliance: https://github.com/rhtyd/tinylinux --

[GitHub] cloudstack pull request #1542: CLOUDSTACK-9379: Support nested virtualizatio...

2016-09-30 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1542#discussion_r81405215 --- Diff: test/integration/smoke/test_nested_virtualization.py --- @@ -0,0 +1,152 @@ +# Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-30 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 @jburwell @rhtyd I included new marvin test for nested virtualization, thanks @rhtyd for your idea! --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 @jburwell based on the latest result results, the errors have significantly gone down. We will still need to re-investigate some of the error (which I thought were fixed, but still failed). ---

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 Trillian test result (tid-89) Environment: vmware-55u3 (x2), Advanced Networking Total time taken: 26541 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/d

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 @PaulAngus @jburwell I've created a tinycore linux basd template here with e1000, e1000e and vmxnet3 driver loaded (lsmod to confirm). Once started, it ships only with openssh server and dhcp ena

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1696 @rhtyd I might be mistaken but you also need to add mapping into cloudstack/vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareGuestOsMapper In the past if not added on vmware it would

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 Trillian test result (tid-88) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 24192 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/d

[GitHub] cloudstack pull request #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 a...

2016-09-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1696 --- 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 issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 The failures from the above result are not related to the change but with the env/tests, most of which have been addressed in #1692 With the enough test results, manual tests and lgtms, I

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1696 Trillian test result (tid-86) Environment: kvm-centos7 (x2), Advanced Networking Total time taken: 24408 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/d

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 @jburwell we may ignore that specific test, or retest the vmnet3 test on vmware against the default built-in (centos5 based) template. @NuxRo ping, can you help fix the macchanina template to incl

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1692 I agree with @PaulAngus on URL fix. So if we want to have this PR to get in asap I am on board but some smoke tests will continue failing until driver issue in the template is addressed --- If

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread PaulAngus
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1692 @jburwell you need both the url fix (as the original file type was not valid) AND the macchininia template to have the e1000 driver in it. for the tests to work on vSphere. Nux said t

[GitHub] cloudstack issue #1697: CLOUDSTACK-4858 Honors the snapshot.backup.rightafte...

2016-09-30 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1697 @nathanejohnson we making a hard push on #1692 to get all of the smoke tests to be super green. Once that PR is merged up, we will rebase and re-test this PR and there will be no more expected

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1692 @serg38 raises a good point. @rhtyd do we explicitly need that commit to get the smoke tests to pass? As I understand, the macchininia template is (very nice) performance optimization. --- I

[GitHub] cloudstack issue #1697: CLOUDSTACK-4858 Honors the snapshot.backup.rightafte...

2016-09-30 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1697 Here are results running the following tests: component/test_snapshots.py component/test_snapshots_improvement.py component/test_snapshot_gc.py component/test_snapshot_limits.py

[GitHub] cloudstack issue #1697: CLOUDSTACK-4858 Honors the snapshot.backup.rightafte...

2016-09-30 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1697 @jburwell @kiwiflyer @bvbharatk This is the new PR, I will post test results soon. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request #1697: CLOUDSTACK-4858 Honors the snapshot.backup.ri...

2016-09-30 Thread nathanejohnson
GitHub user nathanejohnson opened a pull request: https://github.com/apache/cloudstack/pull/1697 CLOUDSTACK-4858 Honors the snapshot.backup.rightafter configuration v… …ariable Unhides snapshot.backup.rightafter from global configuration If snapshot.backup.right

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1692 The same concerns about macchinina template remains. Since it doesn't have correct E1000/vmxnet3 driver embedded any smoke test in Vmware environment will continue failing. --- If your project

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-30 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1684 @rhtyd I agree with you for the documentation part. For sure in-place upgrade must be transparent to user and not required manual steps. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

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

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 Thanks @ustcweizhou --- 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 s

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 @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 appe

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 @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 and wishes

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1696 tested on KVM, LGTM --- 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 wis

[GitHub] cloudstack pull request #:

2016-09-30 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/commit/2ed1df0186776c3ea592284cd6a28eca2b84bb34#commitcomment-19238615 @rhtyd can you please open a JIRA ticket for this issue and amend the commit message to include it? --- If your project is

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

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

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 @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 appe

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 @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 and wishes

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 Reran tests with fixed tests: Test | Result | Time (s) --- | --- | --- test_router_dhcphosts | Success | 248.223 test_isolate_network_password_server | Success | 59.451 --- I

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1696 LGTM 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 enabled a

[GitHub] cloudstack issue #1688: Marvin Tests: Fix VPC network offering selection

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1688 I've moved the changes to PR #1692 this can be closed 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 issue #1650: fix macchinina template specificied for vmware in Ma...

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1650 I've moved the changes to PR #1692 this can be closed 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 issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1696 @rhtyd 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 reply ap

[GitHub] cloudstack issue #1696: [4.9/lts] CLOUDSTACK-9364: Add Ubuntu 16.04 as a gue...

2016-09-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1696 @ustcweizhou @abhinandanprateek @murali-reddy @jburwell review? Manual tests confirm that it works on XenServer 6.5. @blueorangutan test --- If your project is set up for it, you can re