RE: Just checked in a minor change to the DB

2014-03-10 Thread Rajesh Battala
I prefer server side hook because ppl use mail clients in mobiles/web etc and applying client side rules is not effective. And moreover we need to keep exporting this client side rule and associated mail client support is required for the same. Best option is to have a server side hook .

Re: Review Request 16385: Fix for CloudStack JIRA 4406

2014-03-10 Thread Mandar Barve
Guess in addition to the command level flag that we have Parameter walk will need to be done only for the already identified sensitive responses as discussed on the thread so this may be fine. Thanks, Mandar On Mon, Mar 10, 2014 at 9:34 AM, Mandar Barve mandar.ba...@sungard.comwrote: We

Regarding CLOUDSTACK-5825

2014-03-10 Thread Namita Chaudhari
Hi Chris, I tried to create volume snapshot on 4.3 version. When the volume snapshot creation failed, the API threw an exception Failed to create the snapshot due to an internal error for volume x. I have attached the screenshot and log file for the same in the jira. Please let me know your

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

2014-03-10 Thread Bharat Kumar
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

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

2014-03-10 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-slowbuild/399/changes Changes: [talluri] CLOUDSTACK-5626: Simplifying VM Migrate code [talluri] CLOUDSTACK-6147: Adding first set of test caess for Dynamic Compute Offering feature -- [...truncated

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

2014-03-10 Thread Jayapal Reddy Uradi
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

Build failed in Jenkins: build-master #437

2014-03-10 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/437/changes Changes: [Daan Hoogland] findbugs: String for Service type mismatch (plus test for dummy data) -- [...truncated 1191 lines...] [INFO] --- maven-resources-plugin:2.5:testResources

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

2014-03-10 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master/438/changes

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

2014-03-10 Thread Ashutosh Kelkar
--- 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.

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

2014-03-10 Thread Daan Hoogland
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

Sample of templatetag field from RegisterTemplate?

2014-03-10 Thread Donal Lafferty
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

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

2014-03-10 Thread Wido den Hollander
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

DeployVM failing on master with NPE

2014-03-10 Thread Koushik Das
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

RE: Sample of templatetag field from RegisterTemplate?

2014-03-10 Thread Rajesh Battala
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

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

2014-03-10 Thread jenkins
See http://jenkins.buildacloud.org/job/build-master-slowbuild/400/changes

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

2014-03-10 Thread Sanjay Tripathi
On Feb. 28, 2014, 10:06 a.m., Koushik Das wrote: core/src/com/cloud/agent/api/StopCommand.java, line 27 https://reviews.apache.org/r/17889/diff/3/?file=505671#file505671line27 Shouldn't there be a similar change in StartCommand as well? For VM deployment, the GPU details are

RE: OVS plugin in CloudStack 4.3.0

2014-03-10 Thread Florin Dumitrascu
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: [DISCUSS] realhostip.com going away

2014-03-10 Thread France
Please let us know, what was the Citrixes response to community run realhostip.com service.

[DISCUSS] Enabling databse upgrades on master branch

2014-03-10 Thread Miguel Ferreira
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

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

2014-03-10 Thread Mike Tutkowski
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 mandar.ba...@sungard.comwrote: Hi Mike, Did you get a chance to look at this? Thanks, Mandar On Wed, Mar 5, 2014 at 10:12 AM,

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

2014-03-10 Thread Gaurav Aradhye
--- 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.

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

2014-03-10 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18964/ --- Review request for cloudstack and Abhinandan Prateek. Bugs:

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

2014-03-10 Thread Gaurav Aradhye
--- 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.

Re: DeployVM failing on master with NPE

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

[CS4.1] Error during ip range creation

2014-03-10 Thread nicolas.lamirault
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: Just checked in a minor change to the DB

2014-03-10 Thread Marcus
Was actually bitten by another db change today that I hadn't pulled in yet: commit 830328b63da3c720712263d792f0393ea547466d Author: Nitin Mehta nitin.me...@citrix.com Date: Wed Mar 5 16:40:44 2014 -0800 CLOUDSTACK-6199: Hide action events for Vm/Volume commands when the resources have

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

2014-03-10 Thread daan Hoogland
--- 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.

RE: cherry-pick to 4.3 request

2014-03-10 Thread Animesh Chaturvedi
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

Re: DeployVM failing on master with NPE

2014-03-10 Thread Koushik Das
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 shadow...@gmail.com wrote: Mike has sent out several emails about the DB changes he made. On Mon, Mar 10, 2014 at 5:28 AM, Koushik Das koushik@citrix.com wrote: I am

Re: DeployVM failing on master with NPE

2014-03-10 Thread Nitin Mehta
I am facing the same issue as well. On 10/03/14 10:09 AM, Koushik Das koushik@citrix.com 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 shadow...@gmail.com wrote: Mike has sent out several emails about the DB

Re: Just checked in a minor change to the DB

2014-03-10 Thread Nitin Mehta
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

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

2014-03-10 Thread Marcus
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 bharat.ku...@citrix.com wrote: Hi All, the roodiskresize is no longer valid. as there is no code which is

Re: DeployVM failing on master with NPE

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

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

2014-03-10 Thread Bharat Kumar
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

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

2014-03-10 Thread Marcus
On Mon, Mar 10, 2014 at 1:43 AM, Bharat Kumar bharat.ku...@citrix.com 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,

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

2014-03-10 Thread Marcus
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 bharat.ku...@citrix.com 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

Re: DeployVM failing on master with NPE

2014-03-10 Thread Mike Tutkowski
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: DeployVM failing on master with NPE

2014-03-10 Thread Mike Tutkowski
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.isCustomizedIops()) {

Re: [DISCUSS] realhostip.com going away

2014-03-10 Thread John Kinsella
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

2014-03-10 Thread Nitin Mehta
Thanks Mike for the prompt response :). On 10/03/14 10:44 AM, Mike Tutkowski mike.tutkow...@solidfire.com 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

Re: DeployVM failing on master with NPE

2014-03-10 Thread Mike Tutkowski
No problem :) Sorry I didn't run that test case. On Mon, Mar 10, 2014 at 12:08 PM, Nitin Mehta nitin.me...@citrix.comwrote: Thanks Mike for the prompt response :). On 10/03/14 10:44 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: 1d74daf has the fix. Thanks! On Mon, Mar 10,

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

2014-03-10 Thread Marcus
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: [DISCUSS]{BEHAVIORAL-CHANGE]Domain-Account-User Sync Up Among Multiple Regions

2014-03-10 Thread Chiradeep Vittal
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

Re: system vm disk space issue in ACS 4.3

2014-03-10 Thread Chiradeep Vittal
Would¹ve been nice to raise a bug about it :) On 3/8/14, 10:32 AM, Marcus shadow...@gmail.com 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

[Merge] CloudStack IAM branch to master

2014-03-10 Thread Min Chen
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).

Re: system vm disk space issue in ACS 4.3

2014-03-10 Thread Marcus
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,

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

2014-03-10 Thread Mike Tutkowski
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: Review Request 18759: HTTP support for console proxy and making it default

2014-03-10 Thread Amogh Vasekar
--- 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

Review Request 18978: Ability to manipulate guest OS mappings

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

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

2014-03-10 Thread Animesh Chaturvedi
Done -Original Message- From: John Kinsella [mailto:j...@stratosec.co] Sent: Sunday, March 09, 2014 2:08 PM To: dev@cloudstack.apache.org 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.

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

2014-03-10 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18978/#review36694 ---

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

2014-03-10 Thread Amogh Vasekar
--- 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

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

2014-03-10 Thread Alex Ough
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 for

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

2014-03-10 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18683/#review36699 --- Commit 859a78e1d321a936efd18dc6f0a55ab336cd3b70 in cloudstack's

RE: 4.3 vote

2014-03-10 Thread Animesh Chaturvedi
-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, March 07,

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

2014-03-10 Thread Nux!
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

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

2014-03-10 Thread Antonio Fornié Casarrubios
Hi all, My initial intention was to get rid of the strange MapMapString, Object, 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

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

2014-03-10 Thread Marcus
Please test that fix and close the case if it works. On Mon, Mar 10, 2014 at 3:18 PM, Nux! n...@li.nux.ro 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

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

2014-03-10 Thread Mike Tutkowski
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

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

2014-03-10 Thread Marcus
Of course... On Mon, Mar 10, 2014 at 3:45 PM, Nux! n...@li.nux.ro 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: system vm disk space issue in ACS 4.3

2014-03-10 Thread Anirban Chakraborty
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: system vm disk space issue in ACS 4.3

2014-03-10 Thread Marcus
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 abc...@juniper.net 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

Mascot

2014-03-10 Thread Mike Tutkowski
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:

Re: [Merge] CloudStack IAM branch to master

2014-03-10 Thread Chiradeep Vittal
Any new jar dependencies? On 3/10/14, 11:34 AM, Min Chen min.c...@citrix.com 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

Re: system vm disk space issue in ACS 4.3

2014-03-10 Thread Chiradeep Vittal
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 shadow...@gmail.com wrote: There have been several raised, actually regarding /var/log. As for the

Re: [Merge] CloudStack IAM branch to master

2014-03-10 Thread Min Chen
No new jar dependencies. -min Sent from my iPhone On Mar 10, 2014, at 7:22 PM, Chiradeep Vittal chiradeep.vit...@citrix.com wrote: Any new jar dependencies? On 3/10/14, 11:34 AM, Min Chen min.c...@citrix.com wrote: Hi, Prachi and I would like to merge CloudStack Identity and

[VOTE] Apache CloudStack 4.3.0 (seventh round)

2014-03-10 Thread Animesh Chaturvedi
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:

RE: Just checked in a minor change to the DB

2014-03-10 Thread Rajesh Battala
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

Re: Just checked in a minor change to the DB

2014-03-10 Thread Mike Tutkowski
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

[QUESTION] XenServer issue attaching a disk to a VM

2014-03-10 Thread Mike Tutkowski
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

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

2014-03-10 Thread Mike Tutkowski
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

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

2014-03-10 Thread Devdeep Singh
--- 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: Just checked in a minor change to the DB

2014-03-10 Thread Rajesh Battala
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