Thanks Kishan for your input.

Since we will now rely on Events Framework feature to publish events that could 
be used by admins to maintain data sync between regions , I have created test 
plan for this feature- 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Events+Framework+Test+plan
 .
This test plan addresses only a part of Event Framework feature that is needed 
by AWS regions feature for achieving data sync of Account/Domain/User objects 
between regions.
Please provide your feedback.

-Thanks
Sangeetha

-----Original Message-----
From: Kishan Kavala [mailto:kishan.kav...@citrix.com] 
Sent: Tuesday, March 12, 2013 5:53 AM
To: cloudstack-dev@incubator.apache.org
Subject: RE: [CLOUDSTACK-241 ]Regions update

Please find my response inline.

> -----Original Message-----
> From: Sangeetha Hariharan [mailto:sangeetha.hariha...@citrix.com]
> Sent: Saturday, 9 March 2013 1:14 AM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [CLOUDSTACK-241 ]Regions update
> 
> I have updated the test cases according to the  latest functional Spec 
> which is to make use of Event frameworks instead of api based sync for 
> domain/account/users.
> Test cases can be found here -
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/AWS+Regions+T
> est+Plan . Please provide your feedback.
> 
> I had the following questions:
> 
> 1. In case of update/delete action from a region that is not the owner 
> , API call will succeed as long as it has been able to forward the 
> request to owner region and the owner region actually executed the 
> command . Even after receiving a success from the API call, the 
> changes will not be committed in this region. This seems like a 
> confusing experience . Can we expect any message indicating to the user about 
> the redirection of request?
> 
[KK]  Forwarding to owner region should be removed. Data sync is external to 
CloudStack.

> 2. In case of deleting domains/accounts from a region that is not the 
> owner of this region , this will result in the domain/account being 
> actually deleted from the owner region which will also result in all 
> the resources like network and Vms being deleted. Is this acceptable?
[KK]  When account/domain is deleted all resources will also be deleted.

> 
> 3. Consider the case of domains/accounts (owner Region 1) that have 
> users that are owned by 2 different regions , say user1 (owner Region 
> 1) and user2 (owner Region2) . Which region is responsible for 
> deletion of domain/account  , since this deletion will also involve 
> deletion of users which are now owned by 2 different regions?
> 
[KK] Since data sync is external to CloudStack, we don't need the notion of 
owner region. 

> 4. Why is there a need for getUser API ? When will this get used ?
[KK]  getUser API is for components like object-store to authenticate and get 
account information
> 
> 5. Can you include the response parameters for all the new API calls ?
[KK]  I've added the response parameters to the spec.
> 
> -Thanks
> Sangeetha
> 
> 
> -----Original Message-----
> From: Kishan Kavala [mailto:kishan.kav...@citrix.com]
> Sent: Thursday, February 28, 2013 12:53 AM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [CLOUDSTACK-241 ]Regions update
> 
> > -----Original Message-----
> > From: Kishan Kavala [mailto:kishan.kav...@citrix.com]
> > Sent: Wednesday, 27 February 2013 12:07 PM
> > To: cloudstack-dev@incubator.apache.org
> > Subject: RE: [CLOUDSTACK-241 ]Regions update
> >
> > > -----Original Message-----
> > > From: Chip Childers [mailto:chip.child...@sungard.com]
> > > Sent: Tuesday, 26 February 2013 8:22 PM
> > > To: cloudstack-dev@incubator.apache.org
> > > Subject: Re: [CLOUDSTACK-241 ]Regions update
> > >
> > > On Tue, Feb 26, 2013 at 02:34:13PM +0530, Kishan Kavala wrote:
> > > > Removed API based account sync for Regions as discussed earlier [1] .
> > > Event framework will be used instead. I pushed the changes to master.
> > > commit: e37fdce125511535961c77a77d253ce232a954d0.
> > > > I'll be merging the same to 4.1 branch.
> > > >
> > > > [1]
> > > > http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-
> > > dev/2013
> > > > 02.mbox/%3ccd3462cf.64a2%25chiradeep.vit...@citrix.com%3E
> > > >
> > >
> > > Is this something we really want to do now?  Has the feature been 
> > > QA'ed
> > yet?
> > > This seems pretty late to make a change to the basic function of 
> > > the feature for 4.1.
> > >
> > > If my concern are overblown, then say so!
> > >
> > > -chip
> >
> > This commit only removes API based account data sync and it is 
> > fairly isolated part of the code.
> > Events are already in place so no changes in that area.
> >
> 
> Changes were merged to 4.1 branch. I also changed tabs to spaces in 
> the related files, so the diff shows more changes than actual.

Reply via email to