Re: Review Request 14147: marvin tests for Persistent networks without running a VM

2013-10-17 Thread Likitha Shetty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14147/#review27177 --- Please update the diff to remove the change made to base.py as it ha

Re: Wiki access

2013-10-17 Thread Harikrishna Patnala
Hi, Please grant me edit access. My username: harikrishna.patn...@citrix.com Thanks Harikrishna On 17-Oct-2013, at 8:23 PM, Hugo Trippaers wrote: > Added edit permissions for Harm Boertien (CCC EU Organizer) > > Cheers, > > Hugo > > > On 16 okt. 2013, at 16:28, Hugo Trippaers wrote: > >>

Re: Resource Configuration Failure in Simulator mode (v4.2)

2013-10-17 Thread Amogh Vasekar
Can you check the main pom.xml to ensure "tools" is not commented out as a module from developer profile? This should be near the end of pom.xml Thanks, Amogh On 10/17/13 9:34 AM, "Alex Ough" wrote: >All, > >After getting the latest codes of v4.2, I'm trying to configure the >resources using 's

how to use hashes on c.a.o?

2013-10-17 Thread Darren Shepherd
The hashes that are on c.a.o for the releases have a format like http://www.apache.org/dist/cloudstack/releases/4.2.0/apache-cloudstack-4.2.0-src.tar.bz2.sha apache-cloudstack-4.2.0-src.tar.bz2: CC487DF3 7E7B6800 F9DC05A3 5B72DEFD 684E0094 F1666F57 5D694916 CF

Re: Review Request 14469: CLOUDSTACK-4637: Fix failures in test_egress_fw_rules.py

2013-10-17 Thread sangeetha hariharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14469/#review27163 --- Ship it! Ship It! - sangeetha hariharan On Oct. 16, 2013, 2:27 p

Re: Review Request 14715: [CLOUDSTACK-4887] CLVM broken

2013-10-17 Thread edison su
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14715/#review27164 --- Ship it! Ship It! - edison su On Oct. 17, 2013, 6:59 p.m., Chris

Re: Review Request 14469: CLOUDSTACK-4637: Fix failures in test_egress_fw_rules.py

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14469/#review27160 --- Commit 833229c416cfee3e3dec3762d2e9c8e734696f90 in branch refs/heads

Review Request 14729: Fix UCS HTTPS redirect

2013-10-17 Thread Amogh Vasekar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14729/ --- Review request for cloudstack and Frank Zhang. Bugs: https://issues.apache.org/

Re: Review Request 14469: CLOUDSTACK-4637: Fix failures in test_egress_fw_rules.py

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14469/#review27158 --- Commit a65f1ebefca7b22512762faf1832291153782f58 in branch refs/heads

Re: Review Request 14715: [CLOUDSTACK-4887] CLVM broken

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14715/#review27157 --- Commit a504c004bf10555e5ea67ec89fe7bf6f00fe4622 in branch refs/heads

RE: CLVM broken on master

2013-10-17 Thread Edison Su
Marcus, would you mind to check in your fix into master? Thanks. > -Original Message- > From: SuichII, Christopher [mailto:chris.su...@netapp.com] > Sent: Thursday, October 17, 2013 4:35 PM > To: > Subject: Re: CLVM broken on master > > I've done some testing on our plugin and that appea

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
I've done some testing on our plugin and that appears to work fine for me. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 17, 2013, at 6:05 PM, Marcus Sorensen wrote: > The following seems to fix the issue,

regions

2013-10-17 Thread Marcus Sorensen
I'm looking for more information on use cases for regions. I've been through: https://cwiki.apache.org/confluence/display/CLOUDSTACK/AWS-Style+Regions+Functional+Spec And I've set up two management servers as separate regions. From what I can tell, I'm not sure why I'd want to use it. 1) account

Re: [MERGE] txn-refactor

2013-10-17 Thread Darren Shepherd
On Thu, Oct 17, 2013 at 12:59 PM, Hugo Trippaers wrote: > Maybe just mark start() as deprecated then. Would at least put a marker for > anybody writing new code that they should think again about using it. Good idea. > > A unit test would be really nice to have for this piece of code. Especiall

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
The following seems to fix the issue, by the way, but again since I didn't initially change the code I'd like someone else to review/handle it. diff --git a/core/src/com/cloud/storage/resource/StorageSubsystemCommandHandlerBase.java b/core/src/com/cloud/storage/resource/StorageSubsystemCommandHan

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
All of the above mentioned is in core/src/com/cloud/storage/resource/StorageSubsystemCommandHandlerBase.java , by the way. On Thu, Oct 17, 2013 at 4:02 PM, Marcus Sorensen wrote: > Sure, but CopyCommand is being triggered in this code. I've tested > several variations to this one line, some work,

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
Sure, but CopyCommand is being triggered in this code. I've tested several variations to this one line, some work, some don't. On Thu, Oct 17, 2013 at 3:38 PM, Edison Su wrote: > For CLVM, the copy template from secondary to primary and create volume from > template logic is handled by CloudStac

RE: CLVM broken on master

2013-10-17 Thread Edison Su
For CLVM, the copy template from secondary to primary and create volume from template logic is handled by CloudStackPrimaryDataStoreDriverImpl->copyAsync, not in AncientDataMotionStrategy You can check the code: 4fb459355337c874a10f47c0224af72d6fef1ff2. > -Original Message- > From: Marcu

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
I think if we can change this line: if ((srcData.getObjectType() == DataObjectType.TEMPLATE) && (destData.getObjectType() == DataObjectType.TEMPLATE && destData.getDataStore().getRole() == DataStoreRole.Primary)) { to something like: if (srcData.getObjectType() == DataObjectType.TEMPLATE && srcD

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
I ran that through my tester, it didn't like it. That actually kept the system vms from starting. Since CopyCommand is used for both template to template and template to primary, it seems that the original template copy is fine but now this catches the case where the source template is on primary

RE: Cloudstack collab Hackathons

2013-10-17 Thread Donal Lafferty
An example of how to avoid Java can be found in the Hyper-V agent. Hyper-V experts tend to know C# better than Java, so hypervisor control is through an agent on the hypervisor written in C#. To avoid porting the message bus, communications are over HTTP Request. To avoid porting the command

RE: Incomplete API docs?

2013-10-17 Thread Demetrius Tsitrelis
Do you still think there needs to be a bug filed for the missing APIs? -Original Message- From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] Sent: Thursday, October 17, 2013 1:21 PM To: dev@cloudstack.apache.org Subject: RE: Incomplete API docs? +1 Radhika on the 4.3 relea

Re: Issues deploying with marvin

2013-10-17 Thread Will Stevens
Note: I pulled from master yesterday... On Thu, Oct 17, 2013 at 4:28 PM, Will Stevens wrote: > When I run: > > I am getting the following error: > [DEBUG] Executing command line: python > ../marvin/marvin/deployDataCenter.py -i devcloud.cfg > 5Traceback (most recent call last): > File "../mar

CLOUDSTACK-4828 remove nic fails if dhcp wasn't enabled in network offering

2013-10-17 Thread Patrick Miller
Hi all, Is CLOUDSTACK-4828 slated for the 4.2.1 update. We currently do not use the Virtual Routers, Our ACS world share a common network infrastructure with the rest of lab servers. Having to run a clean up sql job each night is not elegant. Thanks for all the great work. Patrick

Issues deploying with marvin

2013-10-17 Thread Will Stevens
When I run: I am getting the following error: [DEBUG] Executing command line: python ../marvin/marvin/deployDataCenter.py -i devcloud.cfg 5Traceback (most recent call last): File "../marvin/marvin/deployDataCenter.py", line 612, in deploy.deploy() File "../marvin/marvin/deployDataCenter.p

RE: Incomplete API docs?

2013-10-17 Thread Animesh Chaturvedi
+1 Radhika on the 4.3 release page I have solicited volunteers for different release activities. Looks like the high level tasks could use a better breakdown, do you want to take a stab at breaking doc into sub tasks and folks interested can fill in their name. Animesh > -Original Messag

Re: Cloudstack collab Hackathons

2013-10-17 Thread Hugo Trippaers
That's a shame, but I could check if I can setup some teleconference gear. If you are interested. Though the time might be a serious bother depending on your timezone. Cheers, Hugo Sent from my iPhone > On 17 okt. 2013, at 17:39, Marcus Sorensen wrote: > > I won't be this time. > > On Thu

Re: [MERGE] txn-refactor

2013-10-17 Thread Hugo Trippaers
Sent from my iPhone > On 17 okt. 2013, at 20:12, Darren Shepherd > wrote: > >> On 10/17/2013 01:53 AM, Hugo Trippaers wrote: >> Hey Darren, >> >> Looking through the code it looks like this more an API change than an >> actual redesign of the transaction code? I like the resulting code a lo

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
Hm, interesting. Since nothing else in the if/else if series there depends on the src being a template, I'd imagine it would be safe to just have the check be: } else if (srcData.getObjectType() == DataObjectType.TEMPLATE && destDataStore.getRole() == DataStoreRole.Primary) { In hindsight, add

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
Actually, I don't think that will fix this (though it probably fixes something :-) The issue I'm having is that we went from 'if source is a template on nfs and destination is primary storage' to 'if source is a template and destination is a template on primary storage'. We aren't copying 'templat

Re: CLVM broken on master

2013-10-17 Thread Marcus Sorensen
Ok, let me test it. On Thu, Oct 17, 2013 at 12:56 PM, SuichII, Christopher wrote: > Oh, I noticed this and created a fix, which I thought I already had submitted > since it was a part of the storage refactoring a couple weeks back. I'll post > the patch for review now. > > -- > Chris Suich > ch

RE: Possible bug in DeploymentPlanner?

2013-10-17 Thread Prachi Damle
-Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Thursday, October 17, 2013 11:50 AM To: Subject: Re: Possible bug in DeploymentPlanner? On 17-Oct-2013, at 11:29 PM, Prachi Damle wrote: > Koushik, > > I don't think you should add this check before calling

Review Request 14715: [CLOUDSTACK-4887] CLVM broken

2013-10-17 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14715/ --- Review request for cloudstack, edison su and Marcus Sorensen. Repository: cloud

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
Oh, I noticed this and created a fix, which I thought I already had submitted since it was a part of the storage refactoring a couple weeks back. I'll post the patch for review now. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Ci

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-17 Thread Chris Suich
> On Oct. 17, 2013, 3:18 p.m., John Burwell wrote: > > engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/DataMotionServiceImpl.java, > > line 62 > > > > > > Add context information to error message to

CLVM broken on master

2013-10-17 Thread Marcus Sorensen
Just posting this to dev for visibility. I think commit 180cfa19 broke CLVM primary storage for KVM. I'm failing VM deploy from template. I've been building a 'sanity check' test that focuses on the KVM specific suff (tests storage types and supported host OS for now), and this bubbled up. Read m

Re: Possible bug in DeploymentPlanner?

2013-10-17 Thread Koushik Das
On 17-Oct-2013, at 11:29 PM, Prachi Damle wrote: > Koushik, > > I don't think you should add this check before calling allocators. It is the > allocators output that should tell that local storage pools were not found. > For your case where you need to put cluster in avoid set when no local

Re: Snapshot Strategy Review

2013-10-17 Thread SuichII, Christopher
My apologies. Everything in the private email was also posted to the dev list, I just wanted to send a personal email to make sure you saw the update in your inbox. -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-17 Thread Chris Suich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14522/ --- (Updated Oct. 17, 2013, 6:46 p.m.) Review request for cloudstack, Brian Federle

Re: 4.2.1 package issues

2013-10-17 Thread Darren Shepherd
On 10/17/2013 11:06 AM, Bjoern Teipel wrote: Compilation failure: [ERROR] error: error reading /home/bteipel/.m2/repository/org/apache/axis2/mex/1.5.4/mex-1.5.4-impl.jar; error in opening zip file [ERROR] error: error reading /home/bteipel/.m2/repository/org/apache/axis2/axis2-mtompolicy/1.5.4/ax

Re: [MERGE] txn-refactor

2013-10-17 Thread Darren Shepherd
On 10/17/2013 01:53 AM, Hugo Trippaers wrote: Hey Darren, Looking through the code it looks like this more an API change than an actual redesign of the transaction code? I like the resulting code a lot better than the existing way of doing it. As far as i can see you wrapped the existing Tran

4.2.1 package issues

2013-10-17 Thread Bjoern Teipel
Hi all, did somebody encounter issues during RPM packaging ? I got jar file read errors and I can't tell where they are coming from. It seems maven tries to download those jars during the compile from an invalid url, because those just contain a webpage Internet2 Shibboleth Project has moved

Fwd: [DISCUSS] Domain/Account/User Sync Up Among Multiple Regions

2013-10-17 Thread Alex Ough
All, Currently, under the environment of cloudstack with multiple regions, each region has its own management server running with a separate database. So if we want to support multiple regions and provide one point of entry for a customer, we need to duplicate domain/account/user information of th

RE: Possible bug in DeploymentPlanner?

2013-10-17 Thread Prachi Damle
Koushik, I don't think you should add this check before calling allocators. It is the allocators output that should tell that local storage pools were not found. For your case where you need to put cluster in avoid set when no local pool is present: a) Only local storage allocator should get in

Re: Review Request 14334: CLOUDSTACK 4705: Fixed domain memory limits test cases

2013-10-17 Thread Daan Hoogland
Gaurav, I will but it doesn't apply to master. Can you update it? regards, On Wed, Oct 16, 2013 at 6:43 PM, Girish Shilamkar wrote: > > >> On Oct. 8, 2013, 11:14 a.m., abhinav roy wrote: >> > Ship It! > > Can someone commit this patch. > > > - Girish > > > ---

Re: Snapshot Strategy Review

2013-10-17 Thread John Burwell
Chris, I have updated my comments in https://reviews.apache.org/r/14522/. I will likely be offline the remainder of the day due to two sick kids + my own sickness. Thanks, -John P.S. I just noticed that this email thread was private to Edison and I. All dev discussions should occur on th

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-17 Thread John Burwell
> On Oct. 17, 2013, 11:18 a.m., John Burwell wrote: > > ui/scripts/storage.js, line 1963 > > > > > > How will a corrupted snapshot failure be surfaced to the user? > > Chris Suich wrote: > It depends on what you

Re: internal LB provider (from the thread: [VOTE] Apache CloudStack 4.2.0 (third round))

2013-10-17 Thread Alena Prokharchyk
I do see the UUID being returned as a part of listNetworkServiceProvidersCmd call ("id" field): { "listnetworkserviceprovidersresponse" : { "count":8 ,"networkserviceprovider" : [ {"name":"InternalLbVm","physicalnetworkid":"235cd520-b73c-458b-aa69-14df2f4fe871","state":"Disabled","id":"83011e

Re: internal LB provider (from the thread: [VOTE] Apache CloudStack 4.2.0 (third round))

2013-10-17 Thread Daan Hoogland
On Thu, Oct 17, 2013 at 7:22 PM, Alena Prokharchyk wrote: > updateNetworkServiceProvider ah, so this works even if the uuid is not returned in listNetworkServiceProvidersCmd?

Re: internal LB provider (from the thread: [VOTE] Apache CloudStack 4.2.0 (third round))

2013-10-17 Thread Alena Prokharchyk
Daan, the API is "updateNetworkServiceProvider" -Alena. From: Daan Hoogland mailto:daan.hoogl...@gmail.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Thursday, October 17, 2013 10:17 AM To: dev mailto:dev@cloudstack.apache.o

Re: internal LB provider (from the thread: [VOTE] Apache CloudStack 4.2.0 (third round))

2013-10-17 Thread Daan Hoogland
Thanks Marcus, this helps, I can now enable it in the interface after configuring it as enabled. Somehow I still have to enable the InternalLbVm provider in the physical network after configuring it as enabled and can't find a api call to do this. On Thu, Oct 17, 2013 at 4:21 PM, Marcus Sorensen

Re: [GSoC] Congratulations and Wrap up

2013-10-17 Thread Wei ZHOU
Thanks all! 2013/10/17 kel...@backbonetechnology.com > Thank you to all the students and mentors for bringing new and interesting > features into cloudstack. With all the hard work and dedication, we ended > up with some great additions. > > See you in Amsterdam. > > Sent from my HTC > > -

Re: [Discuss] Donation of RDP client code

2013-10-17 Thread Wei ZHOU
It looks very nice. Thanks! Hugo are working on the refactor of systemvm. Maybe it need to be modified. 2013/10/17 Donal Lafferty > This donation addresses our need for an RDP client free of IP restrictions > that can be included with Apache CloudStack's console VM. > > > > RDP is of interest,

Re: [GSoC] Congratulations and Wrap up

2013-10-17 Thread kel...@backbonetechnology.com
Thank you to all the students and mentors for bringing new and interesting features into cloudstack. With all the hard work and dedication, we ended up with some great additions. See you in Amsterdam. Sent from my HTC - Reply message - From: "sebgoa" To: "dev@cloudstack.apache.org" S

Re: [GSoC] Congratulations and Wrap up

2013-10-17 Thread Ian Duffy
Thank you Sebastien and Abhi along with the Cloudstack Community for making this opportunity possible. It was a very interesting and educational experience. I hope to keep up work with you guys. I will be a bit quieter on here due to college work over the next few months. Looking forward to meeti

Re: [GSoC] Congratulations and Wrap up

2013-10-17 Thread Abhinandan Prateek
I would like to thank Ian who did a fabulous job on ldap plugin and also Sebastian who was always around to guide the students and mentors. It would be good to see them at Amsterdam as I am also going to be around. -abhi On 17/10/13 7:49 pm, "sebgoa" wrote: >Hi, > >This week end is the Google

[Discuss] Donation of RDP client code

2013-10-17 Thread Donal Lafferty
This donation addresses our need for an RDP client free of IP restrictions that can be included with Apache CloudStack's console VM. RDP is of interest, because Hyper-V exposes VM consoles over RDP. However, Apache CloudStack does not have an RDP client that is easy to build, can easily run

Resource Configuration Failure in Simulator mode (v4.2)

2013-10-17 Thread Alex Ough
All, After getting the latest codes of v4.2, I'm trying to configure the resources using 'setup/dev/advanced.cfg', but it seems to be failing with this error. "Could not find the selected project in the reactor: :cloud-marvin" I'm using this command and it was working ok before getting the latest

Re: Possible bug in DeploymentPlanner?

2013-10-17 Thread Koushik Das
Prachi, I am planning to add the following code in findSuitablePoolsForVolumes() method. If the VM has a local disk and the cluster doesn't have a local pool then the cluster will be added to avoid set. Let me know if you see any issues with this. if (useLocalStorage) {

RE: Cloudstack collab Hackathons

2013-10-17 Thread Santhosh Edukulla
Marcus, 1/ For me, its not about the debate of Java VS python, or supporting anything to re-write existing already. Its an effort, learning,idea and thought of contribution being new. Having said that, we worked earlier on building an agent prototype where we had to deploy agents and push po

Re: Cloudstack collab Hackathons

2013-10-17 Thread Darren Shepherd
Marcus, Okay, well I'll give a fully articulated argument for this eventually. Regardless, no reason to discourage a hackathon on the topic. Might as well talk and hack on it a bit and then we will know if it's a terrible idea. Darren > On Oct 17, 2013, at 8:39 AM, Marcus Sorensen wrote: >

Re: Cloudstack collab Hackathons

2013-10-17 Thread sebgoa
On Oct 17, 2013, at 5:19 PM, Marcus Sorensen wrote: > I'm not at all against it, I just haven't heard anyone give any reason > as to why. Keep in mind that we'd essentially be asking anyone who > has written code for the agent to re-do their work (midonet, vxlan, > storage plugins). My impress

Re: templates being deleted after download

2013-10-17 Thread Darren Shepherd
I haven't looked at this at all, but could it be related to an issue I brought up before that commands go to the "LocalEndpoint" if the SSVM is unavailable. I've seen this too where before the ssvm starts the management stack tries to do the download instead of waiting for the ssvm to become av

Re: Cloudstack collab Hackathons

2013-10-17 Thread Marcus Sorensen
I won't be this time. On Thu, Oct 17, 2013 at 9:35 AM, Darren Shepherd wrote: > Marcus, > > Will you be at CCC? I think it's immensely important and can give plenty of > good reasons, but too lazy to type at the moment. And I need to do a bit > more analysis on the scope. > > Darren > >> On O

Re: Cloudstack collab Hackathons

2013-10-17 Thread Koushik Das
Agree with Marcus here. If we are planning to rewrite, I think that there should be some discussion around the possibility of making it a direct agent like XS/Vmware. If this is possible then installing the KVM agent on individual hosts will be eliminated. On 17-Oct-2013, at 8:49 PM, Marcus Sor

Re: Cloudstack collab Hackathons

2013-10-17 Thread Darren Shepherd
I'd like to throw in the topic of API clients. So specifically what's the state of clients in python, ruby, java, and go. With the api discovery feature it should be simple to write dynamic clients that are always up to date with the api. Darren > On Oct 14, 2013, at 7:22 AM, Sebastien Goasg

Re: [JENKINS] systemvm build jobs

2013-10-17 Thread Hugo Trippaers
Heya, The systemvm builds are working again on master. The created templates can be downloaded from: http://jenkins.buildacloud.org/job/build-systemvm-master/ http://jenkins.buildacloud.org/job/build-systemvm64-master/ Cheers, Hugo On 17 okt. 2013, at 12:30, Hugo Trippaers wrote: > Heya, >

Re: Cloudstack collab Hackathons

2013-10-17 Thread Darren Shepherd
Marcus, Will you be at CCC? I think it's immensely important and can give plenty of good reasons, but too lazy to type at the moment. And I need to do a bit more analysis on the scope. Darren > On Oct 17, 2013, at 8:19 AM, Marcus Sorensen wrote: > > I'm not at all against it, I just haven'

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-17 Thread Chris Suich
> On Oct. 17, 2013, 3:18 p.m., John Burwell wrote: > > engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/DataMotionServiceImpl.java, > > line 62 > > > > > > Add context information to error message to

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-17 Thread John Burwell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14522/#review27127 --- Overall, this patch is very close to ready. I really like the way t

Re: Cloudstack collab Hackathons

2013-10-17 Thread Marcus Sorensen
I'm not at all against it, I just haven't heard anyone give any reason as to why. Keep in mind that we'd essentially be asking anyone who has written code for the agent to re-do their work (midonet, vxlan, storage plugins). My impression is that people just like the 'feel' of it being in Python,

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-17 Thread John Burwell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14522/#review27126 --- engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/

templates being deleted after download

2013-10-17 Thread Marcus Sorensen
In writing some full buildout tests I'm running into some intermittent issues. If I register a template after zone is created, but before secondary storage vm starts, the template is pretty much guaranteed to be deleted. If I watch for the SSVM to come up, and its agent to start, then there's may

Re: Wiki access

2013-10-17 Thread Hugo Trippaers
Added edit permissions for Harm Boertien (CCC EU Organizer) Cheers, Hugo On 16 okt. 2013, at 16:28, Hugo Trippaers wrote: > Kishan, > > look like you have the permissions already on account kishan. Can you check? > > Cheers, > > Hugo > > > On Oct 16, 2013, at 3:45 PM, Kishan Kavala wrot

Re: internal LB provider (from the thread: [VOTE] Apache CloudStack 4.2.0 (third round))

2013-10-17 Thread Marcus Sorensen
Yes, it should supposedly never happen under normal circumstances. The issue I had was because I was using a Marvin zone config that needed to be updated for the new feature. Added: { "broadcastdomainrange": "ZONE",

[GSoC] Congratulations and Wrap up

2013-10-17 Thread sebgoa
Hi, This week end is the Google Summer of Code summit. This year we had the chance to get five students in the program: Dharmesh, Meng, Ian, Nguyen and Shiva and I'd be remiss not to acknowledge their hard work. I would like to congratulate all of them on working hard this past summer (program

internal LB provider (from the thread: [VOTE] Apache CloudStack 4.2.0 (third round))

2013-10-17 Thread Daan Hoogland
Marcus. I ran into this on today's master. Did you follow up on this in 4.2? Or do you have a ticket or wiki page for it? tahnks Daan On Thu, Aug 29, 2013 at 1:57 AM, Marcus Sorensen wrote: > 2. can't figure out how to deploy VPC. In 4.2 it tells me to jump > through a few extra hoops regarding

Re: Review Request 14591: Fixng the issue of wrong netmask value is allowed in public traffic wizard while creating a zone

2013-10-17 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14591/#review27122 --- utils/src/com/cloud/utils/net/NetUtils.java

hackathons so far

2013-10-17 Thread Daan Hoogland
Users and developers, So far the list of hackathons on CCC Europe has taken a serious shape. Don't forget to check and make sure your personal or company interest is represented at https://cwiki.apache.org/confluence/display/CLOUDSTACK/CCC+Europe: h2. hackathon proposals || chair || subject || d

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-17 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14459/#review27067 --- test/integration/component/test_base_image_updation.py

Re: Review Request 14531: CLOUDSTACK-702: Verify Userdata and Password service on alias ip on VR

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14531/#review27119 --- Commit 8b0894b27c45c3e89824ece3a6c47c8143d3ae76 in branch refs/heads

Re: Review Request 14531: CLOUDSTACK-702: Verify Userdata and Password service on alias ip on VR

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14531/#review27118 --- Commit 7a7fb61a17911987838a2b6744d594c840df7c2e in branch refs/heads

Re: Review Request 14556: CLOUDSTACK-4766: Add timeout if vm does not reach running state

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14556/#review27117 --- Commit 9cc557509265d8faacca134e8bfeb3a31ddbeb31 in branch refs/heads

Re: Review Request 14556: CLOUDSTACK-4766: Add timeout if vm does not reach running state

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14556/#review27116 --- Commit e3bcdc16a11d7452b5bf6ce5e5993dcd008526a6 in branch refs/heads

[JENKINS] systemvm build jobs

2013-10-17 Thread Hugo Trippaers
Heya, The systemvm jobs on master are failing because the netinst iso for 7.0.0 is no longer available from the debian mirrors. I've added the images to the userContent on the master so we can use the copy-to-slave plugin to get them. I'm updating and testing the jobs now. Cheers, Hugo

Re: Review Request 14557: CLOUDSTACK-4747: Rename testcase name to use lesser characters

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14557/#review27115 --- Commit 08e69f130f142b0b034b6f42715ae0d1ca390c12 in branch refs/heads

Re: Review Request 14557: CLOUDSTACK-4747: Rename testcase name to use lesser characters

2013-10-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14557/#review27114 --- Commit c5e1c4725c3658cae967c63cbae0ffe598f227ef in branch refs/heads

Re: Review Request 14556: CLOUDSTACK-4766: Add timeout if vm does not reach running state

2013-10-17 Thread sanjeev n
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14556/#review27113 --- Ship it! Patch is good to go. - sanjeev n On Oct. 9, 2013, 9:50

Snapshot Create on 4.2

2013-10-17 Thread Antonio Petrocelli
Hello, after upgrade to 4.2 snapshot creation is stuck on *Ready *status. before status was *BackedUp. *Is there a parameter to set? Thanks! * *

Re: [MERGE] txn-refactor

2013-10-17 Thread Hugo Trippaers
Hey Darren, Looking through the code it looks like this more an API change than an actual redesign of the transaction code? I like the resulting code a lot better than the existing way of doing it. As far as i can see you wrapped the existing TransactionLegacy way of doing it (txn.start / txn.c

Re: [MERGE] txn-refactor

2013-10-17 Thread Sebastien Goasguen
On Oct 16, 2013, at 9:08 PM, Darren Shepherd wrote: > How do these merge emails work? Do I just wait and certain amount of time > and if nobody says no I just do it? I've tested the branch as much as I'd > like, so just waiting for input. > I think there is a 72 hours window, but we mig

Re: [MERGE] txn-refactor

2013-10-17 Thread Hugo Trippaers
Hey Darren, I'm having a look at the branch. Takes some time so i will get back to you when i have something. Did you run the bvt test suite on this branch already? Cheers, Hugo On Oct 16, 2013, at 6:59 PM, Darren Shepherd wrote: > I need as many people as possible to review this branch.

Re: Review Request 14628: AutoScaling without NetScaler

2013-10-17 Thread Nguyen Anh Tu
2013/10/17 Sowmya Krishnan > Ok. Thanks. So will live storage migration of VM (using Xen motion) be > supported with this as well? Can I for ex: migrate one of the autoscaled > VMs across clusters? > > If yes, does it recalculate the scale up condition again after migrating > the VM? And wond