Re: Review Request 22510: [Windows] Can not create Template from ROOT snapshot. Fixed the same by adding pathSeparator to the DataStoreTO.java

2014-06-16 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22510/#review45720 --- api/src/com/cloud/agent/api/to/NfsTO.java

Review Request 22613: CLOUDSTACK-6912: 4.2 32 bit system vm template job failing in jenkins

2014-06-16 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22613/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: CLOUDSTACK-6912

Re: Review Request 22613: CLOUDSTACK-6912: 4.2 32 bit system vm template job failing in jenkins

2014-06-16 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22613/#review45721 --- Ship it! Ship It! - Abhinandan Prateek On June 16, 2014, 6:10

Re: Review Request 22613: CLOUDSTACK-6912: 4.2 32 bit system vm template job failing in jenkins

2014-06-16 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22613/#review45722 --- Commit a2b620c0b43dc6739dab3fdb854421bf63f72f1b in cloudstack's

Re: Review Request 22510: [Windows] Can not create Template from ROOT snapshot. Fixed the same by adding pathSeparator to the DataStoreTO.java

2014-06-16 Thread Damodar Reddy Talakanti
On June 16, 2014, 6:04 a.m., Koushik Das wrote: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/XenServerStorageProcessor.java, line 1407 https://reviews.apache.org/r/22510/diff/1/?file=608108#file608108line1407 Is the intent to replace all occurrences

RE: Code Documentation

2014-06-16 Thread Santhosh Edukulla
Agreed, If we are referring to proper naming convention for self explaining the code, yes. But, still adding better and more comment strings or doc strings at appropriate places, is not going to harm i believe, it just compliments and aids better understanding. We used to add specific

Re: Add S3 secondary storage

2014-06-16 Thread Sebastien Goasguen
On Jun 16, 2014, at 12:14 AM, Sanjeev Neelarapu sanjeev.neelar...@citrix.com wrote: Hi Sebastien, #1 For adding S3 object store as secondary storage use addImageStore API with provider as s3 http://cloudstack.apache.org/docs/api/apidocs-4.3/root_admin/addImageStore.html Wow, looking at

Re: Can we delete a template after a VM is instantiated from it?

2014-06-16 Thread Nux!
Hi, You can remove it from the UI, but not directly from the disk as it is used (in many cases) as a backing file for the VMs spawned from it. HTH Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - From: Sanjeev Neelarapu

RE: Add S3 secondary storage

2014-06-16 Thread Sanjeev Neelarapu
addS3 api has been deprecated. From 4.3 onwards we use only addImageStore API for any type of secondary storage (e.g NFS/S3/Swift) -Sanjeev -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Monday, June 16, 2014 12:59 PM To: dev@cloudstack.apache.org Subject:

Re: Code Documentation

2014-06-16 Thread Rajani Karuturi
IMO, comments are part of code. By writing code comments, we aren’t maintaining comments and code but rather helping others understand what the code is doing. As a developer, if I can easily understand what the code is doing, I won’t even look at the comments. In our codebase we have functions

Re: Review Request 22256: CLOUDSTACK-6830 :[hyper-v] fixed vm migration failing between different clusters if volumes are on zone wide primary store

2014-06-16 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22256/ --- (Updated June 16, 2014, 9:58 a.m.) Review request for cloudstack, Devdeep

Re: Review Request 20316: CLOUDSTACK-1466: Adding automation test cases for Primary Storage Limits

2014-06-16 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20316/#review45730 --- Gentle Reminder. - Gaurav Aradhye On June 6, 2014, 3:10 p.m.,

Re: Review Request 20316: CLOUDSTACK-1466: Adding automation test cases for Primary Storage Limits

2014-06-16 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20316/#review45731 --- Gentle Reminder. - Gaurav Aradhye On June 6, 2014, 3:10 p.m.,

Re: Review Request 22501: CLOUDSTACK-6282 : Modified IPAddresses tests for the failed testcases to create an account at test level not at class level. Also modified Templates to read register data fro

2014-06-16 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22501/#review45733 --- Commit 89c35abb4db0f362f9159ea39a6eb70ad4351921 in cloudstack's

Re: Review Request 22501: CLOUDSTACK-6282 : Modified IPAddresses tests for the failed testcases to create an account at test level not at class level. Also modified Templates to read register data fro

2014-06-16 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22501/#review45734 --- Commit 99e4da15da73ac2ccf5597e11d805cb1438831cd in cloudstack's

Review Request 22626: CLOUDSTACK-6914 :Fixed the issue

2014-06-16 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22626/ --- Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-6914

Re: Review Request 22626: CLOUDSTACK-6914 :Fixed the issue

2014-06-16 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22626/ --- (Updated June 16, 2014, 11:31 a.m.) Review request for cloudstack and

Re: Review Request 22626: CLOUDSTACK-6914 :Fixed the issue

2014-06-16 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22626/#review45735 --- Ship it! Ship It! - Abhinandan Prateek On June 16, 2014, 11:31

Re: Review Request 22626: CLOUDSTACK-6914 :Fixed the issue

2014-06-16 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22626/#review45736 --- Commit 54e4c075287957821e57368f1baf401761f7bda2 in cloudstack's

Re: Review Request 22544: CLOUDSTACK-6793: Fixed the issue in master

2014-06-16 Thread Abhinandan Prateek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22544/#review45737 --- Ship it! Ship It! - Abhinandan Prateek On June 13, 2014, 6:21

Re: Review Request 22544: CLOUDSTACK-6793: Fixed the issue in master

2014-06-16 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22544/#review45738 --- Commit f89100ed72bd8e854dc01af244907fe02e7525a3 in cloudstack's

Re: Review Request 22525: Fixed few reported resource leak issues

2014-06-16 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22525/#review45741 --- engine/schema/src/com/cloud/vm/dao/UserVmDaoImpl.java

Re: Review Request 22525: Fixed few reported resource leak issues

2014-06-16 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22525/#review45742 --- Ship it! 3c5f64c97ab071451aee7f232d8f2cec6d89044d in 4.4-forward

Re: Review Request 22525: Fixed few reported resource leak issues

2014-06-16 Thread daan Hoogland
On June 16, 2014, 12:23 p.m., daan Hoogland wrote: 3c5f64c97ab071451aee7f232d8f2cec6d89044d in 4.4-forward d86b6778adbd754a7e98c29d854b3c265954bca8 in master - daan --- This is an automatically generated e-mail. To reply, visit:

Build failed in Jenkins: build-master #969

2014-06-16 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/969/changes Changes: [Daan Hoogland] Fixed few resource leak issues -- [...truncated lines...] Results : Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 [INFO]

Blogs

2014-06-16 Thread sebgoa
The website points to our blog at: https://blogs.apache.org/cloudstack/ I am pretty sure we used to aggregate bunch of our own blogs there, but it does not look like it's being done now. Did something change ? Also, where is our planet ? I thought we got it done -sebastien

Review Request 22631: supporting nfs protocol and resize feature for managed storage in xenserver.

2014-06-16 Thread punith s
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22631/ --- Review request for cloudstack, amit kumar das, edison su, and Mike Tutkowski.

Re: Review Request 22608: Fixed resource leaks, null derefernces and other issues reported

2014-06-16 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22608/#review45746 --- Ship it! dbfce8b11e25855a8cb21ec2bbbfe20940bb788d - daan Hoogland

Jenkins build is back to normal : build-master #970

2014-06-16 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/970/changes

[ACS44] are we RC-ready?

2014-06-16 Thread Daan Hoogland
s far as I can tell the only remaining blocker issue has been fixed last week. There are 84 criticals. Do we feel we are RC ready as we are or are some of those critical to critical to go and release? If so, what needs to be done to get there. Please respond promptly or show yourself on

[ACS44] Cherry Pick Request

2014-06-16 Thread Santhosh Edukulla
Daan, Please pick up the below fixes for resource leaks and other Coverity issues. 4ba3aae2ea0290ca012623083e13625e08a9dde5 3c5f64c97ab071451aee7f232d8f2cec6d89044d dbfce8b11e25855a8cb21ec2bbbfe20940bb788d As well, please check the below commit, its fix for CLOUDSTACK-6793

Re: Blogs

2014-06-16 Thread David Nalley
http://planet.apache.org/cloudstack On Mon, Jun 16, 2014 at 9:24 AM, sebgoa run...@gmail.com wrote: The website points to our blog at: https://blogs.apache.org/cloudstack/ I am pretty sure we used to aggregate bunch of our own blogs there, but it does not look like it's being done now.

#cloudstack-meeting

2014-06-16 Thread Daan Hoogland
I can't connect to irc.freenode.net anybody else got problems with it? -- Daan

Re: #cloudstack-meeting

2014-06-16 Thread Pierre-Luc Dion
There was this message this morning on IRC: 07:37 ASFBot: [Announcement] Be advised; Our data center in the US is undergoing upgrades to their network this afternoon. This may lead to temporary downtime for many core services during the next few hours, including this lil' IRC bot. We expect the

Re: #cloudstack-meeting

2014-06-16 Thread sebgoa
looks like 13 of us are in there right now On Jun 16, 2014, at 5:09 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: I can't connect to irc.freenode.net anybody else got problems with it? -- Daan

Re: #cloudstack-meeting

2014-06-16 Thread Daan Hoogland
hope you have a RM in :( On Mon, Jun 16, 2014 at 5:12 PM, sebgoa run...@gmail.com wrote: looks like 13 of us are in there right now On Jun 16, 2014, at 5:09 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: I can't connect to irc.freenode.net anybody else got problems with it? -- Daan

Re: Review Request 22631: supporting nfs protocol and resize feature for managed storage in xenserver.

2014-06-16 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22631/#review45757 --- I was curious about the resizing of the VDI to take up all of the

Re: irc meeting on rvr4vpc

2014-06-16 Thread Karl Harris
Outlined below is an overview of the analysis and in process work for adding Virtual Redundant Routing to CloudStack Virtual Private Clouds. Current state: Public networks allow for redundant VR (Virtual Routers). The network topology is static. The network topology consists of a single guest

Re: Review Request 22631: supporting nfs protocol and resize feature for managed storage in xenserver.

2014-06-16 Thread Mike Tutkowski
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22631/#review45762 ---

Re: [ACS44] Cherry Pick Request

2014-06-16 Thread Daan Hoogland
On Mon, Jun 16, 2014 at 4:28 PM, Santhosh Edukulla santhosh.eduku...@citrix.com wrote: Daan, Please pick up the below fixes for resource leaks and other Coverity issues. 4ba3aae2ea0290ca012623083e13625e08a9dde5 was in already? 3c5f64c97ab071451aee7f232d8f2cec6d89044d in

Re: DevCloud replacement

2014-06-16 Thread Leo Simons
Hey Ian, Nice work! I got this working on my home PC on the weekend [1]. The readme on github could probably do with pointers to how to set up the host-only network which is also used for devcloud: https://cwiki.apache.org/confluence/display/CLOUDSTACK/DevCloud#DevCloud-Se

[ACS44] irc meeting on release

2014-06-16 Thread Daan Hoogland
We had a badly attended meeting on the next release here are the minutes http://wilderness.apache.org/archives/cloudstack-meeting-16_06_2014-5933.html next week same time (if still needed) -- Daan

Re: Can we delete a template after a VM is instantiated from it?

2014-06-16 Thread Rafael Weingartner
This way, it seems that you have a bug in CS 4.3.0 when starting a machine that was created from a template that has been deleted. There will happen a null pointer exception in: “858 -if (volTemplateId != null volTemplateId.longValue() != template.getId())” The object, “template” is

Re: [ACS44] irc meeting on release

2014-06-16 Thread Mike Tutkowski
Thanks, Daan! As I mentioned, I had another meeting I had to attend at that time today, but I should be able to attend next week. On Mon, Jun 16, 2014 at 10:20 AM, Daan Hoogland daan.hoogl...@gmail.com wrote: We had a badly attended meeting on the next release here are the minutes

Re: DevCloud replacement

2014-06-16 Thread Rohit Yadav
Hi Leo, Ian's xenserver vagrant works on host-only too. If you do 'vagrant up' and if the VMs boot up you verify with VirtualBox that the second VM (xenserver) will have nic0 (NAT) and nic1 (host-only with promiscuous mode enabled). If that is not the case I suspect some issue in Chef/Vagrant

Re: DevCloud replacement

2014-06-16 Thread Ian Duffy
Hi Leo, Thanks for the feedback. Will add some virtualbox network instructions, its something I always overlook, always set and forget. The corporate firewall issue sounds interesting. What is the name of it just out of interest? I've had issues in the past on a mac with communications being

Re: irc meeting on rvr4vpc

2014-06-16 Thread Daan Hoogland
H Karl, We will have a look at this with the team at Schuberg Philis, thanks. On Mon, Jun 16, 2014 at 5:46 PM, Karl Harris karl.har...@sungardas.com wrote: Outlined below is an overview of the analysis and in process work for adding Virtual Redundant Routing to CloudStack Virtual Private

Build failed in Jenkins: build-master-noredist #2914

2014-06-16 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/2914/ -- Started by upstream project build-master build number 968 originally caused by: Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on

Jenkins build is back to normal : build-master-noredist #2915

2014-06-16 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-noredist/2915/changes

[API]How to check how many IPs are available in guest network

2014-06-16 Thread ilya musayev
Is there a way to check how many guest IPs are available in specific guest network? Current cloudstack reporting is not granular enough, i could not find a command that says listAvailableIPs or something similar. Any help is appreciated. Thanks, ilya

Re: [API]How to check how many IPs are available in guest network

2014-06-16 Thread Wei ZHOU
lisPublicIpAddresses can list all public ips in a shared network, or public ips used in isolated network. state=Free means the ip is available. 2014-06-16 21:21 GMT+02:00 ilya musayev ilya.mailing.li...@gmail.com: Is there a way to check how many guest IPs are available in specific guest

Re: [API]How to check how many IPs are available in guest network

2014-06-16 Thread Chiradeep Vittal
used=$(cloudmonkey list virtualmachines networkid=f5119c7a-a5b8-47aa-bf26-d0f41dfa2d6b | grep ipaddress | wc -l) space=$(cloudmonkey list networks id=f5119c7a-a5b8-47aa-bf26-d0f41dfa2d6b | grep cidr | awk -F: '{print $2}' | awk -F / '{print $2}') space=${space%\,} let space=32 - $space let

RE: [ACS44] are we RC-ready?

2014-06-16 Thread Animesh Chaturvedi
84 is a bug number to call RC ready unless the issues are not really critical -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Monday, June 16, 2014 7:11 AM To: dev Subject: [ACS44] are we RC-ready? s far as I can tell the only remaining blocker issue

[ACS44] Cherry pick request

2014-06-16 Thread Amogh Vasekar
Hi Daan, Can you please cherry-pick de04881da177b28d7cb3efc09adb2d8069f0fdd3 from 4.4-forward to 4.4? This fixes CLOUDSTACK-6915 Thanks, Amogh

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-16 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/#review45853 --- Alex, * add since=version attribute to all new parameters you've

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-16 Thread Alex Ough
On Mon, Jun 16, 2014 at 7:07 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20099/#review45853

Re: Review Request 20099: Domain-Account-User Sync Up Among Multiple Regions (Core Changes)

2014-06-16 Thread Alena Prokharchyk
On 6/16/14, 4:13 PM, Alex Ough alex.o...@sungardas.com wrote: On Mon, Jun 16, 2014 at 7:07 PM, Alena Prokharchyk alena.prokharc...@citrix.com wrote: --- This is an automatically generated e-mail. To reply, visit:

Re: #cloudstack-meeting

2014-06-16 Thread ilya musayev
Hi Guys, Sorry i could not attend as i'm having busy day @ $dayjob. Are we planning on making these meetings regular occurrence? if so, i can block out some time in my calendar. Thanks, ilya On 6/16/14, 8:09 AM, Daan Hoogland wrote: I can't connect to irc.freenode.net anybody else got

Re: #cloudstack-meeting

2014-06-16 Thread Mike Tutkowski
It sounds like there will be at least one more next Monday at the same time. On Mon, Jun 16, 2014 at 9:43 PM, ilya musayev ilya.mailing.li...@gmail.com wrote: Hi Guys, Sorry i could not attend as i'm having busy day @ $dayjob. Are we planning on making these meetings regular occurrence? if

Review Request 22672: CLOUDSTACK-6914: Fixed the mentioned issue for vgpu cases along with tagging

2014-06-16 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22672/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: CLOUDSTACK-6914

Re: [API]How to check how many IPs are available in guest network

2014-06-16 Thread ilya musayev
Thanks Chiradeep and Wei Unfortunately neither gave me what i needed, so i wrote this SQL script: SELECT b.name, count(*) FreeIPCount FROM cloud.user_ip_address a, cloud.networks b WHERE b.id=a.source_network_id AND a.state = Free and b.name is not null group by b.name; Chiradeep, for some

Re: [API]How to check how many IPs are available in guest network

2014-06-16 Thread Chiradeep Vittal
Can you explain does not work -- Chiradeep On Jun 16, 2014, at 10:02 PM, ilya musayev ilya.mailing.li...@gmail.com wrote: Thanks Chiradeep and Wei Unfortunately neither gave me what i needed, so i wrote this SQL script: SELECT b.name, count(*) FreeIPCount FROM cloud.user_ip_address

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-16 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22554/#review45910 --- tools/marvin/marvin/deployDataCenter.py

Re: [API]How to check how many IPs are available in guest network

2014-06-16 Thread ilya musayev
It returns nothing, just a blank like. I will be upgrading our CloudStack instances this week to see if the issue still persists with 4.3.0, but i dont believe it will change my result. On 6/16/14, 10:14 PM, Chiradeep Vittal wrote: Can you explain does not work -- Chiradeep On Jun 16,

Re: [API]How to check how many IPs are available in guest network

2014-06-16 Thread Wei ZHOU
try adding listall=true to cloudmonkey command? 2014-06-17 7:44 GMT+02:00 ilya musayev ilya.mailing.li...@gmail.com: It returns nothing, just a blank like. I will be upgrading our CloudStack instances this week to see if the issue still persists with 4.3.0, but i dont believe it will change