Re: Review Request 20210: List templates returns destroyed temples when copying the same template after deleting.

2014-04-11 Thread bharat kumar
On April 10, 2014, 5:19 p.m., Nitin Mehta wrote: engine/storage/src/org/apache/cloudstack/storage/datastore/ObjectInDataStoreManagerImpl.java, line 134 https://reviews.apache.org/r/20210/diff/2/?file=554427#file554427line134 Why do we need to update the old entry and not create a

Re: Review Request 20210: List templates returns destroyed temples when copying the same template after deleting.

2014-04-11 Thread bharat kumar
On April 10, 2014, 5:25 p.m., Min Chen wrote: The root cause for this issue is that we didn't remove template_store_ref entry from db when a template is deleted from a zone and only mark them as 'Destroyed'. In addition, template_view db view is created by joining with those

Re: Review Request 19875: Fixed few issues for CLOUDSTACK-5674, CLOUDSTACK-6316

2014-04-11 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19875/#review40139 --- Commit ca67e25f2399d4c2bb7b7fde30616f6acf683f16 in cloudstack's

Jenkins build is back to normal : cloudstack-4.3-forward-maven-build #160

2014-04-11 Thread jenkins
See http://jenkins.buildacloud.org/job/cloudstack-4.3-forward-maven-build/160/changes

Cloudstack support for Xen (on Ubuntu)

2014-04-11 Thread Pradeep Cloudstack
I have a server running Ubuntu 13.04. I am planning to install Xen Hypervisor on that (https://help.ubuntu.com/community/XenProposed#Xen_and_XAPI) Can I then have it managed (as a host) by Cloudstack 4.2 ? -pradeep

Re: Cloudstack support for Xen (on Ubuntu)

2014-04-11 Thread Nux!
On 11.04.2014 10:10, Pradeep Cloudstack wrote: I have a server running Ubuntu 13.04. I am planning to install Xen Hypervisor on that (https://help.ubuntu.com/community/XenProposed#Xen_and_XAPI) Can I then have it managed (as a host) by Cloudstack 4.2 ? Not yet, there are plans to make it

NodeJS Client for CloudStack

2014-04-11 Thread Madan Ganesh V
Hello there, I’m writing nodeJS client application to interact with CloudStack. I notice csclient and node-cloudstack modules. csclient throws API error 432 for listVirtualMachines. Anybody has played with these modules in Node JS.  Appreciate your pointers/ recommendations. Cheers, Madan

Instance can't use data disk in Devcloud

2014-04-11 Thread dimas yoga pratama
Hi all, I am trying to build cloudstack 4.2 using my laptop with Devcloud by following this guidehttp://www.leaseweblabs.com/2014/02/test-apache-cloudstack-4-2-local-machine/and, everything went smooth. But when I tried to create an instance with data disk offering I've got this following error

RE: can't get the simulator running on 4.4

2014-04-11 Thread Gabor Apati-Nagy
Hi Tina, I have tried to get the Simulator working on branch 4.4 and I can confirm it works fine on Ubuntu. Now, the following entries are in my version table: 4.0.0 4.1.0 4.2.0 4.2.1 4.3.0 4.4.0 Maybe you could try cloning 4.4 to a different directory, drop DB and see if that helps.

Re: Instance can't use data disk in Devcloud

2014-04-11 Thread Ian Duffy
I assume you are using local storage for primary? If so you need to adjust the disk offering to use local storage. On Apr 11, 2014 11:48 AM, dimas yoga pratama smid...@gmail.com wrote: Hi all, I am trying to build cloudstack 4.2 using my laptop with Devcloud by following this guide

Re: Review Request 19922: CLOUDSTACK-6251: Removed XenServer stub code and used xapi.jar from the central repository instead.

2014-04-11 Thread Alex Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19922/#review40150 --- Tina, I missed this on the first review. The current change adds

Re: Cloudstack support for Xen (on Ubuntu)

2014-04-11 Thread Rafael Weingartner
True that, just xen hypervisor is not yet supported. However, if you also install the XCP over xen hypervisor it will work. On Fri, Apr 11, 2014 at 7:05 AM, Nux! n...@li.nux.ro wrote: On 11.04.2014 10:10, Pradeep Cloudstack wrote: I have a server running Ubuntu 13.04. I am planning to

Ceph RBD speed and snapshot usage improvements for CloudStack 4.4

2014-04-11 Thread Wido den Hollander
Hi, I just pushed to commits to master and cherry picked them to the 4.4 branch to resolve a long outstanding issue with the time it takes to deploy templates and back up snapshots of RBD. The two commits [0][1] also address the space used by RBD snapshots on Secondary Storage. Before

Some problem registering ISO

2014-04-11 Thread Giri Prasad
Hi All,  I have installed cloudstack 4.1 on Ubuntu 12.04 LTS. The system vm's were correctly created and are running.  I have also installed apache2 and configured it as shown below a few lines. I copied the ubuntu iso file to /var/www dir.   I tried to Templates - Select View ISO, Register

Problem Registering an ISO

2014-04-11 Thread Giri Prasad
Hi All,  I have installed cloudstack 4.1 on Ubuntu 12.04 LTS. The system vm's were correctly created and are running.  I have also installed apache2 and configured it as shown below a few lines. I copied the iso file to /var/www dir.   I tried to Templates - Select View ISO, Register ISO,

Re: Review Request 19875: Fixed few issues for CLOUDSTACK-5674, CLOUDSTACK-6316

2014-04-11 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19875/#review40154 --- Commit b2543efaca31bc7932ebe6ecefa963849049f39f in cloudstack's

Re: [RFC]Bypass Libvirt storage pool for NFS

2014-04-11 Thread Wido den Hollander
On 03/20/2014 07:51 PM, Nux! wrote: On 20.03.2014 18:48, Wido den Hollander wrote: And it just went upstream! How great is that? Pretty great. That's how open source works. :) Quick question: there is no problem if instead of using NFS directly we use the shared mount point option, is

Review Request 20257: CLOUDSTACK-6390 : System vm failed to build with openssl installation error,

2014-04-11 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20257/ --- Review request for cloudstack, Abhinandan Prateek and Frank Zhang. Bugs:

Re: Review Request 19922: CLOUDSTACK-6251: Removed XenServer stub code and used xapi.jar from the central repository instead.

2014-04-11 Thread Konstantina Chremmou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19922/ --- (Updated April 11, 2014, 2:44 p.m.) Review request for cloudstack and Alex

Review Request 20258: CLOUDSTACK-1466: Automation tests for Primary Storage Limits Feature

2014-04-11 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20258/ --- Review request for cloudstack, Girish Shilamkar and Santhosh Edukulla. Bugs:

Security Group bug impeding system VMs functionality

2014-04-11 Thread Nux!
Hello, I'm on 4.3 right now, CentOS6.5 + KVM and SG ADV zone. What happens is that after a reboot or after disabling a zone, when the system VMs come back the iptables rules required for their proper functioning do not get set. It seems to be happening randomly and it may not be affecting both

Review Request 20261: CLOUDSTACK-6233: Add new tab GPU in Host detailView for gpu enabled hosts

2014-04-11 Thread Mihaela Stoica
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20261/ --- Review request for cloudstack, Brian Federle and Jessica Wang. Repository:

Re: Review Request 20261: CLOUDSTACK-6233: Add new tab GPU in Host detailView for gpu enabled hosts

2014-04-11 Thread Mihaela Stoica
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20261/ --- (Updated April 11, 2014, 4:49 p.m.) Review request for cloudstack, Brian

RE: Review Request 19616: Added check for null return.

2014-04-11 Thread Alex Hitchins
All, I've made the following changes to public Snapshot takeSnapshot(Long volumeId, Long policyId, Long snapshotId, Account account, boolean quiescevm), is this the preferred approach? I don't get any compilation issues with this. I want to ensure I've taken your comments on fully. try {

Re: Instance can't use data disk in Devcloud

2014-04-11 Thread dimas yoga pratama
Hey it works! Many Thanks. On Fri, Apr 11, 2014 at 6:33 PM, Ian Duffy i...@ianduffy.ie wrote: I assume you are using local storage for primary? If so you need to adjust the disk offering to use local storage. On Apr 11, 2014 11:48 AM, dimas yoga pratama smid...@gmail.com wrote: Hi all,

Coding Standards Questions

2014-04-11 Thread Alex Hitchins
All, As I've been looking through the code, I've seen a fair number of places where return statements are called within if statements and the like. I've always found that having one place to return is easier to debug and follow the code flow. Are there any guidelines on this? Or is it not

Re: Cloudstack support for Xen (on Ubuntu)

2014-04-11 Thread Tracy Phillips
Rafael, Once you put XCP on, do you gain features that belong to Xenserver or is it limited in someway? The only reason I am not using Xenserver is that I have to manage it differently than my other hosts. Tracy Tracy Phillips Weberize, Inc. On Fri, Apr 11, 2014 at 8:26 AM, Rafael

Re: Problem Registering an ISO

2014-04-11 Thread Amogh Vasekar
Is secstorage.allowed.internal.sites set correctly? Thanks, Amogh On 4/11/14 6:52 AM, Giri Prasad g_p...@yahoo.com wrote: Hi All, I have installed cloudstack 4.1 on Ubuntu 12.04 LTS. The system vm's were correctly created and are running. I have also installed apache2 and configured it as

[CLOUDSTACK-5907] Issue regression / not fixed

2014-04-11 Thread Tanner Danzey
I would like to draw attention to the aforementioned issue. With an out of the box fresh 4.3 release setup using RBD primary, S3 secondary and KVM hypervisors, I experience newly created volumes changing to Ovm format soon after creation. This disables snapshotting of volumes, which worked fine

CS Upgrade from 4.1.x to 4.2.1, registering new System VMs

2014-04-11 Thread Rafael Weingartner
Hi folks, I am planning my upgrade from 4.1.1 to 4.2.1, reading the docs here: http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.2.1/html/Release_Notes/upgrade-instructions.html#upgrade-from-4.1-to-4.2.1 . After I register the new system vm templates, do not I need to remove the old

Re: Coding Standards Questions

2014-04-11 Thread Daan Hoogland
H Alex, I agree with you that would be nicer if your function fits in a screen. Another coding convention we should adhere to. As it is I think it not so much 'not a major concern' as too much to ask for. Feel free to refactor and submit patches;) Daan On Fri, Apr 11, 2014 at 9:54 AM, Alex

Re: Review Request 19616: Added check for null return.

2014-04-11 Thread Daan Hoogland
On Fri, Apr 11, 2014 at 9:50 AM, Alex Hitchins a...@alexhitchins.com wrote: if(snapshotInfoReturn.equals(null)){ throw new ResourceAllocationException(Take snapshot for VolumeId: + volumeId + failed., ResourceType.snapshot); } This will never throw a

Re: Review Request 20257: CLOUDSTACK-6390 : System vm failed to build with openssl installation error,

2014-04-11 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20257/#review40172 --- Ship it! Ship It! - Frank Zhang On April 11, 2014, 2:39 p.m.,

Re: Review Request 20257: CLOUDSTACK-6390 : System vm failed to build with openssl installation error,

2014-04-11 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20257/#review40171 --- Commit a5261cee3f63cbb30db06c7a4e5f35c8a5fa957c in cloudstack's

RE: Review Request 19616: Added check for null return.

2014-04-11 Thread Alex Hitchins
OK, so that should be if(snapshotInfoReturn==null){ throw new ResourceAllocationException(Take snapshot for VolumeId: + volumeId + failed., ResourceType.snapshot); } So I get null - I should raise an exception. Alex Hitchins | 07788 423 969 | 01892 523 587

Re: Review Request 19616: Added check for null return.

2014-04-11 Thread Daan Hoogland
sure, I still don't like the fact that we need to throw an exception that we caught one level deeper. @Edison: can we change the internal api to throw the exception? On Fri, Apr 11, 2014 at 12:47 PM, Alex Hitchins a...@alexhitchins.com wrote: OK, so that should be

RE: Review Request 19616: Added check for null return.

2014-04-11 Thread Alex Hitchins
Cool, once I've been over it with a test I'll update the patch. Alex Hitchins | 07788 423 969 | 01892 523 587 - -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: 11 April 2014 20:56 To: dev Cc: Alex Hitchins;

RE: Coding Standards Questions

2014-04-11 Thread Alex Hitchins
Daan, Are you referring to keeping line lengths up to 80 characters? Sorry - tired eyes. My thoughts were more that in a function there should only be one return statement rather than many, all nested in layers of if/else statements. Alex Hitchins | 07788 423 969 | 01892 523 587

Re: CS Upgrade from 4.1.x to 4.2.1, registering new System VMs

2014-04-11 Thread ilya musayev
Yes you will have to destroy the old ones and new ones will be created based on the new template you've uploaded. Regards ilya On 4/11/14, 11:29 AM, Rafael Weingartner wrote: Hi folks, I am planning my upgrade from 4.1.1 to 4.2.1, reading the docs here:

Re: CS Upgrade from 4.1.x to 4.2.1, registering new System VMs

2014-04-11 Thread Rafael Weingartner
When I asked about removal, I meant the template by themselves. The running vms I knew that I would have to remove. On Fri, Apr 11, 2014 at 10:28 PM, ilya musayev ilya.mailing.li...@gmail.com wrote: Yes you will have to destroy the old ones and new ones will be created based on the new

[ANNOUNCE] Paul Angus as a committer

2014-04-11 Thread Sebastien Goasguen
We are pleased to announce that Paul Angus was invited to be a committer on the Apache Cloudstack project and that he accepted, Join us in welcoming and congratulating Paul Angus, -Sebastien on behalf of the CloudStack PMC

Re: [ANNOUNCE] Paul Angus as a committer

2014-04-11 Thread Chip Childers
Congrats! On Apr 11, 2014, at 8:48 PM, Sebastien Goasguen run...@gmail.com wrote: We are pleased to announce that Paul Angus was invited to be a committer on the Apache Cloudstack project and that he accepted, Join us in welcoming and congratulating Paul Angus, -Sebastien on behalf of

Re: [ANNOUNCE] Paul Angus as a committer

2014-04-11 Thread Nguyen Anh Tu
Congrats, Paul! --Tuna On Sat, Apr 12, 2014 at 9:59 AM, Chip Childers chip.child...@gmail.comwrote: Congrats! On Apr 11, 2014, at 8:48 PM, Sebastien Goasguen run...@gmail.com wrote: We are pleased to announce that Paul Angus was invited to be a committer on the Apache Cloudstack

Re: [ANNOUNCE] Paul Angus as a committer

2014-04-11 Thread Ahmad Emneina
Excellent news, congrats Paul! On Fri, Apr 11, 2014 at 8:04 PM, Nguyen Anh Tu t...@apache.org wrote: Congrats, Paul! --Tuna On Sat, Apr 12, 2014 at 9:59 AM, Chip Childers chip.child...@gmail.com wrote: Congrats! On Apr 11, 2014, at 8:48 PM, Sebastien Goasguen run...@gmail.com