cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kishan Kavala <Kishan.Kav...@citrix.com>
Subject RE: [CLOUDSTACK-241 ]Regions update
Date Thu, 28 Feb 2013 08:52:59 GMT
> -----Original Message-----
> From: Kishan Kavala [mailto:Kishan.Kavala@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.childers@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.vittal@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.

Mime
View raw message