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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1762 Due to the previous discussion, I am -1 on merging this PR. --- 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

[GitHub] cloudstack pull request #1765: Cloudstack 9586: When using local storage wit...

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

4.8.2.0/4.9.1.0/4.10.0.0 RC Status

All, I apologize for being relatively radio silent. We have mage good progress towards getting RCs out for 4.8.2.0, 4.9.1.0, and 4.10.0.0. On 31 October 2016, we 17 outstanding PRs to be merged. As of today (17 Nov 2016), we have 9 PRs to merge with pending one potential blocker/critical def

[GitHub] cloudstack pull request #1765: Cloudstack 9586: When using local storage wit...

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

[GitHub] cloudstack pull request #1765: Cloudstack 9586: When using local storage wit...

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

[GitHub] cloudstack issue #1745: CLOUDSTACK-9503: Increased the VR script timeout. Mo...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1745 @rhtyd where do we stand determining the root cause of the test 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

[GitHub] cloudstack pull request #1674: CLOUDSTACK-9460: For long running transaction...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1674 --- 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 pull request #1673: CLOUDSTACK-9071: Properly parse stats.output....

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1673 --- 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 #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1673 @wido we are seeing these errors on other PRs. Based on the fact that they are occurring on other PRs and change does not impact these tests, I am LGTM for tests. --- If your project is set

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1684 @pdion891 I don't care for the idea of potentially ignoring a custom security configuration as it may unexpectedly reduce a user's security. By default, we should take the most conservative p

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1681 @murali-reddy have you had a chance to investigate the [test failures](https://github.com/apache/cloudstack/pull/1681#issuecomment-257717551)? --- If your project is set up for it, you can rep

[GitHub] cloudstack issue #1757: CLOUDSTACK-9583: VR: In CsDhcp.py preseed both hosta...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1757 @rhtyd @swill @abhinandanprateek @karuturi could you review the test results/perform further testing for a second LGTM? I would like to get this PR into 4.8.2.0/4.9.1.0/4.10.0.0 as it greatly

[GitHub] cloudstack pull request #1676: CLOUDSTACK-9502: DS template copies don’t g...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1676 --- 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 #1580: CLOUDSTACK-9402 : Support for underlay features (Sou...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1580 @prashanthvarma #1578 has been merged. Can you rebase this PR and squash the commits? Once that is done and Jenkins and Travis are green, I will kick regression tests via blueorganutan. ---

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1579 @prashanthvarma #1578 has been merged. Can you rebase this PR and squash the commits? Once that is done and Jenkins and Travis are green, I will kick regression tests via blueorganutan. ---

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1577 @prashanthvarma @nlivens can you please investigate the Jenkins failures? Once Jenkins and Travis go green, I will kick blueorangutan to regression test this PR. --- If your project is set u

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378691 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -1209,25 +1209,95 @@ public String toString() {

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378673 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -1209,25 +1209,95 @@ public String toString() {

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378451 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -776,6 +779,13 @@ public boolean configure

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378255 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -252,6 +253,8 @@ protected String

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378602 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -1209,25 +1209,95 @@ public String toString() {

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88377937 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtDomainXMLParser.java --- @@ -175,6 +180,26 @@ public boolean parseDomain

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378575 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -1209,25 +1209,95 @@ public String toString() {

[GitHub] cloudstack pull request #1545: CLOUDSTACK-8715: Add channel to Instances for...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r88378111 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -252,6 +253,8 @@ protected String

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

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1542 @jburwell 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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1542 @blueorangutan test centos7 vmware-55u3 test_nested_virtualization.py --- 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 pro

[GitHub] cloudstack issue #1676: CLOUDSTACK-9502: DS template copies don’t get dele...

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1676 @jburwell @rhtyd @karuturi This PR also seems to be ready for merging . --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

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

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1542 @karuturi @jburwell As per @rhtyd we can run additional tests with blueorangutan . Can we execute them and merge this PR since it passes the standard test suite just fine? --- If your proje

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed I think this change will merit a separate.PR for the forward merge to master. So, once we get merged to 4.9, you'll open PR against master for the forward merge. In terms of the

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 That sounds good John. Ill reopen this against 4.9 On Wed, Nov 16, 2016 at 18:36 John Burwell wrote: > @syed also, could you please change the base > branc

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed also, could you please change the base branch to 4.9? I would like to include this support in LTS as many XenServer users will be stranded on old hardware without it. --- If your proje

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed I would like to hold this item for the next release cycle. We have a very few PRs left to get 4.8.2.0, 4.9.1.0, and 4.10.0.0 out the door. It will also give us time to run all of the co

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @jburwell I think this is looking good (unless you have any other comments) What can I do to help get this in? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack pull request #1711: XenServer 7 Support

GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1711 XenServer 7 Support This PR adds support for XenServer 7. I have manually done the following tests - Create a new cluster with XenServer7 - Add Primary storage: Should create an SR on XS

[GitHub] cloudstack pull request #1711: XenServer 7 Support

Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1711 --- 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 e

[GitHub] cloudstack issue #1706: CLOUDSTACK-9500: remove the IP from the databag if i...

Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1706 @pdion891 Sorry for being late to the party. We've seen this issue too and resolved it in a different way. Removing the ip from the databag probably won't work too well, as it is used to dep

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

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1765 @abhinandanprateek could you please squash the commits for this PR? --- 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 proje

RE: Apachecon this week

All - I will be a little late. Will seen you around 6:30pm Raj On Tue, Nov 15, 2016 at 9:17 PM +0100, "Rajesh Ramchandani" mailto:rajesh.ramchand...@accelerite.com>> wrote: Confirmed tomorrow at 6pm at the Melia Bar From: Steve Roles Sent: Tuesday, November 15, 5:28 PM Subject: RE: Apacheco

[GitHub] cloudstack issue #1759: CLOUDSTACK-9589 vmName entries from host_details tab...

Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1759 ### ACS CI BVT Run **Sumarry:** Build Number 137 Hypervisor xenserver NetworkType Advanced Passed=102 Failed=3 Skipped=6 _Link to logs Folder (searc

[GitHub] cloudstack pull request #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails ...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1677 --- 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 pull request #798: CLOUDSTACK-8830 - [Vmware] VM snapshot fails f...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/798 --- 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 #1761: CLOUDSTACK-9592 Empty responses from site to site co...

Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1761 ### ACS CI BVT Run **Sumarry:** Build Number 136 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=2 Skipped=6 _Link to logs Folder (searc

[GitHub] cloudstack issue #1578: CLOUDSTACK-9401 : Support for Internal DNS in Nuage ...

Github user krissterckx commented on the issue: https://github.com/apache/cloudstack/pull/1578 Thanks @karuturi Kris --- 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 ena

[GitHub] cloudstack pull request #1765: Cloudstack 9586: When using local storage wit...

GitHub user abhinandanprateek opened a pull request: https://github.com/apache/cloudstack/pull/1765 Cloudstack 9586: When using local storage with Xenserver prepareTemplate does not work with multiple primary store The race condition will happen whenever there are multiple primary s

[GitHub] cloudstack issue #1700: CLOUDSTACK-9359: IPv6 for Basic Networking

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1700 @karuturi 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 #1700: CLOUDSTACK-9359: IPv6 for Basic Networking

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1700 @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