Here is the partial review board update till now. 1. Out of the 44 called out on Wednesday, 14 of the patches have been reviewed and committed
2. There are no updates on following 9 issues that were called out Review Who to take Action Action to be taken 8876 Likitha Respond to review comments, and submit the patch again 8701 Roeland Kuipers Respond to commit and resubmit the patch after fixes 8545 Meghna Respond to comment and resubmit the patch. Patch does not apply cleanly 8355 Meghna Respond to comment 8307 Kelecey Respond to comment and resubmit the patch 8293 Hiroaki Respond to reviewer comment 7802 Joe Review patch and commit if satisified 7594 Anthony Review updated patch 5655 Hugo Check if review still applicable? If not close it or apply a fix. We will keep these open for one more week, if there is no activity on them, we will have to discard them next week 3. Here are the revised action items for other open issues that were updates in last couple of days. Since 4.1 feature freeze date is passed already, patches will need to be committed to master other than the documentation patches for features that made to 4.1 Review Who to take Action Action to be taken 9158 Nitin Review the updated diff 9157 Abhinandan, Koushik Review updated patch and commit if satisfied 9156 Abhinandan, Nitin Review updated patch and commit if satisfied 9118 Murali, Wido, Kelven Yang Respond to review and commit if okay to ship 9110 Devdeep, Chiradeep Review updated patch and commit if satisfied 9097 David Nalley Review updated patch and commit if satisfied 9041 David Nalley, Min Review comment response and commit if satisfied 8991 Chip, Jessica Reference to CloudPlatform is removed. Chip/Jessica to review and commit 8971 Kishan Review updated patch and commit if satisfied 8900 Nitin. Devedeep Review response to review comments, If satisfied commit the patch 8859 Kishan Change the patch to submitted 8813 Jessica Tomechak IP clearance approved already. Commit the patch 8636 Jessica Tomechak Respond to comment 8294 Kawai update the patch to the latest state of master 8180 Jessica Tomechak Ipclearance cleared the patch can be committed 7891 Koushik Respond to review comments and resubmit the patch 7827 Koushik Move to submitted state 7660 Deepti Patch is updated but some referenced files are no longer there. Review and resubmit 7018 Nitin Review patch and commit if satisfied 5. There are 34 new patches (starting with review ID 9165) that have come in since Wednesday and I have not got to looking at them yet. Any volunteers willing to help out? > -----Original Message----- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Wednesday, January 30, 2013 2:57 PM > 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
