You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by Justin Ross <jr...@redhat.com> on 2011/03/01 04:08:58 UTC

0.10 release update - this week we branch for release

Hi, folks.

Our schedule calls for us to create our release branch this Wednesday, 3 
March.  Trunk will remain open for forward development, but all changes to 
the release branch will require approval.

I'd like to enlist your help with the approval process.  When a potential 
fix comes in, it needs two forms of review:

   1. Review by someone who focuses on the release goals and schedule

   2. Review by someone (other than the author) who knows the code being
      patched and can speak to the change's significance and safety

My job is item 1.  For item 2, I need experts in the various components of 
qpid.

So here's my proposal: approval of kinds 1 (from me) and 2 (from someone 
familiar with the code being changed) must be indicated in jira comments 
before a change may be committed to the release branch.

In order to this efficiently, we need clear lines of responsibility.  I'd 
like to use the (still emerging) list of component owners to map incoming 
changes to reviewers.

In cases where the patch submitter is also the component owner, I'd ask 
that he or she find someone else with equivalent experience in that code 
to evaluate the change.

The purpose of requiring approval is to ensure that, as we approach 
release, each change is *discussed*.  I am continually impressed with how 
simple questions can lay bare important deficits.

I don't expect this to be a big burden, though I am conscious of the extra 
coordination involved.  I'll do my best to respond to change requests 
promptly.

Here's what I have so far for component owners:

   C++ clustering                Alan
   C++ client and broker         Gordon
   Java client                   Rajith
   Java broker                   Robbie
   QMF                           Ted

I just scanned that out of the "Default assignees" thread.  Please reply 
on this thread if you'd like to volunteer to review changes for a 
particular component.

Thanks,
Justin

---

https://cwiki.apache.org/qpid/010-release.html

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Re: 0.10 release update - this week we branch for release

Posted by Justin Ross <jr...@redhat.com>.
Andrew, I say fire away.

On Wed, 2 Mar 2011, Andrew Kennedy wrote:

> Justin,
>
> Also, I don't want this to interfere with the 0.10 release process, so let me 
> know if you feel I ought to wait a bit longer so that my review request 
> doesn't get lost in the noise of 0.10 release candidate bugfixes. As you can 
> perhaps tell, I'm just anxious to get the code out there ;)
>
> Andrew.
> -- 
> -- andrew d kennedy ? do not fold, bend, spindle, or mutilate ;
> -- http://grkvlt.blogspot.com/ ? edinburgh : +44 7582 293 255 ;
>
> On 1 Mar 2011, at 15:51, Justin Ross wrote:
>
>> It should trouble all of you that I'm no good with dates.  Thankfully, I 
>> have peers to point out my errors!  The date I intended was Wednesday, 2 
>> March, tomorrow--not 3 March.
>> 
>> We may push this date a little later in the week to allow time for some 
>> fixes to land, but so far it's still set for tomorrow.
>> 
>> Justin
>
> ---------------------------------------------------------------------
> Apache Qpid - AMQP Messaging Implementation
> Project:      http://qpid.apache.org
> Use/Interact: mailto:dev-subscribe@qpid.apache.org
>

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Re: 0.10 release update - this week we branch for release

Posted by Andrew Kennedy <an...@gmail.com>.
Justin,

Also, I don't want this to interfere with the 0.10 release process,  
so let me know if you feel I ought to wait a bit longer so that my  
review request doesn't get lost in the noise of 0.10 release  
candidate bugfixes. As you can perhaps tell, I'm just anxious to get  
the code out there ;)

Andrew.
-- 
-- andrew d kennedy ? do not fold, bend, spindle, or mutilate ;
-- http://grkvlt.blogspot.com/ ? edinburgh : +44 7582 293 255 ;

On 1 Mar 2011, at 15:51, Justin Ross wrote:

> It should trouble all of you that I'm no good with dates.   
> Thankfully, I have peers to point out my errors!  The date I  
> intended was Wednesday, 2 March, tomorrow--not 3 March.
>
> We may push this date a little later in the week to allow time for  
> some fixes to land, but so far it's still set for tomorrow.
>
> Justin

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Re: 0.10 release update - this week we branch for release

Posted by Justin Ross <jr...@redhat.com>.
It should trouble all of you that I'm no good with dates.  Thankfully, I 
have peers to point out my errors!  The date I intended was Wednesday, 2 
March, tomorrow--not 3 March.

We may push this date a little later in the week to allow time for some 
fixes to land, but so far it's still set for tomorrow.

Justin

On Mon, 28 Feb 2011, Justin Ross wrote:

> Hi, folks.
>
> Our schedule calls for us to create our release branch this Wednesday, 3 
> March.  Trunk will remain open for forward development, but all changes to 
> the release branch will require approval.
>
> I'd like to enlist your help with the approval process.  When a potential fix 
> comes in, it needs two forms of review:
>
>  1. Review by someone who focuses on the release goals and schedule
>
>  2. Review by someone (other than the author) who knows the code being
>     patched and can speak to the change's significance and safety
>
> My job is item 1.  For item 2, I need experts in the various components of 
> qpid.
>
> So here's my proposal: approval of kinds 1 (from me) and 2 (from someone 
> familiar with the code being changed) must be indicated in jira comments 
> before a change may be committed to the release branch.
>
> In order to this efficiently, we need clear lines of responsibility.  I'd 
> like to use the (still emerging) list of component owners to map incoming 
> changes to reviewers.
>
> In cases where the patch submitter is also the component owner, I'd ask that 
> he or she find someone else with equivalent experience in that code to 
> evaluate the change.
>
> The purpose of requiring approval is to ensure that, as we approach release, 
> each change is *discussed*.  I am continually impressed with how simple 
> questions can lay bare important deficits.
>
> I don't expect this to be a big burden, though I am conscious of the extra 
> coordination involved.  I'll do my best to respond to change requests 
> promptly.
>
> Here's what I have so far for component owners:
>
>  C++ clustering                Alan
>  C++ client and broker         Gordon
>  Java client                   Rajith
>  Java broker                   Robbie
>  QMF                           Ted
>
> I just scanned that out of the "Default assignees" thread.  Please reply on 
> this thread if you'd like to volunteer to review changes for a particular 
> component.
>
> Thanks,
> Justin
>
> ---
>
> https://cwiki.apache.org/qpid/010-release.html
>
> ---------------------------------------------------------------------
> Apache Qpid - AMQP Messaging Implementation
> Project:      http://qpid.apache.org
> Use/Interact: mailto:dev-subscribe@qpid.apache.org
>
>

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Re: 0.10 release update - this week we branch for release

Posted by Ken Giusti <kg...@redhat.com>.
Seeking approval for inclusion in 0.10:

https://issues.apache.org/jira/browse/QPID-3080
https://issues.apache.org/jira/browse/QPID-3081

Proposed patches attached to each.

thanks,

-K

----- Original Message -----
> Hi, folks.
> 
> Our schedule calls for us to create our release branch this Wednesday,
> 3
> March. Trunk will remain open for forward development, but all changes
> to
> the release branch will require approval.
> 
> I'd like to enlist your help with the approval process. When a
> potential
> fix comes in, it needs two forms of review:
> 
> 1. Review by someone who focuses on the release goals and schedule
> 
> 2. Review by someone (other than the author) who knows the code being
> patched and can speak to the change's significance and safety
> 
> My job is item 1. For item 2, I need experts in the various components
> of
> qpid.
> 
> So here's my proposal: approval of kinds 1 (from me) and 2 (from
> someone
> familiar with the code being changed) must be indicated in jira
> comments
> before a change may be committed to the release branch.
> 
> In order to this efficiently, we need clear lines of responsibility.
> I'd
> like to use the (still emerging) list of component owners to map
> incoming
> changes to reviewers.
> 
> In cases where the patch submitter is also the component owner, I'd
> ask
> that he or she find someone else with equivalent experience in that
> code
> to evaluate the change.
> 
> The purpose of requiring approval is to ensure that, as we approach
> release, each change is *discussed*. I am continually impressed with
> how
> simple questions can lay bare important deficits.
> 
> I don't expect this to be a big burden, though I am conscious of the
> extra
> coordination involved. I'll do my best to respond to change requests
> promptly.
> 
> Here's what I have so far for component owners:
> 
> C++ clustering Alan
> C++ client and broker Gordon
> Java client Rajith
> Java broker Robbie
> QMF Ted
> 
> I just scanned that out of the "Default assignees" thread. Please
> reply
> on this thread if you'd like to volunteer to review changes for a
> particular component.
> 
> Thanks,
> Justin
> 
> ---
> 
> https://cwiki.apache.org/qpid/010-release.html
> 
> ---------------------------------------------------------------------
> Apache Qpid - AMQP Messaging Implementation
> Project: http://qpid.apache.org
> Use/Interact: mailto:dev-subscribe@qpid.apache.org

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org