> -----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.