Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

(Big important topics right now)
Line 1: Line 1:
 
'''Swift Review Dashboard:''' https://goo.gl/eqeGwE
 
'''Swift Review Dashboard:''' https://goo.gl/eqeGwE
 
= Liberty patches: Done! =
 
 
{| class="wikitable"
 
|-
 
! Patch number !! Priority !! Patch owner !! Summary !! Status
 
|-
 
| <strike>https://review.openstack.org/#/c/227133/</strike> || <strike>C</strike> || <strike>Christian</strike> || <strike>Enforce service role validation</strike> ||  merged
 
|-
 
| https://review.openstack.org/#/c/222799 || <strike>C</strike> || <strike>Clay</strike> || <strike>Validate against duplicate part replica device assignment</strike> || merged
 
|-
 
| <strike>https://review.openstack.org/#/c/138498/</strike> || <strike>C</strike> || <strike>Alistair</strike> || <strike>Add POST capability to ssync for .meta files</strike> || merged
 
|-
 
| <strike>https://review.openstack.org/#/c/215276</strike> || <strike>C</strike> || <strike>Alistair</strike> || <strike>Enable object server to return non-durable data</strike> || has been (is being) triaged with other work. will not be in liberty
 
|-
 
| <strike>https://review.openstack.org/#/c/186735</strike> || <strike>C</strike> || <strike>Kota</strike> || <strike>Fix missing container update</strike> || merged
 
|-
 
| <strike>https://review.openstack.org/#/c/211338</strike> || <strike>C</strike> || <strike>Paul</strike> || <strike>Better error handling for EC PUT path when client goes away</strike> || merged
 
|-
 
| <strike>https://review.openstack.org/#/c/220059</strike> || <strike>C</strike> || <strike>Kota</strike> || <strike>Don't send commits for quorum BAD requests on EC</strike> || merged
 
|-
 
| <strike>https://review.openstack.org/#/c/211726/</strike> || <strike>M</strike> || <strike>Tim</strike> || <strike>Add the ability to specify ranges for SLO segments</strike> || merged
 
|-
 
| <strike>https://review.openstack.org/#/c/227569/</strike> || <strike>M</strike> || <strike>Tim</strike> || <strike>Fixes tests from SLORange patch</strike> || merged
 
|-
 
| https://review.openstack.org/#/c/196848 || L || Minwoo || Handoff node to push existing fragment to the correct location || reviewed by Christian
 
|}
 
  
 
= Big important topics right now =
 
= Big important topics right now =
Line 35: Line 8:
 
* global clusters improvements -- eg container sync (https://review.openstack.org/#/q/status:open+owner:%22Eran+Rom+%253Ceranr%2540il.ibm.com%253E%22+project:openstack/swift,n,z)
 
* global clusters improvements -- eg container sync (https://review.openstack.org/#/q/status:open+owner:%22Eran+Rom+%253Ceranr%2540il.ibm.com%253E%22+project:openstack/swift,n,z)
 
* container sharding -- https://review.openstack.org/#/c/218738/
 
* container sharding -- https://review.openstack.org/#/c/218738/
 
=== EC Bugs/Patches Prioritization for undecided/new (9/9/2015) ===
 
'''Updates to the logs should have '''low''' priority until post Liberty:'''
 
* Object server logs don't reflect reconstructor as source of query: https://bugs.launchpad.net/swift/+bug/1491871
 
* Reconstructor complains 404 is invalid responce when building objectL https://bugs.launchpad.net/swift/+bug/1491883
 
<br />
 
'''Should be higher than low:'''
 
*Proxy server ignores additional fragments on primary nodes: https://bugs.launchpad.net/swift/+bug/1484598
 
*Proxy server reports 202 even though EC backend didn't accept object (not just EC related?): https://bugs.launchpad.net/swift/+bug/1491908
 
<br />
 
'''This needs updates from the bug owners:'''
 
*ssync receiver get an odd SSYNC command and raise a exception: https://bugs.launchpad.net/swift/+bug/1468708
 
*Balance of EC ring seems odd compared to replica ring: https://bugs.launchpad.net/swift/+bug/1491669
 
  
 
= Bug Triage Links =
 
= Bug Triage Links =
Line 58: Line 18:
 
=== Swift Dashboard Long URL ===
 
=== Swift Dashboard Long URL ===
  
https://goo.gl/eqeGwE
+
https://goo.gl/iLt6Ld
  
 
(this expands to)
 
(this expands to)
  
https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython%252Dswiftclient+OR+project%3Aopenstack%2Fswift%252Dpython%252Dagent+OR+project%3Aopenstack%2Fswift%252Dbench+OR+project%3Aopenstack%2Fswift%252Dspecs%29+status%3Aopen+NOT+label%3AWorkflow%3C%3D%252D1+NOT+label%3ACode%252DReview%3C%3D%252D2&title=Swift+Review+Dashboard&Starred+%28by+PTL+or+myself%29=%28starredby%3Anotmyname+OR+is%3Astarred%29+AND+status%3Aopen&Swift+Specs=project%3Aopenstack%2Fswift%252Dspecs+NOT+label%3ACode%252DReview%3C%3D%252D2&Needs+Final+Approval+%28to+land+on+master%29=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+NOT+owner%3Aself+label%3ACode%252DReview%3E%3D2+NOT+label%3ACode%252DReview%252D1+NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Amaster&On%252DDisk+Encryption+work+%28feature%2Fcrypto+branch%29=NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Afeature%2Fcrypto+status%3Aopen&Recently+Proposed+to+master=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+label%3AVerified%3E%3D1%252cjenkins+NOT+label%3ACode%252DReview%3E%3D2+NOT+project%3Aopenstack%2Fswift%252Dspecs+%252Dage%3A5d&Older+open+patches=NOT+label%3AWorkflow%3E%3D1+NOT+label%3ACode%252DReview%3C%3D2+age%3A5d+%252Dage%3A6week+NOT+project%3Aopenstack%2Fswift%252Dspecs
+
https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython%252Dswiftclient+OR+project%3Aopenstack%2Fswift%252Dpython%252Dagent+OR+project%3Aopenstack%2Fswift%252Dbench+OR+project%3Aopenstack%2Fswift%252Dspecs%29+status%3Aopen+NOT+label%3AWorkflow%3C%3D%252D1+NOT+label%3ACode%252DReview%3C%3D%252D2&title=Swift+Review+Dashboard&Starred+%28by+PTL+or+myself%29=%28starredby%3Anotmyname+OR+is%3Astarred%29+AND+status%3Aopen&Swift+Specs=project%3Aopenstack%2Fswift%252Dspecs+NOT+label%3ACode%252DReview%3C%3D%252D2&Needs+Final+Approval+%28to+land+on+master%29=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+NOT+owner%3Aself+label%3ACode%252DReview%3E%3D2+NOT+label%3ACode%252DReview%252D1+NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Amaster&On%252DDisk+Encryption+work+%28feature%2Fcrypto+branch%29=NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Afeature%2Fcrypto+status%3Aopen&Recently+Proposed+to+master=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+label%3AVerified%3E%3D1%252cjenkins+NOT+label%3ACode%252DReview%3E%3D2+NOT+project%3Aopenstack%2Fswift%252Dspecs+%252Dage%3A1week&Older+open+patches=NOT+label%3AWorkflow%3E%3D1+NOT+label%3ACode%252DReview%3C%3D2+age%3A1week+NOT+project%3Aopenstack%2Fswift%252Dspecs
  
  

Revision as of 21:20, 20 October 2015

Swift Review Dashboard: https://goo.gl/eqeGwE

Big important topics right now

Bug Triage Links

swiftclient bugs: https://goo.gl/uO4b7l <-- bugs that are marked "New" and need triage swift bugs: https://goo.gl/LMiiDc <-- bugs that are marked "New" and need triage


Swift Dashboard Long URL

https://goo.gl/iLt6Ld

(this expands to)

https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython%252Dswiftclient+OR+project%3Aopenstack%2Fswift%252Dpython%252Dagent+OR+project%3Aopenstack%2Fswift%252Dbench+OR+project%3Aopenstack%2Fswift%252Dspecs%29+status%3Aopen+NOT+label%3AWorkflow%3C%3D%252D1+NOT+label%3ACode%252DReview%3C%3D%252D2&title=Swift+Review+Dashboard&Starred+%28by+PTL+or+myself%29=%28starredby%3Anotmyname+OR+is%3Astarred%29+AND+status%3Aopen&Swift+Specs=project%3Aopenstack%2Fswift%252Dspecs+NOT+label%3ACode%252DReview%3C%3D%252D2&Needs+Final+Approval+%28to+land+on+master%29=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+NOT+owner%3Aself+label%3ACode%252DReview%3E%3D2+NOT+label%3ACode%252DReview%252D1+NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Amaster&On%252DDisk+Encryption+work+%28feature%2Fcrypto+branch%29=NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Afeature%2Fcrypto+status%3Aopen&Recently+Proposed+to+master=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+label%3AVerified%3E%3D1%252cjenkins+NOT+label%3ACode%252DReview%3E%3D2+NOT+project%3Aopenstack%2Fswift%252Dspecs+%252Dage%3A1week&Older+open+patches=NOT+label%3AWorkflow%3E%3D1+NOT+label%3ACode%252DReview%3C%3D2+age%3A1week+NOT+project%3Aopenstack%2Fswift%252Dspecs




Project overview dashboard: https://goo.gl/jL0byl

(this expands to)

https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython%252Dswiftclient+OR+project%3Aopenstack%2Fswift%252Dbench+OR+project%3Aopenstack%2Fswift%252Dspecs%29+status%3Aopen&title=Swift+Project+Dashboard&Starred+Patches+%28by+core%29=%28starredby%3A%22joel.wright%40sohonet.com%22+OR+%22tsuyuzaki.kota%40lab.ntt.co.jp%22+OR+starredby%3Atorgomatic+OR+starredby%3Acschwede+OR+starredby%3A%22alistair.coles%40hp.com%22+OR+starredby%3A%22darrell%40swiftstack.com%22+OR+starredby%3A%22david.goetz%40rackspace.com%22+OR+starredby%3A%22greglange%40gmail.com%22+OR+starredby%3A%22matt%40oliver.net.au%22+OR+starredby%3A%22mike%40weirdlooking.com%22+OR+starredby%3A%22zaitcev%40kotori.zaitcev.us%22+OR+starredby%3A%22paul.e.luse%40intel.com%22+OR+starredby%3Anotmyname%29+AND+status%3Aopen&Needs+Approval=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%252D1+label%3AVerified%3E%3D1%252cjenkins+label%3ACode%252DReview%3E%3D2+NOT+label%3ACode%252DReview%252D1+NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Amaster+NOT+label%3ACode%252DReview%3C%3D%252D2&In+the+gate+queue=label%3AWorkflow%3E%3D1+NOT+label%3ACode%252DReview%3C%3D%252D2&Needs+Reverify=label%3AVerified%3C%3D%252D1%252cjenkins+NOT+project%3Aopenstack%2Fswift%252Dspecs+branch%3Amaster+NOT+label%3ACode%252DReview%3C%3D%252D1+NOT+label%3AWorkflow%3C%3D%252D1+NOT+label%3ACode%252DReview%3C%3D%252D2&Possible+abandoned+changes=status%3Aopen+age%3A4week+%28label%3AVerified%3C%3D%252D1%252cjenkins+OR+label%3ACode%252DReview%3C%3D%252D1%29+NOT+label%3AWorkflow%3C%3D%252D1