Jump to: navigation, search

Difference between revisions of "Neutron/ReviewDays"

(→‎Schedules)
 
(53 intermediate revisions by 15 users not shown)
Line 1: Line 1:
−
__NOTOC__
 
−
We are thinking about having an assigned reviewer for each day.
 
  
−
Review duty rotates between [https://launchpad.net/~quantum-core/+members#active every (human) member of quantum-core]. If you are the assigned reviewer, it's your responsibility to: /* start */
+
----
−
* Set blueprint proposals to "Approved" if appropriate, or request more information.
+
As we enter the Havana release cycle, we are also back to normal with review days. All quantum core devs are therefore invited to sign up in the list below.
 +
 
 +
----
 +
Review duty rotates between [https://launchpad.net/~quantum-core/+members#active every (human) member of quantum-core]. If you are the assigned reviewer, it's your responsibility to:  
 
* Review patches.
 
* Review patches.
−
* Assign domain experts to patches withouth a review. In this case the domain expert might be or might not be a core dev. In the latter case a core dev should probably be selected as well for the branch.
+
* Assign domain experts to patches withouth a review. In this case the domain expert might be or might not be a core dev. In the latter case a core dev should probably be selected as well for the branch.  
 
* Respond to queries from contributors concerning pending reviews or blueprint approvals.
 
* Respond to queries from contributors concerning pending reviews or blueprint approvals.
 
* Pester assigned reviewers to review branches.
 
* Pester assigned reviewers to review branches.
 +
* Assign new bug report to domain experts, or triage then if you happen to be an expert for the domain the bug report refers to.
 +
* Answer questions on Launchpad or on the Openstack-dev mailing list if they are pertinent to your area of expertise (Actually core devs are invited to do so not just on their review day!)
  
−
/* end */
+
Active reviews for both the server and client/CLI repos:
  
−
Active reviews for trunk: https://review.openstack.org/#q,status:open+project:openstack/quantum,n,z
+
* https://review.openstack.org/#q,status:open+project:openstack/quantum,n,z
 +
* https://review.openstack.org/#/q/status:open+project:openstack/python-quantumclient,n,z
  
 
Every human member of quantum-core should be part of this rotation. As we have core developers in several we will try to leverage this as much as possible. For this reason, the table below has 3 columns corresponding to 3 different time "areas": Americas, EMEA, and APAC.
 
Every human member of quantum-core should be part of this rotation. As we have core developers in several we will try to leverage this as much as possible. For this reason, the table below has 3 columns corresponding to 3 different time "areas": Americas, EMEA, and APAC.
  
−
Approximately a core reviewer would be available in the following hours: /* start */
+
== Schedules ==
−
* APAC: 1AM GMT - 9AM GMT
 
−
* EMEA: 9AM GMT - 5PM GMT
 
−
* Americas: 5PM GMT - 1AM GMT
 
  
−
/* end */
+
Approximately a core reviewer would be available in the following hours:
 +
* APAC: 0100 - 0900 UTC
 +
* EMEA: 0900 - 1700 UTC
 +
* Americas: 1700 - 0100 UTC
  
−
''Folsom RC Special Review Guidelines''
+
See [http://www.openstack.org/community/members/ OpenStack Member Directory] for mappings between real names and IRC handles.
−
 
+
{| class="wikitable"
−
* only spend core review cycles on items that are assigned to Folsom RC1: https://launchpad.net/quantum/+milestone/folsom-rc1
+
|-
−
* sign up for two review days a week.  The goal is two have two core devs active on each day to improve round-trip time on reviews. Reviews should be smaller during this period.
+
!colspan="2" |'''Day'''
−
* on your review day, also triage any new quantum bugs to identify any that may need to apply to RC1.  Focus on triaging all bugs in 'New' state, moving them to 'confirmed', 'invalid', or 'incomplete', and assigning them a priority.  See [[BugTriage]] for more details.  If anything is unclear discuss it with PTL.
+
!Americas
−
* send update to to the core team at the start of your review day (describing your plans), and at the end of your review day (describing what you accomplished). Please use subject "Quantum Folsom RC-1" and the launchpad link for contacting quantum-core:https://launchpad.net/~quantum-core/+contactuser
+
!EMEA
−
 
+
!APAC
−
See [[People]] for mappings between real names and IRC handles.
+
|-
−
{| border="1" cellpadding="2" cellspacing="0"
+
| Monday ||  April 29th || || ||
 +
|-
 +
| Tuesday ||  April 30th || Mark McClain || Gary Kotton || 
 +
|-
 +
| Wednesday || May 1st || Dan Wendlandt || ||
 +
|-
 +
| Thursday || May 2nd || Edgar Magana || Gary Kotton ||     
 
|-
 
|-
−
| '''Day'''
+
| Friday || May 3rd || Mark McClain,Sumit Naiksatam  || Salvatore Orlando || Akihiro Motoki
−
| Americas
 
−
| EMEA
 
 
|-
 
|-
−
| Tuesday August 28th
+
| Saturday || May 4th || || ||
−
| Mark [[McClain]]
 
−
| Gary Kotton
 
 
|-
 
|-
−
|<style="text-align:center" |2>|Wednesday August 29th
+
| Sunday || May 5th || || Gary Kotton ||
−
|<style="text-align:center" |2>|Edgar Magana
 
−
| Maru Newby
 
 
|-
 
|-
 +
| Monday ||  May 6th || Bob Kukura || ||  Akihiro Motoki
 
|-
 
|-
−
|<style="text-align:center" |2>|Thursday August 30th
+
| Tuesday || May 7th || Mark McClain || Gary Kotton || 
−
|<style="text-align:center" |2>|Dan Wendlandt
 
−
| Salvatore Orlando
 
 
|-
 
|-
 +
| Wednesday || May 8th || Dan Wendlandt || ||
 
|-
 
|-
−
| Friday August 31st
+
| Thursday || May 9th || Edgar Magana || Gary Kotton ||     
−
| Sumit Naiksatam
 
−
| Salvatore Orlando
 
 
|-
 
|-
−
| Saturday September 1st
+
| Friday || May 10th || Mark McClain,Sumit Naiksatam  || Salvatore Orlando || Akihiro Motoki
−
| Mark [[McClain]]
 
−
| Somik B
 
 
|-
 
|-
−
| Sunday September 2nd
+
| Saturday || May 11th || || ||
−
| Dan Wendlandt
 
−
| Gary Kotton
 
 
|-
 
|-
−
|<style="text-align:center" |2>|Monday September 3rd
+
| Sunday || May 12th || || Gary Kotton ||
−
|<style="text-align:center" |2>|Somik Behera
 
−
| Gary Kotton  
 
 
|-
 
|-
 +
!colspan="2" <#88FF88>| '''Weekly Defaults'''
 +
!<#88FF88>|Americas
 +
!<#88FF88>|EMEA
 +
!<#88FF88>|APAC
 
|-
 
|-
−
| Tuesday September 4th
+
| Monday || || Bob Kukura, Nachi Ueno || ||  Akihiro Motoki
−
| Mark [[McClain]]
 
−
| Salvatore Orlando
 
 
|-
 
|-
−
| Wednesday September 5th
+
| Tuesday || || Mark McClain || Gary Kotton || Yong Sheng Gong(irc:gongysh)
−
| Edgar Magana
 
−
| Gary Kotton  
 
 
|-
 
|-
−
| Thursday September 6th
+
| Wednesday || || Dan Wendlandt, Nachi Ueno || ||
−
| Dan Wendlandt  
 
−
| Salvatore Orlando
 
 
|-
 
|-
−
|<style="text-align:center" |2>|Friday September 7th
+
| Thursday || || Edgar Magana || Gary Kotton || Yong Sheng Gong(irc:gongysh)
−
| Sumit Naiksatam
 
−
|<style="text-align:center" |2>|Maru Newby
 
 
|-
 
|-
 +
| Friday || || Mark McClain,Sumit Naiksatam || Salvatore Orlando || Akihiro Motoki
 
|-
 
|-
−
| Saturday September 8th
+
| Saturday || || || ||
−
|
 
−
| Salvatore Orlando
 
 
|-
 
|-
−
| Sunday September 9th
+
| Sunday || || || Gary Kotton ||
−
| Dan Wendlandt
 
−
| Gary Kotton  
 
 
|}
 
|}

Latest revision as of 00:29, 2 July 2013

As we enter the Havana release cycle, we are also back to normal with review days. All quantum core devs are therefore invited to sign up in the list below.

Review duty rotates between every (human) member of quantum-core. If you are the assigned reviewer, it's your responsibility to:

  • Review patches.
  • Assign domain experts to patches withouth a review. In this case the domain expert might be or might not be a core dev. In the latter case a core dev should probably be selected as well for the branch.
  • Respond to queries from contributors concerning pending reviews or blueprint approvals.
  • Pester assigned reviewers to review branches.
  • Assign new bug report to domain experts, or triage then if you happen to be an expert for the domain the bug report refers to.
  • Answer questions on Launchpad or on the Openstack-dev mailing list if they are pertinent to your area of expertise (Actually core devs are invited to do so not just on their review day!)

Active reviews for both the server and client/CLI repos:

Every human member of quantum-core should be part of this rotation. As we have core developers in several we will try to leverage this as much as possible. For this reason, the table below has 3 columns corresponding to 3 different time "areas": Americas, EMEA, and APAC.

Schedules

Approximately a core reviewer would be available in the following hours:

  • APAC: 0100 - 0900 UTC
  • EMEA: 0900 - 1700 UTC
  • Americas: 1700 - 0100 UTC

See OpenStack Member Directory for mappings between real names and IRC handles.

Day Americas EMEA APAC
Monday April 29th
Tuesday April 30th Mark McClain Gary Kotton
Wednesday May 1st Dan Wendlandt
Thursday May 2nd Edgar Magana Gary Kotton
Friday May 3rd Mark McClain,Sumit Naiksatam Salvatore Orlando Akihiro Motoki
Saturday May 4th
Sunday May 5th Gary Kotton
Monday May 6th Bob Kukura Akihiro Motoki
Tuesday May 7th Mark McClain Gary Kotton
Wednesday May 8th Dan Wendlandt
Thursday May 9th Edgar Magana Gary Kotton
Friday May 10th Mark McClain,Sumit Naiksatam Salvatore Orlando Akihiro Motoki
Saturday May 11th
Sunday May 12th Gary Kotton
Weekly Defaults Americas EMEA APAC
Monday Bob Kukura, Nachi Ueno Akihiro Motoki
Tuesday Mark McClain Gary Kotton Yong Sheng Gong(irc:gongysh)
Wednesday Dan Wendlandt, Nachi Ueno
Thursday Edgar Magana Gary Kotton Yong Sheng Gong(irc:gongysh)
Friday Mark McClain,Sumit Naiksatam Salvatore Orlando Akihiro Motoki
Saturday
Sunday Gary Kotton
Retrieved from "https://wiki.openstack.org/w/index.php?title=Neutron/ReviewDays&oldid=25245"