Re: Review Request: Updated the user permission to acquire ip

2013-06-17 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10727/#review19622 --- server/src/com/cloud/network/NetworkServiceImpl.java

Re: Review Request: Test Snapshot Services

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11904/#review22054 --- This test actually passes on jenkins. What are we changing here? -

Re: Review Request: Test suite test_shared_networks.py not executing since missing import attr

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11917/#review22049 --- Ship it! 76520e8 - Prasanna Santhanam On June 18, 2013, 6:07 a.m

Re: Review Request: Removed-junk-characters-from-test_netscaler_lb.py.patch

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11901/#review22048 --- Ship it! 85ed2160876826fa28088a716657f6c7bcbf84ec - Prasanna Santh

Re: Review Request: remove dead allocations

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11324/#review22047 --- Ship it! 93938ca716f85c24f2d9fb07c5856055f0061415 - Prasanna Santh

Re: Review Request: Test suite test_shared_networks.py not executing since missing import attr

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11917/ --- (Updated June 18, 2013, 6:07 a.m.) Review request for cloudstack, Prasanna Sant

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Sailaja Mada
Congrats Jayapal. Thanks, Sailaja.M -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Monday, June 17, 2013 9:00 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi The Project Management Committee (PMC) for Apache Cloud

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Isaac Chiang
Congrats! :) On Tue, Jun 18, 2013 at 1:28 PM, Nitin Mehta wrote: > Congrats Jayapal :) > > On 18/06/13 10:52 AM, "Sanjay Tripathi" > wrote: > > >Congratulations Jayapal :) > > > >--Sanjay > > > >> -Original Message- > >> From: Chip Childers [mailto:chip.child...@sungard.com] > >> Sent:

Re: Review Request: CLOUDSTACK-1047: tracking in logs using job id

2013-06-17 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11906/#review22043 --- server/src/com/cloud/async/AsyncJobManagerImpl.java

Re: Review Request: Update test case and framework to add test name in account

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11921/#review22045 --- Also - can you upload your diff as opposed to attach a file in revie

Re: Review Request: Update test case and framework to add test name in account

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11921/#review22044 --- We shouldn't depend on the test case author to include the 'testname

Re: Review Request: Test suite test_shared_networks.py not executing since missing import attr

2013-06-17 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11917/#review22042 --- You forgot the diff (again) - Prasanna Santhanam On June 17, 2013

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Prasanna Santhanam
> On June 17, 2013, 3:12 p.m., Prasanna Santhanam wrote: > > Since XCP shares the same resource (XcpOssResource<-CitrixResourceBase) can > > the command ScaleVmCommand be implemented for XCP too? > > Any comments about adding the XCP support? > On June 17, 2013, 3:12 p.m., Prasanna Santhanam

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Nitin Mehta
Congrats Jayapal :) On 18/06/13 10:52 AM, "Sanjay Tripathi" wrote: >Congratulations Jayapal :) > >--Sanjay > >> -Original Message- >> From: Chip Childers [mailto:chip.child...@sungard.com] >> Sent: Monday, June 17, 2013 9:00 PM >> To: dev@cloudstack.apache.org >> Subject: [ANNOUNCE] New

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Sanjay Tripathi
Congratulations Jayapal :) --Sanjay > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, June 17, 2013 9:00 PM > To: dev@cloudstack.apache.org > Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi > > The Project Management Committee (PMC) for

Re: Review Request: remove dead allocations

2013-06-17 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11324/#review22040 --- api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.j

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Radhika Puthiyetath
Welcome aboard Jayapal. Well done.. (I thought you were one, though!) -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Monday, June 17, 2013 9:00 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi The Project Management

Re: Hack Day at CloudStack Collaboration Conference

2013-06-17 Thread Daan Hoogland
Is 'secondary storage maintenance mode' maybe a brake-out of your session John? On Mon, Jun 17, 2013 at 11:20 PM, John Burwell wrote: > All, > > I added a Storage Architecture topic to discuss the next steps in evolving > the Storage layer including (my hobby horse) breaking Storage->Hypervisor

Re: committer wanted for review

2013-06-17 Thread Daan Hoogland
John, I like your ground principles, I will keep looking a bit for a better spot to solve the problem. I am not sure your arguments about technical debt and user input validation apply in the situation at hand. A user will probably be just annoyed if we reject a valid posix path because it does

Re: Query String Request Authentication(QSRA) support by S3 providers

2013-06-17 Thread Min Chen
Hi John, This is regarding extractTemplate api, where for extractable template, users can click "Download Template" button from UI to get a http url to download the template already stored at S3 without providing S3 credentials. In 4.1, we don't have this issue, since the URL returned is the pu

Re: Review Request: Update test case and framework to add test name in account

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11921/ --- (Updated June 18, 2013, 4:28 a.m.) Review request for cloudstack, Prasanna Sant

RE: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Mice Xia
Kelven, After the refactoring, will CS still restart HA enabled VM when it is power off externally (e.g. using xencenter) or internally (user initiated shutdown or crash)? Is seems the method HighAvailabilityManagerImpl .scheduleRestart() is not called when VM's actual state is stopped while e

Re: Query String Request Authentication(QSRA) support by S3 providers

2013-06-17 Thread John Burwell
Min, Why are we mucking with ACLs at all? The best security practice would be to create a bucket for CloudStack's use and assign it a dedicated access key and secret key pair with read/write access only to that bucket. Requiring an administrative account to an object store opens an unnecessarily

Re: Query String Request Authentication(QSRA) support by S3 providers

2013-06-17 Thread Thomas O'Dowd
Hi Min, RiakCS seems to support QSRA according to: http://docs.basho.com/riakcs/latest/cookbooks/Authentication/#Query-String-Authentication I'm not sure about other S3 AWS compatible providers. Perhaps others on the list can give feedback? I had a quick look at Ceph for example and they don't ex

Re: Review Request: Update test case and framework to add test name in account

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11921/ --- (Updated June 18, 2013, 12:04 a.m.) Review request for cloudstack, Prasanna San

Re: Review Request: Removed-junk-characters-from-test_netscaler_lb.py.patch

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11901/ --- (Updated June 17, 2013, 11:47 p.m.) Review request for cloudstack, Prasanna San

Re: Review Request: Update test case and framework to add test name in account

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11921/ --- (Updated June 17, 2013, 11:43 p.m.) Review request for cloudstack, Prasanna San

Review Request: Update test case and framework to add test name in account

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11921/ --- Review request for cloudstack, Prasanna Santhanam and Girish Shilamkar. Descrip

RE: Hack Day at CloudStack Collaboration Conference

2013-06-17 Thread Edison Su
+1:) > -Original Message- > From: John Burwell [mailto:jburw...@basho.com] > Sent: Monday, June 17, 2013 2:21 PM > To: dev@cloudstack.apache.org > Subject: Re: Hack Day at CloudStack Collaboration Conference > > All, > > I added a Storage Architecture topic to discuss the next steps in e

Re: [GSOC] A short description about CloudStack Networking plugin

2013-06-17 Thread Daan Hoogland
good show, as they say over in the west. I wish I had this a month ago :) On Mon, Jun 17, 2013 at 6:25 PM, Chip Childers wrote: > On Mon, Jun 17, 2013 at 03:41:58PM +0700, Nguyen Anh Tu wrote: > > Hi all, > > > > I made an wiki entry about the CloudStack networking design. I think it's > > usefu

Re: doc hacking at Hack Day at CCC13

2013-06-17 Thread Daan Hoogland
Ok, this is not what I mean, though it wouldn't hurt having that kind of info at hand in the source tree with links to and from code. I do mean: 1 class diagrams, though not all relations can be generated from source 2 javadocs at least on class level regards, On Mon, Jun 17, 2013 at 8:44 PM, J

Re: Review Request: Test suite test_shared_networks.py not executing since missing import attr

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11917/ --- (Updated June 17, 2013, 9:21 p.m.) Review request for cloudstack, Prasanna Sant

Re: Hack Day at CloudStack Collaboration Conference

2013-06-17 Thread John Burwell
All, I added a Storage Architecture topic to discuss the next steps in evolving the Storage layer including (my hobby horse) breaking Storage->Hypervisor dependencies. Thanks, -John On Jun 16, 2013, at 6:30 PM, John Burwell wrote: > All, > > I added a topic to discuss Guava's features, and

Review Request: Test suite test_shared_networks.py not executing since missing import attr

2013-06-17 Thread Rayees Namathponnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11917/ --- Review request for cloudstack, Prasanna Santhanam and Girish Shilamkar. Descrip

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Kelven Yang
Haven't created a patch yet, will do it soon after some last wrap-ups. Kelven On 6/17/13 12:03 PM, "John Burwell" wrote: >Kelven, > >Did this patch get pushed to Review Board? If so, what is the URL? > >Thanks. >-John > >On Jun 17, 2013, at 1:40 PM, Kelven Yang wrote: > >> Low level classes w

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
Sounds good...We don't want to add technical debt if it's going to make our work a lot harder in the future. On Mon, Jun 17, 2013 at 3:14 PM, John Burwell wrote: > Mike, > > My goal is not incur further technical debt in 4.2 by adding more > Storage->Hypervisor dependencies that need to be inve

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Mike, My goal is not incur further technical debt in 4.2 by adding more Storage->Hypervisor dependencies that need to be inverted. Recognizing that we are close to 4.2, the question becomes is there a simple approach that will permit this dependency to be inverted? I will dig into the code t

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
I think a hack-day session on this would be great. To me, since we're so late in the game for 4.2, I think we need to take two approaches here: 1) Short-term solution for 4.2 (that hopefully will not make future refactoring work too much more difficult than it might already be) and 2) Long-term so

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Edison, As part of the hack day discussion, I think we need to determine how to establish that layer and invert these dependencies. Hypervisors must know about storage and network devices. A VM is the nexus of a particular set of storage devices/volumes and network devices/interfaces. From a

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
I think Zadara Storage will be looking to implement a plug-in in an upcoming release. They have a similar use case to SolidFire where, I believe, their primary storage represents a SAN at the zone level. On Mon, Jun 17, 2013 at 2:54 PM, Edison Su wrote: > But currently there is no such hypervi

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Edison Su
But currently there is no such hypervisor layer yet, and to me it's related to storage, not related to hypervisor. It's a property of a storage to support one hypervisor, two hypervisors, or all the hypervisors, not a property of hypervisor. I agree, that add a hypervisor type on the storagepool

Re: [MERGE] disk_io_throttling to MASTER

2013-06-17 Thread Mike Tutkowski
Sounds good I have a little bit of clean-up work to do and then I must figure out how to generate a patch (since I was using merge rather than rebase during development), so it will probably be tomorrow. On Mon, Jun 17, 2013 at 2:50 PM, John Burwell wrote: > Mike, > > Great news. Let me know

Re: [MERGE] disk_io_throttling to MASTER

2013-06-17 Thread John Burwell
Mike, Great news. Let me know when Review Board is updated, and I will perform a second round review. Thanks, -John On Jun 17, 2013, at 4:48 PM, Mike Tutkowski wrote: > FYI: I added the IOPS-capacity parameter and related code over the weekend. > > The final bit of work for me comes when W

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Marcus, I am coming of the viewpoint that ImageService (ISOs and Templates), hypervisor snapshotting, and DataMotionService should moved from the Storage layer into the Hypervisor layer for the following reasons: The storage layer should treat the data it stores as opaque. These services deal

Re: [MERGE] disk_io_throttling to MASTER

2013-06-17 Thread Mike Tutkowski
FYI: I added the IOPS-capacity parameter and related code over the weekend. The final bit of work for me comes when Wei's code is merged into master and I pull it down. At that point, I need to add the GUI and API logic to support mutual exclusion of our features. On Sat, Jun 15, 2013 at 11:12

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Edison, For me, this issue comes back to the whole notion of the overloaded StoragePoolType. A hypervisor plugin should declare a method akin to getSupportedStorageProtocols() : ImmutableSet which the Hypervisor layer can use to filter the available DataStores from the Storage subsystem. For

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
What do we do, though, if the storage can only work on a subset of the ones listed in the enum? For example, XenServer and VMware. On Mon, Jun 17, 2013 at 2:27 PM, Edison Su wrote: > There are storages which can only work with one hypervisor, > e.g. Currently, Ceph can only work on KVM. And t

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Edison Su
There are storages which can only work with one hypervisor, e.g. Currently, Ceph can only work on KVM. And the data store created in VCenter, can only work with Vmware. > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Monday, June 17, 2013 1:12

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Marcus Sorensen
I can understand the intention, for example templates are tied to a hypervisor because the OS installed works with that hypervisor (drivers, etc), and templates end up on primary storage. To some extent what's on the volume is hypervisor dependent, AND the storage technology is possibly hypervisor

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
I figured you might have something to say about this, John. :) Yeah, I have no idea behind the motivation for this change other than what Edison just said in a recent e-mail. It sounds like this change went in so that the allocators could look at the VM characteristics and see the hypervisor type

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread John Burwell
Mike, I know my thoughts will come as a galloping shock, but the idea of a hypervisor type being attached to a volume is the type of dependency I think we need to remove from the Storage layer. What attributes of a DataStore/StoragePool require association to a hypervisor type? My thought is

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
Hi Edison, How's about if I add this logic into ZoneWideStoragePoolAllocator (below)? After filtering storage pools by tags, it saves off the ones that are for any hypervisor. Next, we filter the list down more by hypervisor. Then, we add the storage pools back into the list that were for any h

Re: Git Push Summary

2013-06-17 Thread Chiradeep Vittal
David, this is a temporary scratch branch to perform some integration testing since atm the master looks stable. On 6/17/13 11:57 AM, "David Nalley" wrote: >What is this branch for? >On Jun 17, 2013 1:27 PM, wrote: > >> Updated Branches: >> refs/heads/master-6-17-stable [created] fc16e29f9 >>

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Joe Brockmeier
On Mon, Jun 17, 2013, at 10:30 AM, Chip Childers wrote: > Please join me in congratulating Jayapal! Woot! Congrats! Best, jzb -- Joe Brockmeier j...@zonker.net Twitter: @jzb http://www.dissociatedpress.net/

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread John Burwell
Kelven, Did this patch get pushed to Review Board? If so, what is the URL? Thanks. -John On Jun 17, 2013, at 1:40 PM, Kelven Yang wrote: > Low level classes were tested in unit tests(MessageBus, Job framework, Job > dispatchers etc), interface layer changes are guarded through matching the >

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread John Burwell
Congrats and welcome, Jayapal. On Jun 17, 2013, at 12:55 PM, Koushik Das wrote: > Congrats Jayapal > >> -Original Message- >> From: Chip Childers [mailto:chip.child...@sungard.com] >> Sent: Monday, June 17, 2013 9:02 PM >> To: dev@cloudstack.apache.org >> Subject: [ANNOUNCE] New committ

Re: Git Push Summary

2013-06-17 Thread David Nalley
What is this branch for? On Jun 17, 2013 1:27 PM, wrote: > Updated Branches: > refs/heads/master-6-17-stable [created] fc16e29f9 >

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Ahmad Emneina
Awe yeah. Good work Jaypal. Thanks for the contributions so far and look forward to more! Ahmad On Jun 17, 2013, at 8:30 AM, Chip Childers wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Jayapal Reddy Uradi to become a committer and we are > pleased to announ

Re: doc hacking at Hack Day at CCC13

2013-06-17 Thread Joe Brockmeier
On Sat, Jun 15, 2013, at 09:27 AM, Daan Hoogland wrote: > To all of you planning to hack away at documentation: > > There is quite a lot of commeted out code. And there is quite a lot of > code > uncommented; classes and public/protected and package scope methods > without > description. I have

Re: Review Request: remove duplicated VPC router in return value of DomainRouterDaoImpl.listByStateAndNetworkType

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10062/#review22007 --- Wei, Should this still be applied? If so, can you go ahead and do

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Harikrishna Patnala
Congratulations Jayapal. On 17-Jun-2013, at 10:29 PM, Abhinandan Prateek wrote: > Congrats Jayapal ! Well deserved. > > On 17/06/13 9:00 PM, "Chip Childers" wrote: > >> The Project Management Committee (PMC) for Apache CloudStack >> has asked Jayapal Reddy Uradi to become a committer and we

Re: Review Request: CLOUDSTACK-1623 MySQL Database connection can fail to "localhost" on a V6 enabled host

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9852/#review22005 --- Ship it! Applied to master: d4477ba8da60290d21afd58ce0901d12c85de3a9

Re: Review Request: CLOUDSTACK-1623 MySQL Database connection can fail to "localhost" on a V6 enabled host

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

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11910/ --- (Updated June 17, 2013, 6:37 p.m.) Review request for cloudstack, Abhinandan Pr

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
> On June 17, 2013, 3:12 p.m., Prasanna Santhanam wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 1818 > > > > > > can this be made case insensitive? so comparisons can compare as > > equalIgnoreCase?

Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10858/#review22001 --- Commit 3e3e5830b45eeba16b3dacfb5475d53d8d2dee27 in branch refs/head

Re: Review Request: NPE fix in StoragePoolJoinDaoImpl

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11899/#review22000 --- Ship it! Applied to master: 202cd1529054fe60acce0cce54686268797b65b

Re: Review Request: removed 3 NumbersUtils methods

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11894/#review21999 --- I get the following conflicts when attempting to apply this: git am

Re: Review Request: String instantiation is not needed

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11750/#review21997 --- Ship it! Applied to master: 564013bec0d4356232d93ac52a3e44638578bff

Re: enableStorageMaintenance

2013-06-17 Thread La Motta, David
Yep. The purpose of quiescing is exactly as described in that document: taking a backup without powering off the VM. David La Motta Technical Marketing Engineer Citrix Solutions NetApp 919.476.5042 dlamo...@netapp.com On Jun 17, 2013, at 2:09 PM, Edison Su mail

Re: Review Request: use commons-lang StringUtils

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11767/#review21998 --- Ship it! Applied to master: c88d8fb3a2f6c418c6c7af8ff702a93bcdb2d75

Re: Review Request: Removed String instantiation

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11751/#review21996 --- Ship it! Applied to master: ce8ada030d3150087357d7135c3877c25a4702c

RE: enableStorageMaintenance

2013-06-17 Thread Edison Su
> -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Monday, June 17, 2013 8:37 AM > To: > Subject: Re: enableStorageMaintenance > > Along the same lines... is there a REST command coming in 4.2 to quiesce one > or multiple virtual machines? Quiesce me

Re: Review Request: CLOUDSTACK-869-nTier-Apps-2.0_Support-NetScalar-as-external-LB-provider

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

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 05:40:36PM +, Kelven Yang wrote: > Low level classes were tested in unit tests(MessageBus, Job framework, Job > dispatchers etc), interface layer changes are guarded through matching the > old semantics, but changes are tested manually, we are planning to get > this part

RE: enableStorageMaintenance

2013-06-17 Thread Edison Su
> -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Friday, June 14, 2013 7:54 AM > To: > Subject: enableStorageMaintenance > > ...works great for putting down the storage into maintenance mode (looking > forward seeing this for secondary storage as we

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21991 --- Thank you for submitting this patch! I've actually done the changes

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21990 --- Commit 25726277975fb9e39ade7c08d680a93d33dc16b2 in branch refs/heads

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

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

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

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

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Kelven Yang
Low level classes were tested in unit tests(MessageBus, Job framework, Job dispatchers etc), interface layer changes are guarded through matching the old semantics, but changes are tested manually, we are planning to get this part of testing through BVT system after we have re-based the latest mast

Re: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Mike Tutkowski
Hi Edison, I haven't looked into this much, so maybe what I suggest here won't make sense, but here goes: What about a Hypervisor.MULTIPLE enum option ('Hypervisor' might not be the name of the enumeration...I forget). The ZoneWideStoragePoolAllocator could use this to be less choosy about if a s

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

2013-06-17 Thread Edison Su
I think it's due to this https://cwiki.apache.org/confluence/display/CLOUDSTACK/Zone-wide+primary+storage+target There are zone-wide storages, may only work with one particular hypervisor. For example, the data store created on VCenter can be shared by all the clusters in a DC, but only for vmwa

Query String Request Authentication(QSRA) support by S3 providers

2013-06-17 Thread Min Chen
Tom filed a very good bug for ACL setting change on S3 object when users issue extractTemplate API (https://issues.apache.org/jira/browse/CLOUDSTACK-3030), and his recommendation of using Query String Request Authentication (QSRA) alternative sounds like a right approach to fix this bug. Before

[MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Kelven Yang
I'd like to kick off the official merge process. We will start the merge process after the branch has passed necessary tests Kelven On 6/10/13 2:51 PM, "Kelven Yang" wrote: >Hi there, > >Alex Huang and I are targeting to finish the debugging process on VMsync >improvement by the end of this wee

Re: Review Request: CLOUDSTACK-2902: Updating repository refs

2013-06-17 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11908/#review21985 --- Commit 76d3c27bf4c0ab3690840e56ca162935cea91d48 in branch refs/heads

Re: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 04:59:00PM +, Kelven Yang wrote: > I'd like to kick off the official merge process. We will start the merge > process after the branch has passed necessary tests > > Kelven Can you share what testing is being run against the branch?

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Abhinandan Prateek
Congrats Jayapal ! Well deserved. On 17/06/13 9:00 PM, "Chip Childers" wrote: >The Project Management Committee (PMC) for Apache CloudStack >has asked Jayapal Reddy Uradi to become a committer and we are >pleased to announce that they have accepted. > >Being a committer allows many contributors

Re: systemvm.iso not updated in packages

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 10:22:44PM +0530, Prasanna Santhanam wrote: > On Mon, Jun 17, 2013 at 12:08:54PM -0400, Chip Childers wrote: > > On Mon, Jun 17, 2013 at 11:05:43AM +0530, Prasanna Santhanam wrote: > > > Applied yet another fix for this from Rajesh: > > > > > > commit 6d140538c5efc394fda8a4

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Koushik Das
Congrats Jayapal > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, June 17, 2013 9:02 PM > To: dev@cloudstack.apache.org > Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi > > The Project Management Committee (PMC) for Apache CloudStack ha

Re: Object based Secondary storage.

2013-06-17 Thread John Burwell
Min, Cool. I just wanted to make sure we weren't compressing the template and template.properties … Thanks for the clarification, -John On Jun 17, 2013, at 12:49 PM, Min Chen wrote: > John, > > Let me clarify, we didn't do extra compression before sending to S3. > Only > when user pr

Re: systemvm.iso not updated in packages

2013-06-17 Thread Prasanna Santhanam
On Mon, Jun 17, 2013 at 12:08:54PM -0400, Chip Childers wrote: > On Mon, Jun 17, 2013 at 11:05:43AM +0530, Prasanna Santhanam wrote: > > Applied yet another fix for this from Rajesh: > > > > commit 6d140538c5efc394fda8a4ddc7cb72832470d0b3 > > Author: Rajesh Battala > > Date: Sat Jun 15 11:21:46

Re: Object based Secondary storage.

2013-06-17 Thread Min Chen
John, Let me clarify, we didn't do extra compression before sending to S3. Only when user provides a URL pointing to a compressed template during registering, we will just download that template to S3 without decompressing it afterwards as we did for NFS currently. If the register url pro

Re: committer wanted for review

2013-06-17 Thread John Burwell
Daan, Please see my comments in-line below. Thanks, -John On Jun 17, 2013, at 9:40 AM, Daan Hoogland wrote: > John, > > If I understand it correctly, you are stating that my take on the solution > is 'not done/not the way to go'? > > For the record the case I solved was an instance of A, bu

Re: Object based Secondary storage.

2013-06-17 Thread John Burwell
Min, Why are objects being compressed before being sent to S3? Thanks, -John On Jun 17, 2013, at 12:24 PM, Min Chen wrote: > Hi Tom, > > Thanks for your testing. Glad to hear that multipart is working fine by > using Cloudian. Regarding your questions about .gz template, that behavior >

Re: [GSOC] A short description about CloudStack Networking plugin

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 03:41:58PM +0700, Nguyen Anh Tu wrote: > Hi all, > > I made an wiki entry about the CloudStack networking design. I think it's > useful for all network plugins can follow. It's located in my gsoc project > about improving the native SDN controller. Take a look on it > > ht

Re: Object based Secondary storage.

2013-06-17 Thread Min Chen
Hi Tom, Thanks for your testing. Glad to hear that multipart is working fine by using Cloudian. Regarding your questions about .gz template, that behavior is as expected. We will upload it to S3 as its .gz format. Only when the template is used and downloaded to primary storage, we will us

Re: systemvm.iso not updated in packages

2013-06-17 Thread Chip Childers
On Mon, Jun 17, 2013 at 11:05:43AM +0530, Prasanna Santhanam wrote: > Applied yet another fix for this from Rajesh: > > commit 6d140538c5efc394fda8a4ddc7cb72832470d0b3 > Author: Rajesh Battala > Date: Sat Jun 15 11:21:46 2013 +0530 > > CLOUDSTACK-3004: remove duplicate ssvm-check.sh >

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Saksham Srivastava
Congrats Jayapal. -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Monday, June 17, 2013 9:00 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi The Project Management Committee (PMC) for Apache CloudStack has asked Jaya

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Prasanna Santhanam
On Mon, Jun 17, 2013 at 11:30:16AM -0400, Chip Childers wrote: > The Project Management Committee (PMC) for Apache CloudStack > has asked Jayapal Reddy Uradi to become a committer and we are > pleased to announce that they have accepted. > > Being a committer allows many contributors to contribut

  1   2   >