Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of Alex.Huang@citrix.com
 designates 66.165.176.63 as permitted sender)
From: Alex Huang <Alex.Huang@citrix.com>
To: "cloudstack-dev@incubator.apache.org"
	<cloudstack-dev@incubator.apache.org>
Date: Tue, 11 Sep 2012 11:26:05 -0700
Subject: RE: [ASFCS40]Drive to clear the review board...
Thread-Topic: [ASFCS40]Drive to clear the review board...
Thread-Index: Ac2Pj5PcPLQ1JyOZTCe6TCyLxZeSigAuiBjg
Message-ID: 
 <B1DF26ECC0458748AC97CECE2DA98D41012FA02BA997@SJCPMAILBOX01.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

Hi everyone,

Here is the third pass.  We cleared quite a few reviews yesterday but there=
 were more submitted so we actually have more.  The good news the action fo=
r about a quarter of the reviews  is to "Commit it" or "Mark as submitted".=
  Please take a look.  Especially the people with reviews that says "Mark a=
s submitted", please take a quick action item to check the fix is in and ma=
rk it as submitted to get it out of the review board.

Note that for everyone's convenience I added another column call Updated. =
=20
- The first column is the review #.=20
- The second column is for who to take action. =20
- The third column is the version this patch should go into.  If it is 4+, =
you don't have to take action before 4.0 release. =20
- The fourth column is if the review has changed since last push.  If you'r=
e following this email daily, you should be able to just monitor this colum=
n to determine if there's new action item for you since the last push.
- The fifth column is what I believe is the action needed to close out this=
 review.

Review	Assign To	Version	Updated	Action to Take
5625	Vijay	4		Retract the review request and resubmit when review is done
5655	Hugo	4+		Respond to Review
5806	Deepti/Nitin	4	Y	Nitin to respond to diff and commit if okay to ship
5871	Vijay	4	Y	Mark as submit
6287	Edison	4+		Respond to Wido
6328	Jessica T	4		Commit it
6425	Likitha	4		Redo the patch with the new DAO
6427	Jessica T	4		Doc change.  Not sure if Jessica can commit it.  If not t=
hen pradeep.
6428	Likitha	4		Fix according to review?
6431	Anthony	4	Y	Review and Commit
6470	Marcus	4		Take this off the review board as it is committed
6492	Krishna	4+		Respond to review comments
6502	Fang	4		Ship it
6515	Abhi	4+	Y	Review fix
6523	Nitin	4+		Respond to Rohit
6547	Sheng	4		Commit it
6614	Krishna	4+		Respond to review.
6615	Marcus	4		Mark as submit
6658	Marcus	4		Mark as submit
6666	Marcus	4		Mark as submit
6701	Vijay/Murali	4		Make sure the diff has been committed and remove if it=
 is.  If not committed, then Murali needs to commit it.
6702	Vijay	4		Respond to review
6720	Sheng	4		Commit it
6733	Gregg	4	Y	Mark as submit
6765	Anthony	4		Review and Commit.=20
6781	Rohit	4+	Y	Submit new patches
6793	Sheng	4		Review and Commit.
6814	Alena	4		Review and Commit.
6825	Chiradeep	4	Y	Review and Commit.
6845	Alena	4		Review and Commit.
6858	Nitin	4	Y	Commit it
6859	Sheng	4		Commit it
6873	Anthony	4		Review it
6881	Jessica T	4		Review it
6883	Sheng	4		Commit it
6895	Chiradeep	4		Review and Commit
6899	Vijay/Pranav	4	Y	Pranav to provide commit id and Vijya to mark as Subm=
itted
6917	Jessica T	4		Review and Commit
6926	Sheng	4		Review and Commit
6941	Pradeep	4		Review and Commit
6978	Radhika	4		Mark as submit
7009	Hiroaki	4	Y	Mark as submit
7013	Alena	4	Y	Review
7015	Sheng	4	Y	Review
7017	Alena	4+	Y	Review and Commit
7018	Nitin	4	Y	Review
7025	Fang	4	Y	Fix according to review