Return-Path: X-Original-To: apmail-incubator-cloudstack-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-cloudstack-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 08F3BE0D3 for ; Thu, 31 Jan 2013 13:43:46 +0000 (UTC) Received: (qmail 23776 invoked by uid 500); 31 Jan 2013 13:43:45 -0000 Delivered-To: apmail-incubator-cloudstack-dev-archive@incubator.apache.org Received: (qmail 23653 invoked by uid 500); 31 Jan 2013 13:43:45 -0000 Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cloudstack-dev@incubator.apache.org Delivered-To: mailing list cloudstack-dev@incubator.apache.org Received: (qmail 23635 invoked by uid 99); 31 Jan 2013 13:43:44 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 31 Jan 2013 13:43:44 +0000 X-ASF-Spam-Status: No, hits=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of sanjay.tripathi@citrix.com designates 203.166.19.134 as permitted sender) Received: from [203.166.19.134] (HELO SMTP.CITRIX.COM.AU) (203.166.19.134) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 31 Jan 2013 13:43:36 +0000 X-IronPort-AV: E=Sophos;i="4.84,576,1355097600"; d="scan'208";a="629580" Received: from banpmailmx01.citrite.net ([10.103.128.73]) by SYDPIPO01.CITRIX.COM.AU with ESMTP/TLS/RC4-MD5; 31 Jan 2013 13:43:05 +0000 Received: from BANPMAILBOX01.citrite.net ([10.103.128.71]) by BANPMAILMX01.citrite.net ([10.103.128.73]) with mapi; Thu, 31 Jan 2013 19:13:02 +0530 From: Sanjay Tripathi To: "cloudstack-dev@incubator.apache.org" Date: Thu, 31 Jan 2013 19:13:01 +0530 Subject: RE: ReviewBoard: Drive to clear pending reviews Thread-Topic: ReviewBoard: Drive to clear pending reviews Thread-Index: Ac3/PR4VnnywwsnVTF6VKq/bdeSKQwAe37CA Message-ID: <02C38648D4635F4EB02DE11EE81CF1EE012F10D41D38@BANPMAILBOX01.citrite.net> References: <7A92FF96DF135843B4B608FB576BFC3E012DA3F62A9B@SJCPMAILBOX01.citrite.net> In-Reply-To: <7A92FF96DF135843B4B608FB576BFC3E012DA3F62A9B@SJCPMAILBOX01.citrite.net> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Virus-Checked: Checked by ClamAV on apache.org 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 >=20 > Folks >=20 > Here is a list from review board pending items (44 actually) If your nam= e is in > the "Who to take Action" column then check "Action to be taken" column > and respond to your called action >=20 > 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. >=20 > Since tomorrow is our freeze date , it is important to clear out as much = as > humanly possible :) >=20 > Animesh