[GitHub] cloudstack issue #1728: CLOUDSTACK-9551: Move java tmp dir to cloudstack-age...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1728 @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 #1728: CLOUDSTACK-9551: Move java tmp dir to cloudstack-age...

2016-10-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1728 LGTM. @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 #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1724 @murali-reddy 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

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1724 LGTM on code. --- 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

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1724 @murali-reddy thanks, I've added a cleanup step b/w tests to ensure old resources can be cleared. Is there a way to modify/run the test to avoid any such intermittent error?

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1724 all tests in test_privategw_acl.py passed on KVM & XEN except for test_02_vpc_privategw_static_routes test fail in XenServer, which was due to private gateway not getting created

[GitHub] cloudstack pull request #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-24 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1727#discussion_r84767316 --- Diff: setup/db/db/schema-490to4910.sql --- @@ -50,3 +50,8 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, hypervis

[GitHub] cloudstack pull request #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-24 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1727#discussion_r84766346 --- Diff: setup/db/db/schema-490to4910.sql --- @@ -50,3 +50,8 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, hypervis

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1724 Trillian test result (tid-170) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 31761 seconds Marvin logs:

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

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

[GitHub] cloudstack pull request #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-24 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1727#discussion_r84757955 --- Diff: setup/db/db/schema-490to4910.sql --- @@ -50,3 +50,8 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, hypervis

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

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

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-10-24 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1677 @jburwell @rhtyd we have rebased 4.9 branch, will start running tests --- 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

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1726 @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

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

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

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1726 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-91 --- 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 pull request #1727: CLOUDSTACK-9539: Support changing Service off...

2016-10-24 Thread nvazquez
GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1727 CLOUDSTACK-9539: Support changing Service offering for instance with VM Snapshots ## Actual behaviour CloudStack doesn't support changing service offering for vm instances which have vm

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

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

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1642 @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

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

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

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1726 @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 #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1726 @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

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

2016-10-24 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1697#discussion_r84712741 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java --- @@ -374,8 +375,24 @@ public SnapshotInfo

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

2016-10-24 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1726 LGTM :+1: --- 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

[GitHub] cloudstack issue #1451: CLOUDSTACK-9319: Use timeout when applying config to...

2016-10-24 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1451 ### ACS CI BVT Run **Sumarry:** Build Number 120 Hypervisor xenserver NetworkType Advanced Passed=100 Failed=2 Skipped=5 _Link to logs Folder

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-24 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1642 Any way to easily just re-run test_05_stop_ssvm and test_07_reboot_ssvm? I'm not sure how this PR would cause such failures. The only changed Java code does the following:

[GitHub] cloudstack pull request #1726: CLOUDSTACK-9560 Root volume of deleted VM lef...

2016-10-24 Thread yvsubhash
GitHub user yvsubhash opened a pull request: https://github.com/apache/cloudstack/pull/1726 CLOUDSTACK-9560 Root volume of deleted VM left unremoved You can merge this pull request into a Git repository by running: $ git pull https://github.com/yvsubhash/cloudstack

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-24 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1615 ### ACS CI BVT Run **Sumarry:** Build Number 117 Hypervisor xenserver NetworkType Advanced Passed=18 Failed=3 Skipped=2 _Link to logs Folder

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1724 @rhtyd 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 for

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

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

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1724 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-90 --- 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 pull request #1725: CLOUDSTACK-9559 Why allow deleting zone witho...

2016-10-24 Thread yvsubhash
GitHub user yvsubhash opened a pull request: https://github.com/apache/cloudstack/pull/1725 CLOUDSTACK-9559 Why allow deleting zone without deleting the seconda… CLOUDSTACK-9559 allow deleting zone without deleting the secondary storage under the zone You can merge this pull

[GitHub] cloudstack issue #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1724 @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 #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to handle...

2016-10-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1724 @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

[GitHub] cloudstack pull request #1724: CLOUDSTACK-9511: fix test_privategw_acl.py to...

2016-10-24 Thread murali-reddy
GitHub user murali-reddy opened a pull request: https://github.com/apache/cloudstack/pull/1724 CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical network fix to ensure only physical network with guest traffic is picked up for creating a private network

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1615 @karuturi a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) 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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-24 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1615 @blueorangutan test centos7 vmware-60u2 --- 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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-10-24 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1615 @blueorangutan help --- 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 #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-10-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 @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

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

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

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi @mike-tutkowski the oobm related error was an env issue, the vpc/vr related failures are know (and fixes are on its way). I'm not sure what to make of the ssvm/storage related failures.

[GitHub] cloudstack pull request #1709: CLOUDSTACK-7982 - KVM live migration

2016-10-24 Thread marcaurele
Github user marcaurele commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1709#discussion_r84632272 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -943,6 +962,11 @@ public boolean

[GitHub] cloudstack issue #1624: Fixes regarding VOLUME_DELETE events resulting from ...

2016-10-24 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1624 It is now fwd merged to all the branches. Thanks --- 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

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-10-24 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1642 @blueorangutan @rhtyd How are we on this PR? are the failures related? Or can I go ahead and merge this? --- If your project is set up for it, you can reply to this email and have your reply