incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sanjay Tripathi <sanjay.tripa...@citrix.com>
Subject RE: ReviewBoard: Drive to clear pending reviews <PLEASE READ>
Date Thu, 31 Jan 2013 13:43:01 GMT
Resubmitted the patch with changes in place as per review comments.
You can find the patch for review at: https://reviews.apache.org/r/9110/

Thanks,
Sanjay

> -----Original Message-----
> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> Sent: Thursday, January 31, 2013 4:27 AM
> To: cloudstack-dev@incubator.apache.org
> Subject: ReviewBoard: Drive to clear pending reviews <PLEASE READ>
> 
> Folks
> 
> Here is a list from review board pending items (44 actually)  If your name is in
> the "Who to take Action" column then  check  "Action to be taken" column
> and respond to your called action
> 
> Rev# 	Who to take Action	Action to be taken
> 9158	Harikrishna		"Resubmit the patch after addressing
> comments from Nitin, Koushik, Nitin to commit the patch if comments are
> addressed"
> 9157	Harikrishna		"Resubmit the patch after addressing
> comments from Nitin, Koushik, Nitin to commit the patch if comments are
> addressed"
> 9156	Bharat			"Respond to comments, resubmit the patch.
> Tests missing"
> 9152	Rohit			"Seems trivial change, review and commit"
> 9133	Chiradeep		Review updated patch and commit if satisfied
> 9131	"Chiradeep, Hugo"	Chiradeep to provide clarificaton on Kanzhe's
> email and commit if fit
> 9118	"Murali, Wido, Kelven Yang"	Respond to review and commit if
> okay to ship
> 9110	Sanjay			Respond to chiradeep's review comments
> and resubmit the patch
> 9097	David Nalley		Review updated patch and commit if satisfied
> 9041	"David Nalley, Min"	Review comment response and commit if
> satisfied
> 9010	Jayapal			Resolve merge conflict and resubmit the
> patch. Abhi to commit
> 8991	"Chip, Jessica"		Reference to CloudPlatform is removed.
> Chip/Jessica to review and commit
> 8971	Weizhou		Update diff and post for review again. Kishan
> to review and commit
> 8905	Rohit			IP clearance should be approved by
> Wednesday noon. Commit the patch if satisfied on Wednesday
> 8900	Nitin. Devedeep	"Review response to review comments, If
> satisfied commit the patch"
> 8876	Likitha			"Respond to review comments, and submit
> the patch again"
> 8859	Kishan			Commit the patch if satisfied
> 8813	Jessica			IP clearance should be approved by
> Wednesday. Commit the patch if satisfied on Wednesday
> 8742	Prachi			Commit the patch if satisfied
> 8701	Roeland Kuipers	Respond to commit and resubmit the patch
> after fixes
> 8632	Prasanna		Respond to reviewer comments and submit
> the patch again
> 8623	Devdeep		Respond to reviewer comments and submit
> the patch again
> 8613	Prasanna		Respond to reviewer comments and submit
> the patch again
> 8547	Rohit			Review patch and commit if satisified
> 8545	Meghna		Respond to comment and resubmit the
> patch. Patch does not apply cleanly
> 8513	Prachi			Review patch and commit if satisified
> 8492	Prachi			Review patch and commit if satisified
> 8483	Prachi			Review patch and commit if satisified
> 8480	Prachi			Review patch and commit if satisified
> 8468	Prachi			Review patch and commit if satisified
> 8466	Prachi			Review patch and commit if satisified
> 8465	Prachi			Review patch and commit if satisified
> 8355	Meghna		Respond to comment
> 8307	Kelecey		Respond to comment and resubmit the patch
> 8294	Kawai			update the patch to the latest state of master
> 8293	Hiroaki			Respond to reviewer comment
> 8180	Jessica			Some items are under ipclearance and as
> soon as David formally clears IP clearance the patch can  be committed
> 7891	Abhinadan		Review patch and commit if satisified
> 7827	"Murali, Kelven"	Review patch and commit if satisified
> 7802	Joe			Review patch and commit if satisified
> 7660	Deepti			Patch is old and some of the referenced files
> are no longer there. Review and resubmit
> 7594	Anthony		Review updated patch
> 7018	Nitin			Review patch and commit if satisified
> 5655	Hugo			Check if review still applicable? If not close it
> or apply a fix.
> 
> Since tomorrow is our freeze date , it is important to clear out as much as
> humanly possible :)
> 
> Animesh


Mime
View raw message