Hey Marcus,

Could you do us a favor please, and go through all of your reviewboard
submissions that have been committed and mark then as closed /
submitted?  There isn't an admin role on reviewboard, so the
originator is the one that needs to close them out.

This would help Alex tremendously, as he tries to wrangle all of the
outstanding reviews for 4.0.

Thanks!

-chip

On Fri, Sep 14, 2012 at 1:41 PM, Alex Huang <alex.hu...@citrix.com> wrote:
> I've moved out all auto-scaling reviews to 4+ as that didn't make it into 4.0.
>
> All you have to do is look up your name and then look to see if the review 
> has been changed from yesterday.  If you haven't kept track of this daily 
> email, then you need to look at all the items under your name and work on all 
> of them.  Please scan the list for your name.
>
> Does anyone know Marcus and can send him a note to close his review submits?  
> Thanks.
>
> - The first column is the review #.
> - The second column is for who to take action.
> - 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.
> - The fourth column is if the review has changed since last push.  If you're 
> following this email daily, you should be able to just monitor this column 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               Nitin to respond to diff and commit 
> if okay to ship
> 6470    Marcus  4               Take this off the review board as it is 
> committed
> 6473    Jamshid 4               Mark as submit
> 6492    Krishna 4+              Respond to review comments
> 6523    Nitin   4+              Respond to Rohit
> 6614    Krishna 4+              Respond to review.
> 6615    Marcus  4               Mark as submit
> 6658    Marcus  4               Mark as submit
> 6666    Marcus  4               Mark as submit
> 6702    Vijay   4               Responde to review
> 6720    Marcus  4               Review Sheng's comments.  Withdraw or 
> resubmit patch.
> 6733    Gregg   4               Mark as submit
> 6765    Anthony 4               Review and Commit.
> 6781    Alena/Chiradeep 4+              Review new patches/Alena to commit
> 6793    Marcus  4               Mark as submit
> 6858    Nitin   4               Commit it
> 6859    Marcus  4               Mark as submit
> 6881    Jie     4               Mark as submit
> 6883    Sheng   4               Commit it
> 6926    Marcus  4               Mark as submit
> 6928    Sateesh 4               Commit it
> 6937    Pradeep 4               Review
> 6941    Pradeep 4               Review and Commit
> 7013    Pradeep 4               Respond to comment
> 7015    Pradeep 4               Review
> 7018    Nitin   4               Commit it
> 7025    Fang    4               Fix according to review
> 7040    Sheng   4               Review
> 7052    Prasanna/Frank  4               Commit it
> 7057    devdeep/deepak  4+              Review
> 7058    devdeep 4+              Review
> 7078    devdeep/ramganesh       4+              Review
> 7079    Jessica T       4               Review and Commit
> 7083    Edison  4       Y       Respond and Review
> 7099            4+
> 7106    Jessica T       4       Y       Review and Commit
> 7109    Jessica T       4       Y       Review and Commit
> 7113            4+
> 7114    Jessica T       4       Y       Review and Commit
>
>
> --Alex
>

Reply via email to