[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 thanks, I'll bisect them today. --- 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 #1748: KVM enhancements for managed storage

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1748 @mike-tutkowski can you use the JIRA id in the commit summary as: `CLOUDSTACK-: one-line summary ... rest of the commit message... ` --- If your project is set up for it, you

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1749 Thanks @mike-tutkowski can you add a JIRA id for this, and use that id in the commit summary? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd @jburwell I recently ran all of the managed-storage-related integration tests successfully (referenced above). I think we're in a good position to get this into 4.10 now.

[GitHub] cloudstack issue #1748: KVM enhancements for managed storage

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1748 @rhtyd I added that JIRA ticket and referenced it in the PR description. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 @rhtyd I added that JIRA ticket and referenced it in the PR description. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1753 @rhtyd Only 2 PRs were merged in the interval between last successful run and failure: PR1766 and PR1545 1766 was merged to 4.8 and was hypervisor agnostic so all points at PR1545 'Add

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 TestAddRemoveHosts (1, 2, and 4 were successful and 3 failed for an environmental reason): [results.txt](https://github.com/apache/cloudstack/files/613869/results.txt)

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-25 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1749 TestVolumes (resign = true): [results.txt](https://github.com/apache/cloudstack/files/613863/results.txt) TestVolumes (resign = false):

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 @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 #1754: [4.10] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 @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 #1752: [4.8] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1752 @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 #1754: [4.10] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @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 #1752: [4.8] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1752 @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 #1753: [4.9] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 thanks, that's my opinion as well, I'll kick one more round just to be sure, and will bisect recently merged changes and revert any changes if necessary. @blueorangutan test matrix

[GitHub] cloudstack issue #1752: [4.8] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1752 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-256 --- 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 #1753: [4.9] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-257 --- 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 #1754: [4.10] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-255 --- 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 #1753: [4.9] Latest health test run

2016-11-25 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1753 @rhtyd Both 4.9 and 4.10 now fail as well as PR based on 4.9 e.g. PR 1659 with 13 tests on KVM but PRs based on 4.8 succeed e.g. PR1710 and PR1753. Might be some interference from one of the

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 @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 #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-25 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 @rafaelweingartner I might be wrong but 2d came from findAndUpdateDirectAgentToLoad in HostDaoImpl which also creates a large transaction. --- If your project is set up for it, you can reply

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @serg38 Likely environment related, I'll check that /cc @PaulAngus Meanwhile I'll kick another round. @blueorangutan package --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 @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 #1754: [4.10] Latest health test run

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @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 issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

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

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

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

[GitHub] cloudstack issue #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1765 Trillian test result (tid-439) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32353 seconds Marvin logs:

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-25 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 @rafaelweingartner You might be right that pod_vlan_map should be in the join. May be I didn't find the correct methods after all. @jburwell @rhtyd What do you think? I was able to find

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

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

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-25 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1753 @rhtyd All the new failures are due to inability to SSH to VMs. Can we re-run this to filter environmental issue or recent PR merge? --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

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

[GitHub] cloudstack issue #1780: Fix the KVM hypervisor's handling of not-really-SCSI...

2016-11-25 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1780 @ppenchev-storpool I haven't worked with a non-iscsi adaptor for KVM but wouldn't it change the default behaviour now for non-iscsi disks? What if the other adaptors also ignore this? Can we add a

[GitHub] cloudstack issue #1781: CLOUDSTACK-9612: Fixed issue in restarting redundant...

2016-11-25 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1781 ### ACS CI BVT Run **Sumarry:** Build Number 141 Hypervisor xenserver NetworkType Advanced Passed=78 Failed=17 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1733: CLOUDSTACK-9563 ExtractTemplate returns malformed UR...

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

[GitHub] cloudstack issue #1775: CLOUDSTACK-9606: While IP address is released, tag a...

2016-11-25 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1775 can you please provide a description and steps to reproduce --- 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 #1777: clearCLOUDSTACK-9133: Two volume.delete usage events...

2016-11-25 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1777 Where is the other usage event generated? Does this only apply to ROOT volumes? --- 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 #1779: CLOUDSTACK-9610: Disabled Host Keeps Being up status...

2016-11-25 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1779 Hi @priyankparihar, Thanks for the patch. Can you provide a more descriptive message about the bug and how this fix addresses that? --- If your project is set up for it, you can

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2016-11-25 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1727 @rhtyd @koushik-das @serg38 thanks, I'll work on second option --- 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 #1785: CLOUDSTACK-9416 : Enabling Static NAT on an associat...

2016-11-25 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1785 LGTM this PR solves the same issue as #1778 --- 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 #1786: CLOUDSTACK-9618: Load Balancer configuration ...

2016-11-25 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1786 CLOUDSTACK-9618: Load Balancer configuration page does not have "Source" method in the drop down list. If we create an isolated network with NetScaler published service offering for

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-25 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1762 @serg38 if that "AssignIpAddressFromPodVlanSearch" object was being used to generate the SQL; should not we see a join with "pod_vlan_map" too? For me this, this SC is very confusing.

[GitHub] cloudstack issue #1785: CLOUDSTACK-9416 : Enabling Static NAT on an associat...

2016-11-25 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1785 @rhtyd This is a shadow of PR #1592 , rebased against 4.9 as requested. Note: The original PR #1592 has enough LGTMs to merge this PR. Once you merge this PR, I will

[GitHub] cloudstack issue #1592: CLOUDSTACK-9416 (Bug in ACS master GUI) : Enabling S...

2016-11-25 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1592 @rhtyd I had difficulties changing the base branch to 4.9 and re-basing against 4.9 as the source branch on my fork is master (prashanthvarma:master). Anyhow, I have opened a new

[GitHub] cloudstack pull request #1785: CLOUDSTACK-9416 : Enabling Static NAT on an a...

2016-11-25 Thread prashanthvarma
GitHub user prashanthvarma opened a pull request: https://github.com/apache/cloudstack/pull/1785 CLOUDSTACK-9416 : Enabling Static NAT on an associated Public IP to one of the NICs (networks) of a multi-NIC VM fails due to a wrong (default) Guest VM IP being selected in the GUI

[GitHub] cloudstack pull request #1784: CS-505: Marvin test to VR DNS Service

2016-11-25 Thread abhinandanprateek
GitHub user abhinandanprateek opened a pull request: https://github.com/apache/cloudstack/pull/1784 CS-505: Marvin test to VR DNS Service You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack CS-505 Alternatively

[GitHub] cloudstack issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multipl...

2016-11-25 Thread marcaurele
Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1760 right, I forgot about the match required with the version in the pom. I thought I should prepare the stuff for the next 4.9.x release but that cannot work unless the pom files are updated

[GitHub] cloudstack issue #1769: tools: fix Dockerfile

2016-11-25 Thread resmo
Github user resmo commented on the issue: https://github.com/apache/cloudstack/pull/1769 I didn't test #1435 yet, but code looks good. The only difference I see is: ~~~ pip install cryptography --force-reinstall ~~~ And I used the mysql connector from MySQL.com but

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-11-25 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu line 179 should be changed as well, same to line 143 --- 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 #1782: CLOUDSTACK-9617: Fixed enabling remote access after ...

2016-11-25 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1782 @jayapalu what about load balancer ports ? shall we consider it as well? --- 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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules ...

2016-11-25 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/1783 CLOUDSTACK-9615: Fixd applying ingress rules without ports When ingress rule is applied without ports (port start and port end params are not passed) then API/UI is showing rule got applied

[GitHub] cloudstack pull request #1782: CLOUDSTACK-9617: Fixed enabling remote access...

2016-11-25 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/1782 CLOUDSTACK-9617: Fixed enabling remote access after PF configured on … Enabling Remote access Vpn Fails when there is a portforwarding rule of the reserved ports ( 1701 , 500 , 4500) under

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

2016-11-25 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 #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1765 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) 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 #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1765 @blueorangutan test centos7 xenserver-65sp1 --- 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 #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-25 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 #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 @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 #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1710 @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-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-254 --- 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 #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1765 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-253 --- 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 #1780: Fix the KVM hypervisor's handling of not-really-SCSI...

2016-11-25 Thread ppenchev-storpool
Github user ppenchev-storpool commented on the issue: https://github.com/apache/cloudstack/pull/1780 Hi, Thanks for looking at this! Testing this could simply be a matter of adding more than one storage adaptor to a KVM storage pool, then connecting and disconnecting

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-25 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1579 @rhtyd @jburwell We will rebase this PR with the latest master asap, and update you. As mentioned in the previous comments, we wanted to merge this PR after merging PR #1580 as

[GitHub] cloudstack issue #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-25 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1577 @rhtyd @jburwell Thank you for reviewing and helping us merge this PR, much appreciated !! --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

2016-11-25 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1580 @rhtyd @jburwell Thank you for reviewing and helping us merge this PR, much appreciated !! --- If your project is set up for it, you can reply to this email and have your reply appear

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

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 @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 #1765: Cloudstack 9586: When using local storage with Xense...

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

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 @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 issue #1765: Cloudstack 9586: When using local storage with Xense...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1765 @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 #1765: Cloudstack 9586: When using local storage wit...

2016-11-25 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1765#discussion_r89576730 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/Xenserver625StorageProcessor.java --- @@ -100,36 +100,25 @@ protected

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1710 There were some intermittent errors seen, I'll re-kick tests. @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1710: CLOUDSTACK-9538: FIX failure in Deleting Snapshot Fr...

2016-11-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1710 @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 pull request #1710: CLOUDSTACK-9538: FIX failure in Deleting Snap...

2016-11-25 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1710#discussion_r89576049 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java --- @@ -268,7 +268,9 @@ public boolean

[GitHub] cloudstack issue #1664: CLOUDSTACK-8676 Deploy user instance from vm snapsho...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1664 Thanks for sharing @sateesh-chodapuneedi --- 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 #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1579 @nlivens @prashanthvarma @singalrahul please squash your changes, fix conflicts, rebase against latest master. Pending lgtm/review is requested. /cc @jburwell --- If your project is set up for

[GitHub] cloudstack pull request #1577: CLOUDSTACK-9321 : Multiple Internal LB rules ...

2016-11-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1577 --- 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 issue #1577: CLOUDSTACK-9321 : Multiple Internal LB rules (more t...

2016-11-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1577 LGTM. Few failures are related to env, merging this 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 #1710: CLOUDSTACK-9538: FIX failure in Deleting Snap...

2016-11-25 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1710#discussion_r89575783 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java --- @@ -268,7 +268,9 @@ public boolean

[GitHub] cloudstack pull request #1580: CLOUDSTACK-9402 : Support for underlay featur...

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