Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of Kishan.Kavala@citrix.com
 designates 203.166.19.134 as permitted sender)
From: Kishan Kavala <Kishan.Kavala@citrix.com>
To: "cloudstack-dev@incubator.apache.org"
	<cloudstack-dev@incubator.apache.org>
Date: Thu, 28 Feb 2013 14:22:59 +0530
Subject: RE: [CLOUDSTACK-241 ]Regions update
Thread-Topic: [CLOUDSTACK-241 ]Regions update
Thread-Index: Ac4UMO86+s9RRjDtTiCaXMnJJXz8GAAgZ1xAADeLZeA=
Message-ID: 
 <33740054EBF5B64BB213E2E0916F2C13F075FBB94C@BANPMAILBOX01.citrite.net>
References: 
 <33740054EBF5B64BB213E2E0916F2C13F075FBB93F@BANPMAILBOX01.citrite.net>
 <20130226145202.GF81779@USLT-205755.sungardas.corp>
 <33740054EBF5B64BB213E2E0916F2C13F075FBB945@BANPMAILBOX01.citrite.net>
In-Reply-To: 
 <33740054EBF5B64BB213E2E0916F2C13F075FBB945@BANPMAILBOX01.citrite.net>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

> -----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
>=20
> > -----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
>=20
> This commit only removes API based account data sync and it is fairly iso=
lated
> part of the code.
> Events are already in place so no changes in that area.
>=20

Changes were merged to 4.1 branch. I also changed tabs to spaces in the rel=
ated files, so the diff shows more changes than actual.