On Mon, May 20, 2013 at 06:22:03PM +0000, Sudha Ponnaganti wrote: > Also open defects need to be addressed as testing has been done on feature > branch and the numbers seem to be high on this branch - atleast blocker and > critical should be addressed. > > Key Summary Assignee Reporter Priority Status > CLOUDSTACK-2586 [Object_Store_Refactor] Extract template does not give > proper URL to download the template Unassigned Sanjeev N > Critical Open > CLOUDSTACK-2584 [Object_Store_Refactor] Failed to create template from > stopped guest vm's root disk Unassigned Sanjeev N Critical > Open > CLOUDSTACK-2583 [Object_Store_Refactor] Failed to create snapshot from > root disk of a guest VM Unassigned Sanjeev N Blocker Open > CLOUDSTACK-2578 [Object_Store_Refactor] NPE while deleting template > from CS Unassigned Sanjeev N Critical Open > CLOUDSTACK-2538 [Object_Store_Refactor] No option for the user to > provide storage name Unassigned Sanjeev N Major Open > CLOUDSTACK-2528 Object_Store_Refactor - Not able to download volumes. > Unassigned Sangeetha Hariharan Major Open > CLOUDSTACK-2527 Object_Store_Refactor - Default isos (xen-tools.iso and > vmware-tools.iso) are not in "Ready" state and not usable. Unassigned > Sangeetha Hariharan Major Open > CLOUDSTACK-2525 Object_Store_Refactor - In case of template creation > failure , we are not able to remove the tempalte entries. We are not able to > delete these templates later. Unassigned Sangeetha Hariharan Major > Open > CLOUDSTACK-2523 Object_Store_Refactor - Recurring Snapshots are failing > becuase of NullPointerException. Unassigned Sangeetha Hariharan > Blocker Open > CLOUDSTACK-2511 Multiple_Ip_Ranges: Adding guest ip range in > subset/superset to existing CIDR is allowed Unassigned Sanjeev N > Critical Open > CLOUDSTACK-2493 Object_Store_Refactor - Template - ClassCastException > when trying to extract Template. Min Chen Sangeetha Hariharan > Major Reopened > CLOUDSTACK-2481 Object_Store_Refactor - Templates - Not able to create > a template from snapshot. Unassigned Sangeetha Hariharan > Critical Open
+100 - known problems shouldn't be brought into master > > -----Original Message----- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, May 20, 2013 10:53 AM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE]object_store branch into master > > On Fri, May 17, 2013 at 08:19:57AM -0400, David Nalley wrote: > > On Fri, May 17, 2013 at 4:11 AM, Edison Su <edison...@citrix.com> wrote: > > > Hi all, > > > Min and I worked on object_store branch during the last one and half > > > month. We made a lot of refactor on the storage code, mostly related to > > > secondary storage, but also on the general storage framework. The > > > following goals are made: > > > > > > 1. An unified storage framework. Both secondary > > > storages(nfs/s3/swift etc) and primary storages will share the same > > > plugin model, the same interface. Add any other new storages into > > > cloudstack will much easier and straightforward. > > > > > > 2. The storage interface between mgt server and resource is > > > unified, currently there are only 5 commands send out by mgt server: > > > copycommand/createobjectcommand/deletecommand/attachcommand/dettachcommand, > > > and each storage vendor can decode/encode all the > > > entities(volume/snapshot/storage pool/ template etc) by its own. > > > > > > 3. NFS secondary storage is not explicitly depended on by other > > > components. For example, when registering template into S3, template will > > > be write into S3 directly, instead of storing into nfs secondary storage, > > > then push to S3. If s3 is used as secondary storage, then nfs storage > > > will be used as cache storage, but from other components point of view, > > > cache storage is invisible. So, it's possible to make nfs storage as > > > optional if s3 is used for certain hypervisors. > > > The detailed FS is at > > > https://cwiki.apache.org/confluence/display/CLOUDSTACK/Storage+Backu > > > p+Object+Store+Plugin+Framework > > > The test we did: > > > > > > 1. We modified marvin to use new storage api > > > > > > 2. Test_volume and test_vm_life_cycle, test_template under smoke > > > test folder are executed against xenserver/kvm/vmware and devcloud, some > > > of them are failed, it's partly due to bugs introduced by our code, > > > partly master branch itself has issue(e.g. resizevolume doesn't work). We > > > want to fix these issues after merging into master. > > > > > > The basic follow does work: create user vm, attach/detach volume, > > > register template, create template from volume/snapshot, take snapshot, > > > create volume from snapshot. > > > It's a huge change, around 60k LOC patch, to review the code, you can > > > try: git diff master..object_store, will show all the diff. > > > Comments/feedback are welcome. Thanks. > > > > > > > > > > > > Given the amount of change, can we get at least a BVT run against your > > branch done before merge? > > > > --David > > > > +1 to BVT please. >
