Re: Thoughts on Citrix announcement yesterday

2016-01-13 Thread ilya
I agree. I think Citrix ramped down the activity in the last year+ as well (i can only assume it was due to pending sale), hopefully Accelerite folks will bring new energy... On 1/12/16 11:05 PM, Wei ZHOU wrote: > Thanks Citrix for the large contributions ! > > As far as I know, some core members

Re: cloudstack devs at fosdem?

2016-01-13 Thread Sebastien Goasguen
yes,I will be there > On Jan 12, 2016, at 11:20 PM, Laszlo Hornyak wrote: > > Hi List, > > Anyone planing to visit fosdem this year? It would be great to meet there. > > Best regards, > Laszlo > > -- > > EOF

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-171215623 LGTM , I'll look at jenkins again. --- 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 proj

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-171217574 complaining about a license, false positive: server/test/com/cloud/event/ActionEventUtilsTest.java --- If your project is set up for it, you can reply to thi

Build failed in Jenkins: build-master-slowbuild #2974

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

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

2016-01-13 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1333 CLOUDSTACK-9228: Network update with mistmatch in services require forced option Steps to reproduce: === 1.Bring up CloudStack in advanced zone 2.Create isolated

[GitHub] cloudstack pull request: CLOUDSTACK-9229: Autoscale policy creatio...

2016-01-13 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1334 CLOUDSTACK-9229: Autoscale policy creation failing in VPC due to zone id missing in createAutoScaleVmProfile Autoscale policy creation is failing on VPC while creating autoscalevmprofil

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

2016-01-13 Thread phillipkent
Github user phillipkent commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-171242939 @bhaisaab I tested a clean virtualenv install of Cloudmonkey 5.3.2 (from PyPy) and also a clone of the latest Cloudmonkey from github (Python version

[GitHub] cloudstack pull request: CLOUDSTACK-9230: Remove unnecessary retur...

2016-01-13 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1335 CLOUDSTACK-9230: Remove unnecessary return statement from cloudStack.js Removed the unnecessary return statement. The statement is never reached. You can merge this pull request into

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-171250460 LGTM, pinging @remibergsma --- 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 doe

[GitHub] cloudstack pull request: CLOUDSTACK-9160: Remove unused folder(s)/...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1238#issuecomment-171251445 @borisroman can you post any results if you have them? If you care you can force push to have jenkins validate the PR but as far as I can see no added value to

[GitHub] cloudstack pull request: Remove template ulimit from createtmplt.s...

2016-01-13 Thread lttmtins
Github user lttmtins commented on the pull request: https://github.com/apache/cloudstack/pull/1223#issuecomment-171251743 Actually the createtmplt.sh script starts with the hardcoded ulimit -f, but there is an '-S" flag within the script which would set the ulimit -f value to the Size

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-01-13 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-171251852 2xLGTM, no test results. Did anyone verify this works? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: Thoughts on Citrix announcement yesterday

2016-01-13 Thread Ian Rae
Sebastien, echoing your comments and sentiments. We understand Citrix has a market driven imperative to focus on their core products and services, and thank them for kickstarting and enabling this awesome Apache community that represents an end-user driven cloud orchestration tool of great value, t

[GitHub] cloudstack pull request: Remove template ulimit from createtmplt.s...

2016-01-13 Thread lttmtins
Github user lttmtins commented on the pull request: https://github.com/apache/cloudstack/pull/1223#issuecomment-171262924 However just removing the ulimit as per suggestion also works for me. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217 script to ...

2016-01-13 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-171271657 @DaanHoogland You will need two primary storage pools to test this , no advanced setup is needed to test the script. The script will skip if there are less than

Build failed in Jenkins: build-master-slowbuild #2975

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-171274511 ok, i am convinced. it has nothing to do with the code at hand. I will rerun the intergration to be sure but if I don't come back here: LGTM --- If your projec

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

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-171276640 @syed thanks for picking this up. Can you - link this to a ticket - describe any testing you did (I can't test this) --- If your project is set up for

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

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1332#issuecomment-171277602 looks good @syed Can you add test data/description --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1319#issuecomment-171278817 @nitin-maharana I see two LGTM in the other PR based on code review. One without explanation and none based on testing. I can queue this but you don't want to

[GitHub] cloudstack pull request: CLOUDSTACK-9230: Remove unnecessary retur...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1335#discussion_r49585171 --- Diff: ui/scripts/cloudStack.js --- @@ -213,8 +213,6 @@ domainid: g_domainid }

[GitHub] cloudstack pull request: CLOUDSTACK-9230: Remove unnecessary retur...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1335#issuecomment-171279858 LGTM, this return statement is immediately following another return statement and is thus unreachable. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-171282358 @remibergsma @ProjectMoon did himself, as I read it. This is a systemd installation script it won't be touched by the integration tests. You are not satisfied

[GitHub] cloudstack pull request: CLOUDSTACK-9198: Virtual router gets depl...

2016-01-13 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1278#issuecomment-171284609 @anshul1886 I am sorry if I wasn't clear. You are using the com.cloud.network.router.NetworkHelperImpl.startVirtualRouter(DomainRouterVO, User, Account, Map

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-01-13 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-171287727 @DaanHoogland This is not systemd, this is a python script to configure management. Why don't we build an environment and try this script with its parameters. E

[GitHub] cloudstack pull request: Followup fix for #1162: Add support for n...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1330#issuecomment-171288280 I know, I said 'systemd install script'. I am not taking the time to test this now. I can ask @davidamorimfaria . David? --- If your project is set up for it,

[GitHub] cloudstack pull request: Remove template ulimit from createtmplt.s...

2016-01-13 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1223#issuecomment-171289431 @syed Thanks for the fix, let's get it it. I'd like to merge it to 4.7 branch. Can you please close this PR, make a new one against 4.7 and then refer the new o

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217 script to ...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-171290722 @pavanb018 , running the test with a definition like below or should I have two clusters as well? ``` "clusters": [

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217 script to ...

2016-01-13 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-171298675 The above configuration is good. No need to have two clusters --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

Build failed in Jenkins: build-master-slowbuild #2976

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

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

2016-01-13 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1332#discussion_r49605888 --- Diff: plugins/storage/image/swift/src/org/apache/cloudstack/storage/datastore/driver/SwiftImageStoreDriverImpl.java --- @@ -67,7 +73,28 @@ public Data

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

2016-01-13 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1332#discussion_r49606475 --- Diff: plugins/storage/image/swift/src/org/apache/cloudstack/storage/datastore/driver/SwiftImageStoreDriverImpl.java --- @@ -67,7 +73,28 @@ public Data

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

2016-01-13 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1332#discussion_r49606572 --- Diff: plugins/storage/image/swift/src/org/apache/cloudstack/storage/datastore/driver/SwiftImageStoreDriverImpl.java --- @@ -67,7 +73,28 @@ public Data

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

2016-01-13 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1332#discussion_r49606982 --- Diff: utils/src/main/java/com/cloud/utils/SwiftUtil.java --- @@ -236,4 +247,60 @@ public static boolean deleteObject(SwiftClientCfg cfg, String path) {

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217 script to ...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-171349179 @pavanb018 no luck [1322.results.txt](https://github.com/apache/cloudstack/files/89168/1322.results.txt) --- If your project is set up for it, you can r

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217 script to ...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-171350897 I have no time to look in this further right now. maybe you recognise the problem right away. --- If your project is set up for it, you can reply to this emai

RE: Thoughts on Citrix announcement yesterday

2016-01-13 Thread Giles Sirett
Sebastien - thanks for your message I also echo your thoughts: I agree this is not an issue for the project or ASF to comment on but hopefully, at long last this, *may* stop people referring to "Citrix CloudStack" ! I hope that the individual contributors who are affected by this news are able

[GitHub] cloudstack pull request: CLOUDSTACK-9231: Root volume migration fr...

2016-01-13 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1336 CLOUDSTACK-9231: Root volume migration from one primary to another primary storage within the same cluster is failing EXPECTED BEHAVIOUR: Root Volume migrati

[GitHub] cloudstack pull request: CLOUDSTACK-9235: Autoscale button is miss...

2016-01-13 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1337 CLOUDSTACK-9235: Autoscale button is missing in VPC Autoscale button is missing. This should not be the case since we are able to add NS as the external LB provider in VPC. Step

[GitHub] cloudstack pull request: CLOUDSTACK-9235: Autoscale button is miss...

2016-01-13 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1337#issuecomment-171373038 Hi @nitin-maharana, What about a little change in your code? The complexity of that if/else structure that you are working on is too big.

[GitHub] cloudstack pull request: CLOUDSTACK-9236: Load Balancing Health Ch...

2016-01-13 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1338 CLOUDSTACK-9236: Load Balancing Health Check button displayed when non-NetScaler offering is used Load balancing health check option / button should only be displayed when a NetScaler b

[GitHub] cloudstack pull request: CLOUDSTACK-9236: Load Balancing Health Ch...

2016-01-13 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1338#issuecomment-171379436 @nitin-maharana, if you create the function I suggested you to create in PR #1337, you could re-use it here. The code block 3637-3650 is the same as the o

[GitHub] cloudstack pull request: CLOUDSTACK-9236: Load Balancing Health Ch...

2016-01-13 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1338#issuecomment-171381086 @rafaelweingartner : Super I will make the modification on both PRs. Thanks. --- If your project is set up for it, you can reply to this email and have you

Build failed in Jenkins: build-master-slowbuild #2977

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack-docs pull request: Update nuage-plugin.rst

2016-01-13 Thread FFincannon
GitHub user FFincannon opened a pull request: https://github.com/apache/cloudstack-docs/pull/16 Update nuage-plugin.rst Revised supported version to "Nuage VSP 3.2" and changed "VirtualRouter" to "VpcVirtualRouter". You can merge this pull request into a Git repository by running:

Build failed in Jenkins: build-master-slowbuild #2978

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2016-01-13 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-171442487 Thanks @DaanHoogland ! --- 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 hav

AW: Thoughts on Citrix announcement yesterday

2016-01-13 Thread Walter , André
Being quite new to the community, it is hard for me to judge on this announcement/change. But anyway, for me and my colleagues this does not change our (hopefully growing ;-)) engagement and I cannot see any negative impact. I am just looking forward for contributing for a good thing (therefore

Build failed in Jenkins: build-master-slowbuild #2979

2016-01-13 Thread jenkins
See -- [...truncated 28733 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2016-01-13 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1293#discussion_r49679830 --- Diff: plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/resource/wrapper/NiciraCheckHealthCommandWrapper.java --- @@ -0,0

[GitHub] cloudstack pull request: Add Health Check Command to NSX plugin

2016-01-13 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1293#discussion_r49679998 --- Diff: engine/orchestration/src/com/cloud/agent/manager/AgentManagerImpl.java --- @@ -821,9 +819,9 @@ protected boolean handleDisconnectWithI

Build failed in Jenkins: build-master-slowbuild #2980

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-8950 Hypervisor Paramete...

2016-01-13 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/928#discussion_r49688571 --- Diff: server/src/com/cloud/template/TemplateAdapterBase.java --- @@ -293,9 +298,15 @@ public TemplateProfile prepare(GetUploadParamsForTemplat

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217 script to ...

2016-01-13 Thread pavanb018
Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-171534991 From the results it looks like , suitable pools for migration for the volume returned no pools... I too didn't look at the file in detail. If you have setup y

[GitHub] cloudstack pull request: CLOUDSTACK-9198: Virtual router gets depl...

2016-01-13 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1278#issuecomment-171536737 Refer commits 7928963d16 and 11e1e585 to get info about Calling context. --- If your project is set up for it, you can reply to this email and have your reply ap

Unable to create VMs on master because of virtual router failing to start

2016-01-13 Thread Syed Mushtaq
Hi All, I am getting the following error on master when I try to create an instance SSH execution of command /opt/cloud/bin/router_proxy.sh get_template_version.sh 169.254.3.35 null has an error status code in return. result output: I couldn't find the script on the VR. I mounted the systemvm.is

Build failed in Jenkins: build-master-slowbuild #2981

2016-01-13 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Re: Unable to create VMs on master because of virtual router failing to start

2016-01-13 Thread Mike Tutkowski
Although my system VMs all start up on master, I do see this in the management server console output: WARN [c.c.n.r.VirtualNetworkApplianceManagerImpl] (RouterStatusMonitor-1:ctx-df8189be) (logid:fac1827c) Unable to get alerts from router r-4-VM bash: /opt/cloud/bin/getRouterAlerts.sh: No such fi

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2016-01-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-171553618 I am afraid the error is persistent and in two environments. I will have a look if I see the same behaiavure with other PRs --- If your project is set up for i

Re: Unable to create VMs on master because of virtual router failing to start

2016-01-13 Thread Syed Mushtaq
I can't find that file in the ISO too Mike. On Thu, Jan 14, 2016 at 1:27 AM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Although my system VMs all start up on master, I do see this in the > management server console output: > > WARN [c.c.n.r.VirtualNetworkApplianceManagerImpl] > (Ro

Re: LTS release or not

2016-01-13 Thread John Burwell
All, It seems that there is general agreement on the following: 1. There are users that need periodic releases that only focus on stability to minimize change and deployment risk (i.e. LTS releases) 2. There are users that need frequent releases that deliver new capabilities (i.e. monthly relea