RE: introduction

2016-03-24 Thread Kishan Kavala
Welcome back Murali! -Original Message- From: Abhinav Roy [mailto:abhinav@accelerite.com] Sent: 24 March 2016 10:53 AM To: dev@cloudstack.apache.org Subject: RE: introduction Welcome back Murali :) Regards, Abhinav -Original Message- From: Murali Reddy [mailto:murali.re...

[DISCUSS] Request for comments : VPC Inline LoadBalancer (new plugin)

2016-03-24 Thread Nick LIVENS
Hi all, I'd like to propose a new plugin called the "VPC Inline LB" plugin. The design document can be found at : https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=61340894 Looking forward to hear your reviews / thoughts. Thanks! Kind regards, Nick Livens

Migrating CloudStack content from download.cloud.com

2016-03-24 Thread Raja Pullela
Hi, Citrix has been hosting "download.cloud.com" for quite some time now and it holds the System Templates for all the releases and some tools. Going forward, this content needs to be moved from "download.cloud.com". So, we will be moving this content to "cloudstack.a

Re: Migrating CloudStack content from download.cloud.com

2016-03-24 Thread Wido den Hollander
> Op 24 maart 2016 om 13:33 schreef Raja Pullela : > > > Hi, > > Citrix has been hosting "download.cloud.com" for quite some time now > and it holds the System Templates for all the releases and some tools. > Going forward, this content needs to be moved from > "download.cl

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1450#discussion_r57313534 --- Diff: server/src/com/cloud/network/IpAddressManagerImpl.java --- @@ -1698,6 +1698,22 @@ public String acquireGuestIpAddress(Network network,

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1450#issuecomment-200825346 It is always great to see parts of a bigger method being extracted to smaller ones, and then test cases and java docs being used. I believe the te

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1450#discussion_r57313299 --- Diff: server/test/com/cloud/network/IpAddressManagerTest.java --- @@ -0,0 +1,71 @@ +// Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-03-24 Thread ProjectMoon
Github user ProjectMoon commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1450#discussion_r57317877 --- Diff: server/test/com/cloud/network/IpAddressManagerTest.java --- @@ -0,0 +1,71 @@ +// Licensed to the Apache Software Foundation (ASF) under

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1450#discussion_r57319355 --- Diff: server/test/com/cloud/network/IpAddressManagerTest.java --- @@ -0,0 +1,71 @@ +// Licensed to the Apache Software Foundation (ASF)

RE: introduction

2016-03-24 Thread Somesh Naidu
Good to have you back Murali :) Regards, Somesh -Original Message- From: Kishan Kavala [mailto:kishan.kav...@accelerite.com] Sent: Thursday, March 24, 2016 3:03 AM To: dev@cloudstack.apache.org Subject: RE: introduction Welcome back Murali! -Original Message- From: Abhinav Roy

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200882489 this was merged without functional tests (integration tests)? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200883526 output was posted for test run against master and 4.7. maybe I jumped the gun? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1430 --- 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: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200884247 i will make sure that integration test are run against everything going forward. this judgment call may have been premature. --- If your project is set up for it, y

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200888647 The test output you mean was the one posted by @kiwiflyer ? I had not reviewed this PR (sorry for that), but there are two (2) things here that called

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200901622 I want to point out that this is taking care of the case where you have a load balancer between the agents and the management server (see original issue notes). I

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200897012 I see your point and I agree with you. What do you think is a reasonable amount of time to check before we throw the error? --- If your project is set up for it, yo

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200903313 so you are basically saying that if the connection can not be established, it is not functional anyway, so there is no point in this timing out? --- If your project

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200905706 That is it; Well, giving that the “_shell.getBackoffAlgorithm().waitBeforeRetry()” uses a parameter that indicates the amount of time between retr

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200906752 @rafaelweingartner This is used in both the system VM agent and the host (hypervisor) agent. --- If your project is set up for it, you can reply to this em

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200908383 @swill This exception is thrown when the NIO operations used to establish the connection to the management server on port 8250 fail.When this exception gets thro

RE: introduction

2016-03-24 Thread Sateesh Chodapuneedi
Welcome back Murali :) Looking forward to work with you. Regards, Sateesh Chief Product Engineer, CloudPlatform Development, Accelerite. Off: +91 80 6772 1329 | EMail: sateesh.chodapune...@accelerite.com > -Original Message- > From: Somesh Naidu [mailto:somesh.na...@citrix.com] > Sent:

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200914880 Ok, So, this agent class is executed as a service in the operating system (OS) the OS will call a service that calls the main method that instantiates

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57346137 --- Diff: api/src/com/cloud/network/Network.java --- @@ -243,7 +245,7 @@ public static Capability getCapability(String capabilityName) {

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57346529 --- Diff: api/src/com/cloud/network/vpc/QuaggaZoneConfig.java --- @@ -0,0 +1,331 @@ +// Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57346679 --- Diff: api/src/com/cloud/network/vpc/QuaggaZoneConfig.java --- @@ -0,0 +1,331 @@ +// Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200918379 @rafaelweingartner Yes, you are correct. I like your suggestion. I'll log the exception with the failure. In terms of the persistent connections, I believ

Re: [DISCUSS] Request for comments : VPC Inline LoadBalancer (new plugin)

2016-03-24 Thread ilya
Hi Nick, Being fan of SDN, I gave this proposal a thorough read. I do have only 1 comment - that you can perhaps can use to reconsider: "Each appliance will have 2 nics, one for management, and one in the guest network. " In general, 2 nics - one going to management and one going to guest - is

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200919876 @kiwiflyer, it is nice that we achieved a consensus. What do you think @swill? How do we proceed now? This PR has already been merged and forw

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200926830 @rafaelweingartner I agree. I think it makes the most sense to open a new PR to add the exception log. @kiwiflyer would you mind doing that for us? Would you mind

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200931266 You are welcome. I try to go over as many PRs as I can, but I always end up overlooking some. You can always ping me on slack or mail to check a PR bef

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200936609 Thank you sir. I really appreciate it. Once I get my CI online I will be able to start getting through this backlog in a much more efficient manner. --- If your pr

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57354506 --- Diff: api/src/com/cloud/network/vpc/QuaggaZoneConfig.java --- @@ -0,0 +1,331 @@ +// Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57354705 --- Diff: api/src/com/cloud/network/vpc/QuaggaZoneConfig.java --- @@ -0,0 +1,331 @@ +// Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57354938 --- Diff: api/src/com/cloud/network/vpc/QuaggaZoneConfig.java --- @@ -0,0 +1,331 @@ +// Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57360768 --- Diff: api/src/com/cloud/network/vpc/VpcProvisioningService.java --- @@ -26,13 +25,11 @@ public VpcOffering getVpcOffering(long vpcOffe

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57360755 --- Diff: api/src/com/cloud/network/vpc/VpcProvisioningService.java --- @@ -26,13 +25,11 @@ public VpcOffering getVpcOffering(long vpcOffe

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57360954 --- Diff: api/src/com/cloud/network/vpc/VpcProvisioningService.java --- @@ -49,4 +46,9 @@ public VpcOffering createVpcOffering(String name, String displ

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57360933 --- Diff: api/src/com/cloud/network/vpc/VpcProvisioningService.java --- @@ -49,4 +46,9 @@ public VpcOffering createVpcOffering(String name, String displ

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57362193 --- Diff: api/src/org/apache/cloudstack/api/command/admin/vpc/VPCQuaggaConfigCmd.java --- @@ -0,0 +1,73 @@ +// Licensed to the Apache Software Found

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57362292 --- Diff: api/src/org/apache/cloudstack/api/command/admin/vpc/VPCQuaggaConfigCmd.java --- @@ -0,0 +1,73 @@ +// Licensed to the Apache Software Found

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57362722 --- Diff: api/src/org/apache/cloudstack/api/command/admin/vpc/VPCQuaggaConfigCmd.java --- @@ -0,0 +1,73 @@ +// Licensed to the Apache Software Found

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57362918 --- Diff: api/src/org/apache/cloudstack/api/command/admin/vpc/VPCQuaggaConfigUpdateCmd.java --- @@ -0,0 +1,204 @@ +// Licensed to the Apache Softwa

[GitHub] cloudstack pull request: CLOUDSTACK-9285 for 4.7.x

2016-03-24 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1430#issuecomment-200956335 @swill Yes, I'll work on this either today or tomorrow and get a new PR submitted. Thanks guys. --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: CLOUDSTACK-8302: Removing snapshots on RB...

2016-03-24 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/1230#issuecomment-200958562 Yes, @kiwiflyer, the naming XenserverSnapshotStrategy appears to be old and no longer accurate. I believe it should be renamed to something like DefaultSnaps

Re: [VOTE] Move 'apache/cloudstack' -> 'apache-cloudstack/cloudstack'

2016-03-24 Thread Will Stevens
Thank you for all the support everyone. This vote is now closed. We have unanimous support for this proposal with the following votes: Totals -- +1: 20 (including myself) +0: 0 -1: 0 I have opened a support ticket with github to get a clearer understanding of what the process will be to swi

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57368304 --- Diff: api/src/org/apache/cloudstack/api/command/admin/vpc/VPCQuaggaConfigUpdateCmd.java --- @@ -0,0 +1,204 @@ +// Licensed to the Apache Softwa

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57368469 --- Diff: api/src/org/apache/cloudstack/api/command/admin/vpc/VPCQuaggaConfigUpdateCmd.java --- @@ -0,0 +1,204 @@ +// Licensed to the Apache Softwa

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-24 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57369359 --- Diff: api/src/org/apache/cloudstack/api/command/user/network/CreateNetworkCmd.java --- @@ -211,7 +201,7 @@ public Boolean getDisplayNetwork() {

[GitHub] cloudstack pull request: CLOUDSTACK-9228: Network update with mist...

2016-03-24 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1333#issuecomment-201016129 _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0 ### ACS CI BVT Run **Sumarry:**

Re: introduction

2016-03-24 Thread Syed Mushtaq
Welcome back Murali. On Thu, Mar 24, 2016 at 12:28 PM, Sateesh Chodapuneedi < sateesh.chodapune...@accelerite.com> wrote: > Welcome back Murali :) > Looking forward to work with you. > > Regards, > Sateesh > Chief Product Engineer, CloudPlatform Development, Accelerite. > Off: +91 80 6772 1329 |

[DISCUSSION] cloudstackcollab.org

2016-03-24 Thread Pierre-Luc Dion
Hi, Since we are going ahead with a CCCNA16 in Montreal, we would like to revive the website http://cloudstackcollab.org/. The url is curently pointing to linuxfoundation events site but our event and the one in SaoPaolo on September will not use linuxfoundation services :-S So we were thinking

[GitHub] cloudstack pull request: Add ability to download templates in Swif...

2016-03-24 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1332#issuecomment-201084616 _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0 ### ACS CI BVT Run **Sumarry:**

Re: External fork of Cloudstack (was Re: [GitHub] cloudstack pull request: Is the project attempting a fork on Githu...)

2016-03-24 Thread Pierre-Luc Dion
Hi, Sorry to step in late on that thread. I have a question for Jim, David, Sam. As it seams, a lot of us in the "CloudStack community" would love to work with github features to speedup the project development cycle, The fact that some other ASF projects are interested too. Would it be thinkabl

[GitHub] cloudstack pull request: Fix Sync of template.properties in Swift

2016-03-24 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-201139141 _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0 ### ACS CI BVT Run **Sumarry:**

RE: [DISCUSS] Request for comments : VPC Inline LoadBalancer (new plugin)

2016-03-24 Thread Sanjeev Neelarapu
Hi Nick Livens, I have gone through the FS and following are my review comments: 1. Will this LB appliance be placed between guest vms and the Nuage VSP provider(Nuage VSP and lb appliance will have one nic in guest network)? 2. Is there any specific reason for traffic filtering on lb appliance

[GitHub] cloudstack pull request: CLOUDSTACK-9317: Enable/disable static NA...

2016-03-24 Thread jayapalu
Github user jayapalu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1450#discussion_r57422486 --- Diff: server/src/com/cloud/network/router/CommandSetupHelper.java --- @@ -778,10 +778,11 @@ public int compare(final PublicIpAddress o1, final Publi

Re: Migrating CloudStack content from download.cloud.com

2016-03-24 Thread ilya
Raja Would you know if cloud.com domain will be transferred to accelerite? Regards ilya On 3/24/16 5:38 AM, Wido den Hollander wrote: > >> Op 24 maart 2016 om 13:33 schreef Raja Pullela : >> >> >> Hi, >> >> Citrix has been hosting "download.cloud.com" for quite some time now >> and it h