Re: [PROPOSAL] List VM API enhancement

Created a review request https://reviews.apache.org/r/19022/. Since this is a small change not creating a separate merge request mail. -Koushik On 13-Feb-2014, at 2:23 PM, Daan Hoogland wrote: > Koushik, > > I am afraid I am not. Using English for identifiers there is a semantic > differnce of

Review Request 19022: List VM enhancement to support querying with multiple VM IDs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19022/ --- Review request for cloudstack, daan Hoogland and Min Chen. Bugs: CLOUDSTACK-605

Re: [DISCUSS] Enabling databse upgrades on master branch

Hi Miguel, This is in-line with discussions related to db changes we are having at [1] and [2] I think it would be better to use existing tools like [liquibase] or [flyway] instead of writing a new one. A good comparison of the both is available at [3]. Also, how do we join the efforts? Is the

RE: Just checked in a minor change to the DB

Second task it will solve the purpose of the first one. It will awesome if both are done. Thanks Rajesh Battala -Original Message- From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Tuesday, March 11, 2014 9:24 AM To: dev@cloudstack.apache.org Subject: Re: Just checked in a

Re: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17889/#review36759 --- Ship it! Ship It! - Devdeep Singh On Feb. 27, 2014, 12:35 p.m.,

Re: [QUESTION] XenServer issue attaching a disk to a VM

Sorry...obvious issue, but wanted to point out here from my previous e-mail: "The problem is in the needMoveVolume method in the VolumeApiServiceImpl method." I meant VolumeApiServiceImpl class...not method, of course. :) On Mon, Mar 10, 2014 at 10:38 PM, Mike Tutkowski < mike.tutkow...@solidfi

[QUESTION] XenServer issue attaching a disk to a VM

Hi, I noticed a minor problem trying to attach a volume that is at the cluster scope to a VM whose root disk is at the zone scope IF the VM is in the stopped state (this works fine when the VM is in the running state). The problem is in the needMoveVolume method in the VolumeApiServiceImpl method

Re: Just checked in a minor change to the DB

How's about this? It sounds like Daan is looking into Rajesh's idea about putting in a server-side Git hook to notify dev@ when *.sql changes. That is good. If it is reasonably possible to tell dev@ how to manually upgrade the DB, it would be great if the developer in question sent out such an e-

RE: Just checked in a minor change to the DB

Master to be stable all the times. We should conclude on the thread about notification on schema changes. It will be so much pain when about to push the patch and rebased with latest master to test and figured out that we have to deploy the db again. Thanks Rajesh Battala -Original Messag

[VOTE] Apache CloudStack 4.3.0 (seventh round)

Hi All, I've created a 4.3.0 release, with the following artifacts up for a vote: Git Branch and Commit SH: https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.3 Commit: ec4db7bbff60f0be96e39f51c0f17b12e0de440c List of changes: New Features in 4.3: htt

Re: [Merge] CloudStack IAM branch to master

No new jar dependencies. -min Sent from my iPhone > On Mar 10, 2014, at 7:22 PM, "Chiradeep Vittal" > wrote: > > Any new jar dependencies? > >> On 3/10/14, 11:34 AM, "Min Chen" wrote: >> >> Hi, >> >> Prachi and I would like to merge CloudStack Identity and Access >> Management(IAM) plugin

Re: system vm disk space issue in ACS 4.3

Yes, it was deliberate. I can¹t find the discussion, but it revolved around a security best practice of having separate partitions for /, /swap, home directories On 3/10/14, 11:35 AM, "Marcus" wrote: >There have been several raised, actually regarding /var/log. As for >the system vm partitioni

Re: [Merge] CloudStack IAM branch to master

Any new jar dependencies? On 3/10/14, 11:34 AM, "Min Chen" wrote: >Hi, > >Prachi and I would like to merge CloudStack Identity and Access >Management(IAM) plugin services to the master branch. Development for >this effort has been done by Prachi and me on ACS rbac branch >(https://git-wip-us.apa

Mascot

FYI: My cat's decided he's the new Apache CloudStack mascot: http://i.imgur.com/zMcLGN5.jpg Fortunately I wasn't in need of the keys in the upper left of my keyboard this afternoon. ;) -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e: mike.tutkow...@solidfire.com o: 303.746.7

Re: system vm disk space issue in ACS 4.3

we can maybe switch to an agent properties for the system vms that use a size-based roll for logging. On Mon, Mar 10, 2014 at 5:30 PM, Anirban Chakraborty wrote: > Thanks for all the responses. I do not see cloud.log and cloud.out logs are > zipped in /var/log and /var/log/cloud respectively. On

Re: system vm disk space issue in ACS 4.3

Thanks for all the responses. I do not see cloud.log and cloud.out logs are zipped in /var/log and /var/log/cloud respectively. Only file that was zipped was cron.log. The two largest files are: cloud.out.2 with following: — + keyvalues=' root=LABEL console=tty0 xencons=ttyS0,115200 console=hvc0

Re: [4.4] Cloudstack agent fails to start due to broken init script

Of course... On Mon, Mar 10, 2014 at 3:45 PM, Nux! wrote: > On 10.03.2014 21:37, Marcus wrote: >> >> Please test that fix and close the case if it works. > > > It doesn't, it needs to be: > `basename "$0"` > > > -- > Sent from the Delta quadrant using Borg technology! > > Nux! > www.nux.ro

Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

Here is the comment I just added in JIRA for this ticket. Thanks! Hi, Here is how I reproduced it: I created an iSCSI volume on my SAN that is only 2 GB. I created a XenServer SR based on this SAN volume. I created Primary Storage in CloudStack based on this XenServer SR. I created a Disk Off

Re: [4.4] Cloudstack agent fails to start due to broken init script

On 10.03.2014 21:37, Marcus wrote: Please test that fix and close the case if it works. It doesn't, it needs to be: `basename "$0"` -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro

Re: [4.4] Cloudstack agent fails to start due to broken init script

Please test that fix and close the case if it works. On Mon, Mar 10, 2014 at 3:18 PM, Nux! wrote: > Hello, > > While on 4.4 testing Marcus' root resize patches I got a hard time getting > the cloudstack-agent to work, this is due to a broken init script. > Opened https://issues.apache.org/jira/br

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Hi all, My initial intention was to get rid of the strange Map>, because most people found it counter intuitive and unnecessary (at least nobody knows why yet). But I cannot proceed with the change that I proposed any more because I cannot assume that all the maps come in the same format. I cannot

[4.4] Cloudstack agent fails to start due to broken init script

Hello, While on 4.4 testing Marcus' root resize patches I got a hard time getting the cloudstack-agent to work, this is due to a broken init script. Opened https://issues.apache.org/jira/browse/CLOUDSTACK-6220 for it, please fix. Lucian -- Sent from the Delta quadrant using Borg technology!

RE: 4.3 vote

> -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Friday, March 07, 2014 1:08 PM > To: dev@cloudstack.apache.org > Subject: RE: 4.3 vote > > > > > -Original Message- > > From: John Kinsella [mailto:j...@stratosec.co] > > Sent: Friday

Re: Review Request 18683: CLOUDSTACK-6194: Failed to increase Shared network IP Range

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18683/#review36699 --- Commit 859a78e1d321a936efd18dc6f0a55ab336cd3b70 in cloudstack's bran

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

Please see my reply/question in blue. Thanks Alex Ough On Mon, Mar 10, 2014 at 1:25 PM, Chiradeep Vittal < chiradeep.vit...@citrix.com> wrote: > I haven¹t looked at it because it is huge. But preliminary scan: > > - there are unit tests missing for changes to core CS code > Unit tests

Re: Review Request 18978: Ability to manipulate guest OS mappings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18978/ --- (Updated March 10, 2014, 8:25 p.m.) Review request for cloudstack and Nitin Meh

Re: Review Request 18978: Ability to manipulate guest OS mappings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18978/#review36694 --- api/src/org/apache/cloudstack/api/command/admin/guest/ListGuestOsMa

RE: [4.3][Cherry-pick] realhostip changes

Done > -Original Message- > From: John Kinsella [mailto:j...@stratosec.co] > Sent: Sunday, March 09, 2014 2:08 PM > To: > Subject: [4.3][Cherry-pick] realhostip changes > > Animesh - please pick the commit below from 4.3-forward into 4.3. This is for > CLOUDSTACK-6204. > > 2fe7aeea23dde

Review Request 18978: Ability to manipulate guest OS mappings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18978/ --- Review request for cloudstack and Nitin Mehta. Bugs: https://issues.apache.org/

Re: Review Request 18759: HTTP support for console proxy and making it default

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18759/ --- (Updated March 10, 2014, 7:17 p.m.) Review request for cloudstack, Demetrius Ts

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Just as an FYI: I recently added support to pass in a Min and Max IOPS for the root disk of a VM to create. I first investigated how this was being done with the custom root disk and then copied that approach. That being the case, I leveraged the map instead of making new parameters. I do agree we

Re: system vm disk space issue in ACS 4.3

There have been several raised, actually regarding /var/log. As for the system vm partitioning, it was explicitly changed from single to multiple partitions last year. I have no idea why, but I generally don't file bugs without community discussion on things that seem deliberate. On Sat, Mar 8, 2

[Merge] CloudStack IAM branch to master

Hi, Prachi and I would like to merge CloudStack Identity and Access Management(IAM) plugin services to the master branch. Development for this effort has been done by Prachi and me on ACS rbac branch (https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/rbac). Checkl

Re: system vm disk space issue in ACS 4.3

Would¹ve been nice to raise a bug about it :) On 3/8/14, 10:32 AM, "Marcus" wrote: >Yeah, I've just seen on busy systems where even with log rotation working >properly the little space left in var after OS files is barely enough, for >example the conntrackd log on a busy VPC. We actually ended u

Re: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

I haven¹t looked at it because it is huge. But preliminary scan: - there are unit tests missing for changes to core CS code - uses com.amazonaws.util.json (why?) - code format does not follow coding convention ( placement of {}, camel case api_interface ) - package namespace is com.cloud instead o

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

I suppose its also worth pointing out that the api params and how we pass them around internally are two separate things. It's easy to just add a new custom api parameter and then put it into the details map (or an object) to be passed around, and it provides a much better user experience from the

Re: DeployVM failing on master with NPE

No problem :) Sorry I didn't run that test case. On Mon, Mar 10, 2014 at 12:08 PM, Nitin Mehta wrote: > Thanks Mike for the prompt response :). > > On 10/03/14 10:44 AM, "Mike Tutkowski" > wrote: > > >1d74daf has the fix. > > > >Thanks! > > > > > >On Mon, Mar 10, 2014 at 11:42 AM, Mike Tutkowsk

Re: DeployVM failing on master with NPE

Thanks Mike for the prompt response :). On 10/03/14 10:44 AM, "Mike Tutkowski" wrote: >1d74daf has the fix. > >Thanks! > > >On Mon, Mar 10, 2014 at 11:42 AM, Mike Tutkowski < >mike.tutkow...@solidfire.com> wrote: > >> OK, I'm sorry about that - I missed a case in testing. >> >> Line 191 in Cloud

Re: [DISCUSS] realhostip.com going away

I mentioned their response on 3/3. Basically "their position is they think they’d be doing the community a disfavor by passing the torch” (quoting my previous email, not a direct quote from them but this is their position) The realhostip cert provides a false sense of security, so I can’t think

Re: DeployVM failing on master with NPE

1d74daf has the fix. Thanks! On Mon, Mar 10, 2014 at 11:42 AM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > OK, I'm sorry about that - I missed a case in testing. > > Line 191 in CloudOrchestrator should be this: > > if (offering.isCustomizedIops() != null && offering.isCustomizedIop

Re: DeployVM failing on master with NPE

OK, I'm sorry about that - I missed a case in testing. Line 191 in CloudOrchestrator should be this: if (offering.isCustomizedIops() != null && offering.isCustomizedIops()) { isCustomizedIops() returns a Boolean - not a boolean. I'll check in this fix right now. On Mon, Mar 10, 2014 at 11:32

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Yes, we'd have to leave the current one intact, but allow the other as well. On Mon, Mar 10, 2014 at 11:36 AM, Bharat Kumar wrote: > Hi, > i don't know if this is still valid, but there was one more parameter > disksize which which specifies the disk size for data disks. > I would prefer adding

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

On Mon, Mar 10, 2014 at 1:43 AM, Bharat Kumar wrote: > Hi All, > the roodiskresize is no longer valid. as there is no code which is using > rootdiskresize currently. > > As a part of the custom service offering we had to enable specifying custom > values to parameters cpu, memory and cpuCores. >

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Hi, i don’t know if this is still valid, but there was one more parameter disksize which which specifies the disk size for data disks. I would prefer adding both the root disk size and disk size to the details map. ideally we should also change the name disksize to dataDisksize to remove confu

Re: DeployVM failing on master with NPE

Let me see if I can repro it. One sec On Mon, Mar 10, 2014 at 11:11 AM, Nitin Mehta wrote: > I am facing the same issue as well. > > On 10/03/14 10:09 AM, "Koushik Das" wrote: > > >This is a new install using master. So this is not about db changes but > >mostly a bug. > > > >On 10-Mar-2014, a

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

It is valid, as I've implemented it. So we need to decide if we're using 'details' or rootdisksize as an api param. That's why I'm asking. On Mon, Mar 10, 2014 at 1:43 AM, Bharat Kumar wrote: > Hi All, > the roodiskresize is no longer valid. as there is no code which is using > rootdiskresize cu

Re: Just checked in a minor change to the DB

Marcus - I think the thread has just come up where devs are supposed to publicize the db changes. As per me this shouldn't be left to the good intentions of devs, instead it should be automated as some of the folks have suggested. As a general rule of thumb I always expect master to have new db cha

Re: DeployVM failing on master with NPE

I am facing the same issue as well. On 10/03/14 10:09 AM, "Koushik Das" wrote: >This is a new install using master. So this is not about db changes but >mostly a bug. > >On 10-Mar-2014, at 9:58 PM, Marcus wrote: > >> Mike has sent out several emails about the DB changes he made. >> >> On Mon,

Re: DeployVM failing on master with NPE

This is a new install using master. So this is not about db changes but mostly a bug. On 10-Mar-2014, at 9:58 PM, Marcus wrote: > Mike has sent out several emails about the DB changes he made. > > On Mon, Mar 10, 2014 at 5:28 AM, Koushik Das wrote: >> I am seeing the following exception >> >

RE: cherry-pick to 4.3 request

Done From: Alena Prokharchyk Sent: Friday, March 07, 2014 6:46 PM To: Animesh Chaturvedi; dev@cloudstack.apache.org Subject: cherry-pick to 4.3 request Animesh, could you please cherry-pick one more commit for VPC, from forward to 4.3 branch? 164ea3e84f6f282006e66725f22cd2246f0be8f8 CloudStack

Re: Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/#review36661 --- quite frankly I hate to see a binary file committed to the repo. sta

Re: Just checked in a minor change to the DB

Was actually bitten by another db change today that I hadn't pulled in yet: commit 830328b63da3c720712263d792f0393ea547466d Author: Nitin Mehta Date: Wed Mar 5 16:40:44 2014 -0800 CLOUDSTACK-6199: Hide action events for Vm/Volume commands when the resources have display flag=0. Introd

[CS4.1] Error during ip range creation

Hi, I creates a network using API. Then, I try to creates a new IP Range associated to this network. I've got an error. This is the output logs in this paste [1] Searching into the Cloudstack code, I find that : if (vlanId.equals(vlan.getVlanTag()) && newVlanSubnet.equals(otherVlanSubnet)) {

Re: DeployVM failing on master with NPE

Mike has sent out several emails about the DB changes he made. On Mon, Mar 10, 2014 at 5:28 AM, Koushik Das wrote: > I am seeing the following exception > > ERROR [c.c.a.ApiServer] (38419149@qtp-1917357200-11:ctx-116b8207 > ctx-9a54f764) unhandled exception executing api command: > [Ljava.lang.

Review Request 18971: CLOUDSTACK-6216: Skip the test if run on already existing setup

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18971/ --- Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri.

Review Request 18964: Windowsfication of CloudStack Management Server - Changes to support windows OS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/ --- Review request for cloudstack and Abhinandan Prateek. Bugs: https://issues.apac

Review Request 18970: CLOUDSTACK-6215: Updating test case to accomodate condition for VMware

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18970/ --- Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri.

Re: CLOUDSTACK-5583: vmopsSnapshot plug-in (XenServer) does not return an error when it should

I did look at it, but haven't had a chance to try to repo. I should be able to try to repo it today. Thanks! On Sun, Mar 9, 2014 at 10:05 PM, Mandar Barve wrote: > Hi Mike, > Did you get a chance to look at this? > > Thanks, > Mandar > > > On Wed, Mar 5, 2014 at 10:12 AM, Mandar Barve wro

[DISCUSS] Enabling databse upgrades on master branch

Hi all, At Schuberg Philis we are interested in upgrading our running installation of ACS more frequently than the current release cycle. To that end, we are working on tooling to detect potentially conflict introducing changes to the ACS database and upgrade software. By conflict introducing ch

Re: [DISCUSS] realhostip.com going away

Please let us know, what was the Citrixes response to community run realhostip.com service.

RE: OVS plugin in CloudStack 4.3.0

Great, many thanks guys for looking after this, I will give it a try these days. Kind Regards, Florin From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] Sent: Wednesday, March 05, 2014 2:58 PM To: Murali Reddy; Nguyen Anh Tu; dev@cloudstack.apache.org; Florin Dumitrascu Subject: RE:

Re: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

> On Feb. 28, 2014, 10:06 a.m., Koushik Das wrote: > > core/src/com/cloud/agent/api/StopCommand.java, line 27 > > > > > > Shouldn't there be a similar change in StartCommand as well? For VM deployment, the GPU details

Jenkins build is back to normal : build-master-slowbuild #400

See

RE: Sample of templatetag field from RegisterTemplate?

>From the api command template tag will be read from json "templatetag=" its >string value. Here is the cmd for updating a template key/value http://10.102.192.186:8080/client/api?command=createTags&response=json&sessionkey=8%2BkoU3B23zkrfZtcZlrZ1noRuHc%3D&tags%5B0%5D.key=mykey&tags%5B0%5D.val

DeployVM failing on master with NPE

I am seeing the following exception ERROR [c.c.a.ApiServer] (38419149@qtp-1917357200-11:ctx-116b8207 ctx-9a54f764) unhandled exception executing api command: [Ljava.lang.String;@36d30b1d java.lang.NullPointerException at org.apache.cloudstack.engine.orchestration.CloudOrchestrator.creat

Re: master issue calling volume resize via libvirt - Was: libvirt-java upgrade

On 03/09/2014 01:19 AM, Marcus wrote: I imagine the new LTS will have it, but I'm not sure what our OS support policy is. Well, I think we should also keep supporting 12.04 since it's not EOL until 2017. But we can always say that we require the Ubuntu Cloud Archive to be used? wido On

Sample of templatetag field from RegisterTemplate?

Does anyone have a sample of the templatetage field of RegisterTemplate? (see https://cloudstack.apache.org/docs/api/apidocs-4.2/user/registerTemplate.html) This string field should hold a key/value, but it's unclear whether the format is JSON or some other format. DL

[DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

Hi everyody, The people at sungard have been making this change for 4.4 and I want to merge/apply it this week. It is more then a screenfull and might cause issue is a setup or two. have a look at https://reviews.apache.org/r/17790/ a ref to ticket and fs page are in the review request. kind re

Review Request 18956: CLOUDSTACK-6147: Adding next set of tests for Dynamic Compute Offering

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18956/ --- Review request for cloudstack, Santhosh Edukulla and SrikanteswaraRao Talluri.

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

See

Build failed in Jenkins: build-master #437

See Changes: [Daan Hoogland] findbugs: String for Service type mismatch (plus test for dummy data) -- [...truncated 1191 lines...] [INFO] --- maven-resources-plugin:2.5:testResources (default-

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Hi Antonio, I am working on the below feature for 4.4 as part it adding an extra MAP param vmidipmap to assignToLoadBalancerRule API. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configuring+load+balancing+rules+for+VM+nic+secondary+ips I am following the existing way for map access a

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

See Changes: [talluri] CLOUDSTACK-5626: Simplifying VM Migrate code [talluri] CLOUDSTACK-6147: Adding first set of test caess for Dynamic Compute Offering feature -- [...truncated 18

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

Hi All, the roodiskresize is no longer valid. as there is no code which is using rootdiskresize currently. As a part of the custom service offering we had to enable specifying custom values to parameters cpu, memory and cpuCores. instead of adding a parameter for each of these values we changed