[GitHub] cloudstack issue #1862: CLOUDSTACK-9704 Remove dependency on VmwareContext o...

2017-03-04 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1862 tag:mergeready --- 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 #1859: CLOUDSTACK-8672 : NCC Integration with CloudStack

2017-03-04 Thread nitin-maharana
Github user nitin-maharana commented on the issue: https://github.com/apache/cloudstack/pull/1859 ping @rajesh-battala @sowmyakrishn --- 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 #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1776 Trillian test result (tid-930) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 35187 seconds Marvin logs:

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 I think that's the issue at our environment, it's trying to ssh via hostname instead of IP. Can you change that to IP please @nathanejohnson ? `Trying SSH Connection:

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 Hi @serg38 , Here's the management log: ``` 2017-03-04 20:43:40,972 INFO [c.c.h.v.r.VmwareResource] (DirectAgent-249:ctx-585f426d 10.2.2.73, job-29/job-32, cmd:

[GitHub] cloudstack issue #1981: CLOUDSTACK-9806: Nuage domain template selection per...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1981 Thanks, the tests failures looks to be not related to these changes. @prashanthvarma --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1776 @borisstoyanov 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 #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1776 @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 #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1776 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-561 --- 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 #1862: CLOUDSTACK-9704 Remove dependency on VmwareContext o...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1862 yes, @sateesh-chodapuneedi those are addressed in another 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

[GitHub] cloudstack issue #1961: Fix for test_snapshots.py using nfs2 instead of nfs ...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1961 Yes @serg38, but currently testing it. So this batch doesn't have it. --- 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 #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1776 Thanks for this fix @priyankparihar , if it addresses only the input validation should be pretty easy to test. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1776 @borisstoyanov 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 #1776: CLOUDSTACK-9603: 'concurrent.snapshots.threshold.per...

2017-03-04 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1776 @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 #1987: CLOUDSTACK-9814 : Unable to edit a Sub domain, which...

2017-03-04 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1987 @niteshsarda thank you very much :) The less duplicated or useless code around the better. The method 'updateDomain' is huge and you just changed a single line there, so I

Re:Re: :[VOTE] Apache Cloudstack 4.10.0.0

2017-03-04 Thread Rajani Karuturi
If no one else objects before Monday and they have tests, I will merge. ~Rajani Sent from phone. On 4 Mar 2017 8:13 p.m., "Haijiao" <18602198...@163.com> wrote: I think these PRs have met the criteria for merging. Can we have them included in RC2 ? CLOUDSTACK-9569: propagate global

Re:Re: :[VOTE] Apache Cloudstack 4.10.0.0

2017-03-04 Thread Haijiao
I think these PRs have met the criteria for merging. Can we have them included in RC2 ? CLOUDSTACK-9569: propagate global configuration router.aggregation.command.each.timeout to KVM agent #1856 Advanced isolated network egress destination cidr support added#1866 CLOUDSTACK-9698 [VMware]

[GitHub] cloudstack issue #883: CLOUDSTACK-8906: /var/log/cloud/ doesn't get logrotat...

2017-03-04 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/883 ### ACS CI BVT Run **Sumarry:** Build Number 422 Hypervisor xenserver NetworkType Advanced Passed=105 Failed=0 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1980: CLOUDSTACK-9805: Display VR list in network details

2017-03-04 Thread remibergsma
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1980 @rhtyd Looks great! Shall we do the same for the VPC routers, so that the overviews look the same? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1987: CLOUDSTACK-9814 : Unable to edit a Sub domain, which...

2017-03-04 Thread niteshsarda
Github user niteshsarda commented on the issue: https://github.com/apache/cloudstack/pull/1987 @rafaelweingartner : As per your suggestion, I have changed the search query and also removed the unused method updateDomain from managementserviceimpl class. Please check. ---

[GitHub] cloudstack pull request #1987: CLOUDSTACK-9814 : Unable to edit a Sub domain...

2017-03-04 Thread niteshsarda
Github user niteshsarda commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1987#discussion_r104283067 --- Diff: server/src/org/apache/cloudstack/region/RegionManagerImpl.java --- @@ -229,7 +232,7 @@ public boolean deleteUser(DeleteUserCmd cmd) {