[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

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

[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

[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

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 |

[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

[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: 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

[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

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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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_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_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: 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

[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

[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-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

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 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

[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: 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_r57346137 --- Diff: api/src/com/cloud/network/Network.java --- @@ -243,7 +245,7 @@ public static Capability getCapability(String capabilityName) {

[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

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] >

[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

[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

[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

[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 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,

[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).

[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 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,

[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

[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

[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

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-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)

[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_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 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

[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

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

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

[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

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