[GitHub] cloudstack pull request: CLOUDSTACK-8894: Restrict vGPU enabled VM...

2015-09-21 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/868 CLOUDSTACK-8894: Restrict vGPU enabled VMs dynamic scaling if new service offering has different vGPU type You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: CLOUDSTACK-8891: Fixed default iptables r...

2015-09-21 Thread jayapalu
GitHub user jayapalu opened a pull request: https://github.com/apache/cloudstack/pull/867 CLOUDSTACK-8891: Fixed default iptables rules on VR for guest traffic VR default iptables rules in INPUT chain are configured partially. In CsAddress.py rules are configured while configuri

[GitHub] cloudstack pull request: CLOUDSTACK-8843: Fixed issue in default i...

2015-09-21 Thread kishankavala
Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/842#issuecomment-142185868 LGTM integration.smoke.test_vm_life_cycle test case validates ssh access to Vm as part of test_10_attachAndDetach_iso test Would be better test_deploy_vm

[GitHub] cloudstack pull request: CLOUDSTACK-8751 minimise downtime of netw...

2015-09-21 Thread bvbharatk
GitHub user bvbharatk opened a pull request: https://github.com/apache/cloudstack/pull/866 CLOUDSTACK-8751 minimise downtime of network when network update is in progress. more info in the bug link below. https://issues.apache.org/jira/browse/CLOUDSTACK-8751 You can merge this

[GitHub] cloudstack pull request: CLOUDSTACK-8856 Primary Storage Used(type...

2015-09-21 Thread bvbharatk
GitHub user bvbharatk opened a pull request: https://github.com/apache/cloudstack/pull/865 CLOUDSTACK-8856 Primary Storage Used(type tag with value 2) related t… Primary Storage Used(type tag with value 2) related tag is not showing in listCapacity api response You can merge thi

[GitHub] cloudstack pull request: CLOUDSTACK-8860: improve error messages i...

2015-09-21 Thread bvbharatk
GitHub user bvbharatk opened a pull request: https://github.com/apache/cloudstack/pull/864 CLOUDSTACK-8860: improve error messages in VM deployment code path. improved the error messages in vm deployment code path. added some more data to the error messages and also fixed some error

Re: Proposal to get to 100% passrate on BVTs

2015-09-21 Thread Bharat Kumar
Hi, I am already working towards this. We can already replicate the test infra using puppet. But i need some help in generating reports, sharing the test results and archiving them. particularly archiving them since we need some space which is accessible by everyone. I have also made a propos

[GitHub] cloudstack pull request: CLOUDSTACK-8843: Fixed issue in default i...

2015-09-21 Thread jayapalu
Github user jayapalu commented on the pull request: https://github.com/apache/cloudstack/pull/842#issuecomment-142178961 @wilderrodrigues Can you please review and give LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

Re: Proposal to get to 100% passrate on BVTs

2015-09-21 Thread Sanjeev N
I would be happy to lend my hand in generalizing the BVT Infra. -Sanjeev On Mon, Sep 21, 2015 at 4:34 PM, Raja Pullela wrote: > Rohit, not sure about the efforts required to do this. Can certainly work > with you or someone to do this. Please let me know, > > best, > Raja > From: Rohit Yadav

[GitHub] cloudstack pull request: CLOUDSTACK-8889: delete volume doesnt dec...

2015-09-21 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/860#issuecomment-142174642 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 e

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-21 Thread harikrishna-patnala
Github user harikrishna-patnala commented on the pull request: https://github.com/apache/cloudstack/pull/861#issuecomment-142174711 Thanks Koushik, here is the commit which removed the class XenServer602Resource. https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-21 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/861#issuecomment-142173025 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 e

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8880: calculate free m...

2015-09-21 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/847#issuecomment-142146181 @kishankavala Would it be possible to move your newly implemented try catch structure into it's own method? That way don't have a try catch within a try catch.

[GitHub] cloudstack pull request: [BLOCKER][4.6]CLOUDSTACK-8883: Resolved c...

2015-09-21 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/863 [BLOCKER][4.6]CLOUDSTACK-8883: Resolved connect/reconnect issue. Hi! @wilderrodrigues by implementing Callable you switched a couple of methods and fields. I switched them some more!

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2015-09-21 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-142081363 I understand. I'm working on a task from last week and once I complete it I can take a stab at some until tests for this PR. --- If your project is

[GitHub] cloudstack pull request: CLOUDSTACK-8605: KVM: Config Drive and ge...

2015-09-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/577#issuecomment-142064215 @kishankavala There are 5 merge commits now... please remove them. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: CLOUDSTACK-8851 Redundant VR getting star...

2015-09-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/839 --- 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: Bug-ID: CLOUDSTACK-8880: calculate free m...

2015-09-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/847#issuecomment-142061248 @bhaisaab did you test this code against a KVM hypervisor or does that still need to be done? --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-142060873 @sudhansu7 I think in the mean while this problem fixed itself: @wido contributed systemd profiles so it's not needed any more to fix the CentOS 7 sysv scripts.

Re: BVT report - 9/21

2015-09-21 Thread Remi Bergsma
Thanks for the report! It’d really help if people would prepend their PR with ‘[BLOCKER]’ (like Boris did) so we can identify them and make sure they get the right attention. @Wilder shall we share the load a bit more here and ask other to help with the issues currently assigned to you? Also,

[GitHub] cloudstack pull request: Removed the empty class 'com.cloud.deploy...

2015-09-21 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/856#issuecomment-142042882 Reopening PR that was closed by mistake --- 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: Removed the empty class 'com.cloud.deploy...

2015-09-21 Thread pedro-martins
GitHub user pedro-martins reopened a pull request: https://github.com/apache/cloudstack/pull/856 Removed the empty class 'com.cloud.deploy.PlannerBase' Removed the PlannerBase class because it is does not bring contribution to ACS' code. We changed com.cloud.deploy.FirstFit

[GitHub] cloudstack pull request: Removed the empty class 'com.cloud.deploy...

2015-09-21 Thread pedro-martins
Github user pedro-martins closed the pull request at: https://github.com/apache/cloudstack/pull/856 --- 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 feat

[GitHub] cloudstack pull request: CLOUDSTACK-7151: Support multiple physica...

2015-09-21 Thread sureshanaparti
Github user sureshanaparti commented on the pull request: https://github.com/apache/cloudstack/pull/862#issuecomment-142004394 Tested database upgrade and updated API cmds addcluster and updatecluster to support multiple physical networks at cluster level. --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-7151: Support multiple physica...

2015-09-21 Thread sureshanaparti
GitHub user sureshanaparti opened a pull request: https://github.com/apache/cloudstack/pull/862 CLOUDSTACK-7151: Support multiple physical networks at cluster level for guest traffic over VMware deployments. CLOUDSTACK-7151: Support multiple physical networks at cluster level for g

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-141986296 @jburwell, I've commented on some of your comments, and I'm working on implementing the other comments --- If your project is set up for it, you can reply to this e

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39967011 --- Diff: plugins/network-elements/nuage-vsp/src/net/nuage/vsp/acs/client/NuageVspElementClient.java --- @@ -24,12 +24,17 @@ public interface

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39967023 --- Diff: plugins/network-elements/nuage-vsp/src/net/nuage/vsp/acs/client/NuageVspManagerClient.java --- @@ -0,0 +1,41 @@ +// +// Licensed to the

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39967016 --- Diff: plugins/network-elements/nuage-vsp/src/net/nuage/vsp/acs/client/NuageVspGuruClient.java --- @@ -21,23 +21,21 @@ import java.util.Col

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39966948 --- Diff: plugins/network-elements/nuage-vsp/src/net/nuage/vsp/acs/client/NuageVspApiClient.java --- @@ -23,8 +23,8 @@ public void login()

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39966738 --- Diff: plugins/network-elements/nuage-vsp/src/net/nuage/vsp/acs/NuageVspPluginClientLoader.java --- @@ -19,28 +19,86 @@ package net.nuage.v

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39966248 --- Diff: plugins/network-elements/nuage-vsp/src/com/cloud/network/resource/NuageVspResource.java --- @@ -262,15 +260,19 @@ public String getName() {

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2015-09-21 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-141960991 Hi @mike-tutkowski Unfortunately, I don't think I will have the time to help you with unit tests for this PR. I will again need to direct my sp

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39964563 --- Diff: plugins/network-elements/nuage-vsp/src/com/cloud/network/manager/NuageVspManagerImpl.java --- @@ -285,60 +507,341 @@ public boolean deleteNuag

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39963698 --- Diff: plugins/network-elements/nuage-vsp/src/com/cloud/network/manager/NuageVspManagerImpl.java --- @@ -285,60 +507,341 @@ public boolean deleteNuag

[GitHub] cloudstack pull request: CLOUDSTACK-8826: XenServer - Use device i...

2015-09-21 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/792#discussion_r39962919 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -3407,6 +3390,18 @@ public void ha

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39962640 --- Diff: plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java --- @@ -358,19 +441,22 @@ private String getVirtual

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39962051 --- Diff: plugins/network-elements/nuage-vsp/src/com/cloud/network/guru/NuageVspGuestNetworkGuru.java --- @@ -201,51 +234,82 @@ public NicProfile allocat

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8880: calculate free m...

2015-09-21 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/847#issuecomment-141947703 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 enab

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39961184 --- Diff: plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java --- @@ -454,6 +563,80 @@ protected boolean applyACLRules(

[GitHub] cloudstack pull request: Cloudstack-8816 some of the events do not...

2015-09-21 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/849#discussion_r39960628 --- Diff: api/src/org/apache/cloudstack/context/CallContext.java --- @@ -90,7 +90,13 @@ public void putContextParameter(Object key, Object value) {

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-21 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-141945369 @bhaisaab For CentOS 7 we have we cloud-management.service and cloud-management.config files. I don't see any other .rc file for cloud-management service.

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-21 Thread harikrishna-patnala
GitHub user harikrishna-patnala opened a pull request: https://github.com/apache/cloudstack/pull/861 CLOUDSTACK-: Xenserver 6.0.2 host stuck in disconnected state aft… Xenserver 6.0.2 hosts are stuck in disconnected state after CS upgrade to 4.6. This is because xenserver602re

RE: Proposal to get to 100% passrate on BVTs

2015-09-21 Thread Raja Pullela
Rohit, not sure about the efforts required to do this. Can certainly work with you or someone to do this. Please let me know, best, Raja From: Rohit Yadav [mailto:rohit.ya...@shapeblue.com] Sent: Monday, September 21, 2015 2:53 PM To: dev@cloudstack.apache.org Subject: Re: Proposal to get to 10

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-09-21 Thread nlivens
Github user nlivens commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/801#discussion_r39958866 --- Diff: plugins/network-elements/nuage-vsp/pom.xml --- @@ -28,6 +28,13 @@ 4.6.0-SNAPSHOT ../../pom.xml + + +

[GitHub] cloudstack pull request: CLOUDSTACK-8889: delete volume doesnt dec...

2015-09-21 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/860 CLOUDSTACK-8889: delete volume doesnt decrement primary store resource count Primary Storage count for an account does not decrease when a Data Disk is deleted belonging to the account unles

Build failed in Jenkins: simulator-4.5-singlerun #273

2015-09-21 Thread jenkins
See -- [...truncated 270 lines...] Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM change Started by an SCM ch

Jenkins build is back to normal : build-systemvm-master #628

2015-09-21 Thread jenkins
See

Re: Proposal to get to 100% passrate on BVTs

2015-09-21 Thread Rohit Yadav
Great initiative Raja! Any effort in generalizing the BVT infra so the infra can be reproduced using Puppet/Ansible? Regards, Rohit Yadav Software Architect, ShapeBlue [cid:9DD97B41-04C5-45F0-92A7-951F3E962F7A] M. +91 88 262 30892 | rohit.ya...@shapeblue.com

Build failed in Jenkins: build-systemvm-master #627

2015-09-21 Thread jenkins
See Changes: [wido] CLOUDSTACK-8840: Systemd service for the Usage Server [wido] CLOUDSTACK-8840: Fix the source path of the service file [wido] CLOUDSTACK-8840: Do not include old systemd wrapper --

[GitHub] cloudstack pull request: Fixed: Error given when creating VPN user...

2015-09-21 Thread kansal
Github user kansal commented on the pull request: https://github.com/apache/cloudstack/pull/826#issuecomment-141897458 @jayapalu can you review 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 your project does not

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8882: calculate networ...

2015-09-21 Thread kishankavala
GitHub user kishankavala opened a pull request: https://github.com/apache/cloudstack/pull/859 Bug-ID: CLOUDSTACK-8882: calculate network offering usage per nic instead of per network offering Issues occurs when: A Vm is created with mutiple nics: If 2 networks use same netwo