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.