You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Animesh Chaturvedi <an...@citrix.com> on 2013/02/12 08:24:14 UTC
RE: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Folks
I have sent a number of reminder emails for some of the review board items that have been pending for a while. Please attend to them at the earliest . We are running at 50+ items in review board. If you are a committer I request you knock off 1-2 reviews per week
Animesh
> -----Original Message-----
> From: Chip Childers [mailto:chip.childers@sungard.com]
> Sent: Tuesday, January 29, 2013 9:18 AM
> To: cloudstack-dev@incubator.apache.org
> Subject: [ACS41] Yet another reviewboard reminder... for both committers and
> contributors.
>
> Hi all (and specifically people that were listed as "maintainers" of certain code
> modules),
>
> We need committer help in getting the community contributions currently in
> reviewboard (as well as some that we expect to show up in the next day or so)
> reviewed and committed as soon as possible.
>
> If you are a "maintainer", or simply feel that you have enough knowledge to get
> through one or more of these reviews, PLEASE take some time to help clear out
> the board.
>
> Based on our agreed upon schedule, we need everything into master (or
> javelin) by EOD Thursday.
>
> For those of you that have code you want to be reviewed, but have not yet
> submitted, it needs to get in ASAP. Otherwise, there's no chance that someone
> will have the time to review and commit before we cut the release branch.
>
> -chip
RE: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Animesh Chaturvedi <an...@citrix.com>.
We don't have review board guidelines yet, I will work on it next week
> -----Original Message-----
> From: Kelcey Damage (BT) [mailto:kelcey@backbonetechnology.com]
> Sent: Friday, February 15, 2013 5:04 PM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [ACS41] Yet another reviewboard reminder... for both committers
> and contributors.
>
> Thanks, you see, I'm not a committer for code, I have very little programming
> experience other than scripts/automation.
>
> I'm more a marketing/evangelist/teaching/support kind of guy.
>
> So if asked to review code, I would not be a good choice. If asked to review
> policy/procedure/compliance/design/architecture(non-code), than I am your
> guy.
>
> That brings me to a question, how does one use the review board? Is there a
> how-to/guidelines wiki article you can point me too?
>
> Thanks
>
> -kelcey
>
>
> >-----Original Message-----
> >From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> >Sent: Friday, February 15, 2013 4:26 PM
> >To: cloudstack-dev@incubator.apache.org
> >Subject: RE: [ACS41] Yet another reviewboard reminder... for both
> >committers and contributors.
> >
> >Well the submitter of the review https://reviews.apache.org/r/9336/
> >listed you as one of the reviewers and I was calling out based on that.
> >
> >Ok so what should one do if someone lists you as a reviewer? If you
> >cannot
> or
> >not qualified to review you can respond saying that you cannot do that
> >review and submitter can ask another reviewer.
> >
> >I will update the review board guidelines.
> >
> >Animesh
> >
> >
> >> -----Original Message-----
> >> From: Kelcey Damage (BT) [mailto:kelcey@backbonetechnology.com]
> >> Sent: Friday, February 15, 2013 3:05 PM
> >> To: cloudstack-dev@incubator.apache.org
> >> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
> >> committers and contributors.
> >>
> >> My name got attached to 1180, and I'm not sure why. I am not what you
> >> would call capable of reviewing the submission other then from a
> >> legal/compliance perspective.
> >>
> >> Thanks
> >>
> >> -Kelcey
> >>
> >> >-----Original Message-----
> >> >From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> >> >Sent: Friday, February 15, 2013 12:05 PM
> >> >To: cloudstack-dev@incubator.apache.org
> >> >Subject: RE: [ACS41] Yet another reviewboard reminder... for both
> >> >committers and contributors.
> >> >
> >> >Folks
> >> >
> >> >I have sent emails on over 30 issues from review board this week and
> >> >as community we responded to 2/3 rd of them which is good but can be
> >better.
> >> >We were able to knock off only a handful of them. Out of the ones
> >> >that were responded to many of them are awaiting action from
> >> >reviewers. If you were called out please respond to your reviews and
> >> >help us close pending
> >> reviews.
> >> >
> >> >We have 33 pending reviews in review board at this time
> >> >
> >> >Animesh
> >> >
> >> >
> >> >
> >> >> -----Original Message-----
> >> >> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> >> >> Sent: Monday, February 11, 2013 11:24 PM
> >> >> To: cloudstack-dev@incubator.apache.org
> >> >> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
> >> >> committers and contributors.
> >> >>
> >> >> Folks
> >> >>
> >> >> I have sent a number of reminder emails for some of the review
> >> >> board items that have been pending for a while. Please attend to
> >> >> them at the earliest . We are running at 50+ items in review board.
> >> >> If you are a committer I request you knock off 1-2 reviews per
> >> >> week
> >> >>
> >> >> Animesh
> >> >>
> >> >> > -----Original Message-----
> >> >> > From: Chip Childers [mailto:chip.childers@sungard.com]
> >> >> > Sent: Tuesday, January 29, 2013 9:18 AM
> >> >> > To: cloudstack-dev@incubator.apache.org
> >> >> > Subject: [ACS41] Yet another reviewboard reminder... for both
> >> >> > committers and contributors.
> >> >> >
> >> >> > Hi all (and specifically people that were listed as "maintainers"
> >> >> > of certain code modules),
> >> >> >
> >> >> > We need committer help in getting the community contributions
> >> >> > currently in reviewboard (as well as some that we expect to show
> >> >> > up in the next day or so) reviewed and committed as soon as
> possible.
> >> >> >
> >> >> > If you are a "maintainer", or simply feel that you have enough
> >> >> > knowledge to get through one or more of these reviews, PLEASE
> >> >> > take some time to help clear out the board.
> >> >> >
> >> >> > Based on our agreed upon schedule, we need everything into
> >> >> > master (or
> >> >> > javelin) by EOD Thursday.
> >> >> >
> >> >> > For those of you that have code you want to be reviewed, but
> >> >> > have not yet submitted, it needs to get in ASAP. Otherwise,
> >> >> > there's no chance that someone will have the time to review and
> >> >> > commit before we cut the
> >> >> release branch.
> >> >> >
> >> >> > -chip
>
RE: [ACS41] Yet another reviewboard reminder... for both committers and contributors.
Posted by "Kelcey Damage (BT)" <ke...@backbonetechnology.com>.
Thanks, you see, I'm not a committer for code, I have very little
programming experience other than scripts/automation.
I'm more a marketing/evangelist/teaching/support kind of guy.
So if asked to review code, I would not be a good choice. If asked to review
policy/procedure/compliance/design/architecture(non-code), than I am your
guy.
That brings me to a question, how does one use the review board? Is there a
how-to/guidelines wiki article you can point me too?
Thanks
-kelcey
>-----Original Message-----
>From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
>Sent: Friday, February 15, 2013 4:26 PM
>To: cloudstack-dev@incubator.apache.org
>Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>committers and contributors.
>
>Well the submitter of the review https://reviews.apache.org/r/9336/ listed
>you as one of the reviewers and I was calling out based on that.
>
>Ok so what should one do if someone lists you as a reviewer? If you cannot
or
>not qualified to review you can respond saying that you cannot do that
>review and submitter can ask another reviewer.
>
>I will update the review board guidelines.
>
>Animesh
>
>
>> -----Original Message-----
>> From: Kelcey Damage (BT) [mailto:kelcey@backbonetechnology.com]
>> Sent: Friday, February 15, 2013 3:05 PM
>> To: cloudstack-dev@incubator.apache.org
>> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>> committers and contributors.
>>
>> My name got attached to 1180, and I'm not sure why. I am not what you
>> would call capable of reviewing the submission other then from a
>> legal/compliance perspective.
>>
>> Thanks
>>
>> -Kelcey
>>
>> >-----Original Message-----
>> >From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
>> >Sent: Friday, February 15, 2013 12:05 PM
>> >To: cloudstack-dev@incubator.apache.org
>> >Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>> >committers and contributors.
>> >
>> >Folks
>> >
>> >I have sent emails on over 30 issues from review board this week and
>> >as community we responded to 2/3 rd of them which is good but can be
>better.
>> >We were able to knock off only a handful of them. Out of the ones
>> >that were responded to many of them are awaiting action from
>> >reviewers. If you were called out please respond to your reviews and
>> >help us close pending
>> reviews.
>> >
>> >We have 33 pending reviews in review board at this time
>> >
>> >Animesh
>> >
>> >
>> >
>> >> -----Original Message-----
>> >> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
>> >> Sent: Monday, February 11, 2013 11:24 PM
>> >> To: cloudstack-dev@incubator.apache.org
>> >> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>> >> committers and contributors.
>> >>
>> >> Folks
>> >>
>> >> I have sent a number of reminder emails for some of the review
>> >> board items that have been pending for a while. Please attend to
>> >> them at the earliest . We are running at 50+ items in review board.
>> >> If you are a committer I request you knock off 1-2 reviews per week
>> >>
>> >> Animesh
>> >>
>> >> > -----Original Message-----
>> >> > From: Chip Childers [mailto:chip.childers@sungard.com]
>> >> > Sent: Tuesday, January 29, 2013 9:18 AM
>> >> > To: cloudstack-dev@incubator.apache.org
>> >> > Subject: [ACS41] Yet another reviewboard reminder... for both
>> >> > committers and contributors.
>> >> >
>> >> > Hi all (and specifically people that were listed as "maintainers"
>> >> > of certain code modules),
>> >> >
>> >> > We need committer help in getting the community contributions
>> >> > currently in reviewboard (as well as some that we expect to show
>> >> > up in the next day or so) reviewed and committed as soon as
possible.
>> >> >
>> >> > If you are a "maintainer", or simply feel that you have enough
>> >> > knowledge to get through one or more of these reviews, PLEASE
>> >> > take some time to help clear out the board.
>> >> >
>> >> > Based on our agreed upon schedule, we need everything into master
>> >> > (or
>> >> > javelin) by EOD Thursday.
>> >> >
>> >> > For those of you that have code you want to be reviewed, but have
>> >> > not yet submitted, it needs to get in ASAP. Otherwise, there's
>> >> > no chance that someone will have the time to review and commit
>> >> > before we cut the
>> >> release branch.
>> >> >
>> >> > -chip
RE: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Animesh Chaturvedi <an...@citrix.com>.
Well the submitter of the review https://reviews.apache.org/r/9336/ listed you as one of the reviewers and I was calling out based on that.
Ok so what should one do if someone lists you as a reviewer? If you cannot or not qualified to review you can respond saying that you cannot do that review and submitter can ask another reviewer.
I will update the review board guidelines.
Animesh
> -----Original Message-----
> From: Kelcey Damage (BT) [mailto:kelcey@backbonetechnology.com]
> Sent: Friday, February 15, 2013 3:05 PM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [ACS41] Yet another reviewboard reminder... for both committers
> and contributors.
>
> My name got attached to 1180, and I'm not sure why. I am not what you would
> call capable of reviewing the submission other then from a legal/compliance
> perspective.
>
> Thanks
>
> -Kelcey
>
> >-----Original Message-----
> >From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> >Sent: Friday, February 15, 2013 12:05 PM
> >To: cloudstack-dev@incubator.apache.org
> >Subject: RE: [ACS41] Yet another reviewboard reminder... for both
> >committers and contributors.
> >
> >Folks
> >
> >I have sent emails on over 30 issues from review board this week and as
> >community we responded to 2/3 rd of them which is good but can be better.
> >We were able to knock off only a handful of them. Out of the ones that
> >were responded to many of them are awaiting action from reviewers. If
> >you were called out please respond to your reviews and help us close
> >pending
> reviews.
> >
> >We have 33 pending reviews in review board at this time
> >
> >Animesh
> >
> >
> >
> >> -----Original Message-----
> >> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> >> Sent: Monday, February 11, 2013 11:24 PM
> >> To: cloudstack-dev@incubator.apache.org
> >> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
> >> committers and contributors.
> >>
> >> Folks
> >>
> >> I have sent a number of reminder emails for some of the review board
> >> items that have been pending for a while. Please attend to them at
> >> the earliest . We are running at 50+ items in review board. If you
> >> are a committer I request you knock off 1-2 reviews per week
> >>
> >> Animesh
> >>
> >> > -----Original Message-----
> >> > From: Chip Childers [mailto:chip.childers@sungard.com]
> >> > Sent: Tuesday, January 29, 2013 9:18 AM
> >> > To: cloudstack-dev@incubator.apache.org
> >> > Subject: [ACS41] Yet another reviewboard reminder... for both
> >> > committers and contributors.
> >> >
> >> > Hi all (and specifically people that were listed as "maintainers"
> >> > of certain code modules),
> >> >
> >> > We need committer help in getting the community contributions
> >> > currently in reviewboard (as well as some that we expect to show up
> >> > in the next day or so) reviewed and committed as soon as possible.
> >> >
> >> > If you are a "maintainer", or simply feel that you have enough
> >> > knowledge to get through one or more of these reviews, PLEASE take
> >> > some time to help clear out the board.
> >> >
> >> > Based on our agreed upon schedule, we need everything into master
> >> > (or
> >> > javelin) by EOD Thursday.
> >> >
> >> > For those of you that have code you want to be reviewed, but have
> >> > not yet submitted, it needs to get in ASAP. Otherwise, there's no
> >> > chance that someone will have the time to review and commit before
> >> > we cut the
> >> release branch.
> >> >
> >> > -chip
RE: [ACS41] Yet another reviewboard reminder... for both committers and contributors.
Posted by "Kelcey Damage (BT)" <ke...@backbonetechnology.com>.
My name got attached to 1180, and I'm not sure why. I am not what you would
call capable of reviewing the submission other then from a legal/compliance
perspective.
Thanks
-Kelcey
>-----Original Message-----
>From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
>Sent: Friday, February 15, 2013 12:05 PM
>To: cloudstack-dev@incubator.apache.org
>Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>committers and contributors.
>
>Folks
>
>I have sent emails on over 30 issues from review board this week and as
>community we responded to 2/3 rd of them which is good but can be better.
>We were able to knock off only a handful of them. Out of the ones that were
>responded to many of them are awaiting action from reviewers. If you were
>called out please respond to your reviews and help us close pending
reviews.
>
>We have 33 pending reviews in review board at this time
>
>Animesh
>
>
>
>> -----Original Message-----
>> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
>> Sent: Monday, February 11, 2013 11:24 PM
>> To: cloudstack-dev@incubator.apache.org
>> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>> committers and contributors.
>>
>> Folks
>>
>> I have sent a number of reminder emails for some of the review board
>> items that have been pending for a while. Please attend to them at the
>> earliest . We are running at 50+ items in review board. If you are a
>> committer I request you knock off 1-2 reviews per week
>>
>> Animesh
>>
>> > -----Original Message-----
>> > From: Chip Childers [mailto:chip.childers@sungard.com]
>> > Sent: Tuesday, January 29, 2013 9:18 AM
>> > To: cloudstack-dev@incubator.apache.org
>> > Subject: [ACS41] Yet another reviewboard reminder... for both
>> > committers and contributors.
>> >
>> > Hi all (and specifically people that were listed as "maintainers" of
>> > certain code modules),
>> >
>> > We need committer help in getting the community contributions
>> > currently in reviewboard (as well as some that we expect to show up
>> > in the next day or so) reviewed and committed as soon as possible.
>> >
>> > If you are a "maintainer", or simply feel that you have enough
>> > knowledge to get through one or more of these reviews, PLEASE take
>> > some time to help clear out the board.
>> >
>> > Based on our agreed upon schedule, we need everything into master
>> > (or
>> > javelin) by EOD Thursday.
>> >
>> > For those of you that have code you want to be reviewed, but have
>> > not yet submitted, it needs to get in ASAP. Otherwise, there's no
>> > chance that someone will have the time to review and commit before
>> > we cut the
>> release branch.
>> >
>> > -chip
RE: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Animesh Chaturvedi <an...@citrix.com>.
Folks
I have sent emails on over 30 issues from review board this week and as community we responded to 2/3 rd of them which is good but can be better. We were able to knock off only a handful of them. Out of the ones that were responded to many of them are awaiting action from reviewers. If you were called out please respond to your reviews and help us close pending reviews.
We have 33 pending reviews in review board at this time
Animesh
> -----Original Message-----
> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> Sent: Monday, February 11, 2013 11:24 PM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [ACS41] Yet another reviewboard reminder... for both committers
> and contributors.
>
> Folks
>
> I have sent a number of reminder emails for some of the review board items
> that have been pending for a while. Please attend to them at the earliest . We
> are running at 50+ items in review board. If you are a committer I request you
> knock off 1-2 reviews per week
>
> Animesh
>
> > -----Original Message-----
> > From: Chip Childers [mailto:chip.childers@sungard.com]
> > Sent: Tuesday, January 29, 2013 9:18 AM
> > To: cloudstack-dev@incubator.apache.org
> > Subject: [ACS41] Yet another reviewboard reminder... for both
> > committers and contributors.
> >
> > Hi all (and specifically people that were listed as "maintainers" of
> > certain code modules),
> >
> > We need committer help in getting the community contributions
> > currently in reviewboard (as well as some that we expect to show up in
> > the next day or so) reviewed and committed as soon as possible.
> >
> > If you are a "maintainer", or simply feel that you have enough
> > knowledge to get through one or more of these reviews, PLEASE take
> > some time to help clear out the board.
> >
> > Based on our agreed upon schedule, we need everything into master (or
> > javelin) by EOD Thursday.
> >
> > For those of you that have code you want to be reviewed, but have not
> > yet submitted, it needs to get in ASAP. Otherwise, there's no chance
> > that someone will have the time to review and commit before we cut the
> release branch.
> >
> > -chip
Re: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Chip Childers <ch...@sungard.com>.
On Mon, Feb 11, 2013 at 11:24:14PM -0800, Animesh Chaturvedi wrote:
> Folks
>
> I have sent a number of reminder emails for some of the review board items that have been pending for a while. Please attend to them at the earliest . We are running at 50+ items in review board. If you are a committer I request you knock off 1-2 reviews per week
>
> Animesh
Many thanks for wrangling reviewboard Animesh!
-chip
RE: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Animesh Chaturvedi <an...@citrix.com>.
> -----Original Message-----
> From: Joe Brockmeier [mailto:jzb@zonker.net]
> Sent: Wednesday, February 13, 2013 11:35 AM
> To: cloudstack-dev@incubator.apache.org
> Subject: Re: [ACS41] Yet another reviewboard reminder... for both committers
> and contributors.
>
> On Wed, Feb 13, 2013, at 12:25 AM, Animesh Chaturvedi wrote:
> > Joe https://reviews.apache.org/r/8180/
>
> This one should actually be taken off the list for now - apparently it includes a
> number of features that are either not in master or being held for 4.2. In fact,
> I'm going to see about having this broken out into more manageable patches,
> one per feature documented rather than bundling several features that aren't
> all coming in together.
[Animesh>] Yes agreed, Radhika can you take care of this?
>
> Best,
>
> jzb
> --
> Joe Brockmeier
> jzb@zonker.net
> Twitter: @jzb
> http://www.dissociatedpress.net/
Re: [ACS41] Yet another reviewboard reminder... for both committers and
contributors.
Posted by Joe Brockmeier <jz...@zonker.net>.
On Wed, Feb 13, 2013, at 12:25 AM, Animesh Chaturvedi wrote:
> Joe https://reviews.apache.org/r/8180/
This one should actually be taken off the list for now - apparently it
includes a number of features that are either not in master or being
held for 4.2. In fact, I'm going to see about having this broken out
into more manageable patches, one per feature documented rather than
bundling several features that aren't all coming in together.
Best,
jzb
--
Joe Brockmeier
jzb@zonker.net
Twitter: @jzb
http://www.dissociatedpress.net/
Re: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Min Chen <mi...@citrix.com>.
Committed the patch https://reviews.apache.org/r/9110/
Thanks
-min
On 2/12/13 10:25 PM, "Animesh Chaturvedi" <an...@citrix.com>
wrote:
>Prachi, Min, Rohit, Sebastien, Kelcey, Edison thanks for clearing up the
>review board we are now down to 40. In case I missed someone it's my
>oversight.
>
>Reviewer please commit the following patches that are marked "Ship It"
>and move them to submitted
>
>Edison : https://reviews.apache.org/r/9384/
>John Kinsella https://reviews.apache.org/r/8876/
>Min Chen https://reviews.apache.org/r/9110/
>Joe https://reviews.apache.org/r/8180/
>Nitin https://reviews.apache.org/r/7018/
>
>Committers please attend to review board, we certainly appreciate your
>time and effort. Contributors please respond to any queries on your
>patches timely to help us clear the review board
>
>
>> -----Original Message-----
>> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
>> Sent: Monday, February 11, 2013 11:24 PM
>> To: cloudstack-dev@incubator.apache.org
>> Subject: RE: [ACS41] Yet another reviewboard reminder... for both
>>committers
>> and contributors.
>>
>> Folks
>>
>> I have sent a number of reminder emails for some of the review board
>>items
>> that have been pending for a while. Please attend to them at the
>>earliest . We
>> are running at 50+ items in review board. If you are a committer I
>>request you
>> knock off 1-2 reviews per week
>>
>> Animesh
>>
>> > -----Original Message-----
>> > From: Chip Childers [mailto:chip.childers@sungard.com]
>> > Sent: Tuesday, January 29, 2013 9:18 AM
>> > To: cloudstack-dev@incubator.apache.org
>> > Subject: [ACS41] Yet another reviewboard reminder... for both
>> > committers and contributors.
>> >
>> > Hi all (and specifically people that were listed as "maintainers" of
>> > certain code modules),
>> >
>> > We need committer help in getting the community contributions
>> > currently in reviewboard (as well as some that we expect to show up in
>> > the next day or so) reviewed and committed as soon as possible.
>> >
>> > If you are a "maintainer", or simply feel that you have enough
>> > knowledge to get through one or more of these reviews, PLEASE take
>> > some time to help clear out the board.
>> >
>> > Based on our agreed upon schedule, we need everything into master (or
>> > javelin) by EOD Thursday.
>> >
>> > For those of you that have code you want to be reviewed, but have not
>> > yet submitted, it needs to get in ASAP. Otherwise, there's no chance
>> > that someone will have the time to review and commit before we cut the
>> release branch.
>> >
>> > -chip
RE: [ACS41] Yet another reviewboard reminder... for both committers
and contributors.
Posted by Animesh Chaturvedi <an...@citrix.com>.
Prachi, Min, Rohit, Sebastien, Kelcey, Edison thanks for clearing up the review board we are now down to 40. In case I missed someone it's my oversight.
Reviewer please commit the following patches that are marked "Ship It" and move them to submitted
Edison : https://reviews.apache.org/r/9384/
John Kinsella https://reviews.apache.org/r/8876/
Min Chen https://reviews.apache.org/r/9110/
Joe https://reviews.apache.org/r/8180/
Nitin https://reviews.apache.org/r/7018/
Committers please attend to review board, we certainly appreciate your time and effort. Contributors please respond to any queries on your patches timely to help us clear the review board
> -----Original Message-----
> From: Animesh Chaturvedi [mailto:animesh.chaturvedi@citrix.com]
> Sent: Monday, February 11, 2013 11:24 PM
> To: cloudstack-dev@incubator.apache.org
> Subject: RE: [ACS41] Yet another reviewboard reminder... for both committers
> and contributors.
>
> Folks
>
> I have sent a number of reminder emails for some of the review board items
> that have been pending for a while. Please attend to them at the earliest . We
> are running at 50+ items in review board. If you are a committer I request you
> knock off 1-2 reviews per week
>
> Animesh
>
> > -----Original Message-----
> > From: Chip Childers [mailto:chip.childers@sungard.com]
> > Sent: Tuesday, January 29, 2013 9:18 AM
> > To: cloudstack-dev@incubator.apache.org
> > Subject: [ACS41] Yet another reviewboard reminder... for both
> > committers and contributors.
> >
> > Hi all (and specifically people that were listed as "maintainers" of
> > certain code modules),
> >
> > We need committer help in getting the community contributions
> > currently in reviewboard (as well as some that we expect to show up in
> > the next day or so) reviewed and committed as soon as possible.
> >
> > If you are a "maintainer", or simply feel that you have enough
> > knowledge to get through one or more of these reviews, PLEASE take
> > some time to help clear out the board.
> >
> > Based on our agreed upon schedule, we need everything into master (or
> > javelin) by EOD Thursday.
> >
> > For those of you that have code you want to be reviewed, but have not
> > yet submitted, it needs to get in ASAP. Otherwise, there's no chance
> > that someone will have the time to review and commit before we cut the
> release branch.
> >
> > -chip