Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

Line 2: Line 2:
  
 
= Review Priorities =
 
= Review Priorities =
 +
 +
FIX PROBETESTS!!!
 +
    https://review.openstack.org/#/c/274840/ broke proble tests. There are two open patch that promise to fix it.
 +
    Land one or both of
 +
      - https://review.openstack.org/#/c/313150/
 +
      - https://review.openstack.org/#/c/311899/
  
 
Copy as Middleware:
 
Copy as Middleware:
     This patch blocks crypto work and make swift simpler, to boot. It moves the COPY implementation from a proxy
+
     This patch blocks crypto work and make swift simpler, to boot.
     controller and into middleware.
+
     https://review.openstack.org/#/c/156923/
   
 
    - https://review.openstack.org/260179 Decouple versioned writes from COPY
 
  
 
Features/bug-fixes in one patch:
 
Features/bug-fixes in one patch:
     fallocate as a percent: https://review.openstack.org/#/c/288011/
+
     fallocate as a percent: https://review.openstack.org/#/c/288011/ <-- needs 1 more +2
     ionice https://review.openstack.org/#/c/238799/
+
     ionice https://review.openstack.org/#/c/238799/ <-- needs 1 more +2
  
 
Container Sync:
 
Container Sync:
    per container stat report https://review.openstack.org/#/c/281814/
 
 
     process level concurrency https://review.openstack.org/#/c/210099/
 
     process level concurrency https://review.openstack.org/#/c/210099/
 
     thread level concurrency https://review.openstack.org/#/c/225338/
 
     thread level concurrency https://review.openstack.org/#/c/225338/
Line 20: Line 23:
 
Crypto :
 
Crypto :
 
   These reviews are highest priority, in addition to the copy middleware review listed above:
 
   These reviews are highest priority, in addition to the copy middleware review listed above:
    https://review.openstack.org/305794 crypto - only set body crypto meta if body is read and encrypted
 
 
     https://review.openstack.org/#/c/307121  crypto - add functional test to verify container listing detail
 
     https://review.openstack.org/#/c/307121  crypto - add functional test to verify container listing detail
  

Revision as of 17:20, 9 May 2016

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

Review Priorities

FIX PROBETESTS!!!

   https://review.openstack.org/#/c/274840/ broke proble tests. There are two open patch that promise to fix it.
   Land one or both of
     - https://review.openstack.org/#/c/313150/
     - https://review.openstack.org/#/c/311899/

Copy as Middleware:

   This patch blocks crypto work and make swift simpler, to boot.
   https://review.openstack.org/#/c/156923/

Features/bug-fixes in one patch:

   fallocate as a percent: https://review.openstack.org/#/c/288011/  <-- needs 1 more +2
   ionice https://review.openstack.org/#/c/238799/  <-- needs 1 more +2

Container Sync:

   process level concurrency https://review.openstack.org/#/c/210099/
   thread level concurrency https://review.openstack.org/#/c/225338/

Crypto :

  These reviews are highest priority, in addition to the copy middleware review listed above:
   https://review.openstack.org/#/c/307121  crypto - add functional test to verify container listing detail
   Complete list of crypto reviews is here: https://review.openstack.org/#/q/status:open+project:openstack/swift+branch:feature/crypto

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 Review Dashboard

https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython%2Dswiftclient+OR+project%3Aopenstack%2Fswift%2Dpython%2Dagent+OR+project%3Aopenstack%2Fswift%2Dbench+OR+project%3Aopenstack%2Fswift%2Dspecs%29+status%3Aopen+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3ACode%2DReview%3C%3D%2D2+is%3Amergeable&title=Swift+Review+Dashboard&Starred+%28by+PTL+or+myself%29=%28starredby%3Anotmyname+OR+is%3Astarred%29+AND+status%3Aopen&Small+things=delta%3A%3C%3D25+limit%3A10+%28NOT+label%3ACode%2DReview%2D1%2Cswift%2Dcore+OR+%28label%3ACode%2DReview%2D1%2Cswiftclient%2Dcore+AND+project%3Aopenstack%2Fpython%2Dswiftclient%29%29&Needs+Final+Approval+%28to+land+on+master%29=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+owner%3Aself+label%3ACode%2DReview%3E%3D2+%28NOT+label%3ACode%2DReview%2D1%2Cswift%2Dcore+OR+%28label%3ACode%2DReview%2D1%2Cswiftclient%2Dcore+AND+project%3Aopenstack%2Fpython%2Dswiftclient%29%29+NOT+project%3Aopenstack%2Fswift%2Dspecs+branch%3Amaster&Open+Backport+Proposals=NOT+project%3Aopenstack%2Fswift%2Dspecs+branch%3A%5Estable%2F.%2A+status%3Aopen&Feature+Branches=NOT+project%3Aopenstack%2Fswift%2Dspecs+branch%3A%5Efeature%2F.%2A+status%3Aopen&Recently+Proposed+to+master=NOT+label%3AWorkflow%3E%3D1+%28NOT+label%3ACode%2DReview%2D1%2Cswift%2Dcore+OR+%28label%3ACode%2DReview%2D1%2Cswiftclient%2Dcore+AND+project%3Aopenstack%2Fpython%2Dswiftclient%29%29+label%3AVerified%3E%3D1%2Cjenkins+NOT+label%3ACode%2DReview%3E%3D2+NOT+project%3Aopenstack%2Fswift%2Dspecs+%2Dage%3A1week+branch%3Amaster&Older+open+patches=NOT+label%3AWorkflow%3E%3D1+NOT+label%3ACode%2DReview%3C%3D2+age%3A1week+NOT+project%3Aopenstack%2Fswift%2Dspecs+%28NOT+label%3ACode%2DReview%2D1%2Cswift%2Dcore+OR+%28label%3ACode%2DReview%2D1%2Cswiftclient%2Dcore+AND+project%3Aopenstack%2Fpython%2Dswiftclient%29%29&Swift+Specs=project%3Aopenstack%2Fswift%2Dspecs+NOT+label%3ACode%2DReview%3C%3D%2D2


https://goo.gl/mtEv1C




Swift Project Overview Dashboard

https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython%2Dswiftclient+OR+project%3Aopenstack%2Fswift%2Dbench+OR+project%3Aopenstack%2Fswift%2Dspecs%29+status%3Aopen+is%3Amergeable&title=Swift+Project+Dashboard&Starred+Patches+%28by+core%29=%28%28starredby%3Atorgomatic+OR+starredby%3Acschwede+OR+starredby%3A%22alistair.coles%40hpe.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+OR+starredby%3A%22tsuyuzaki.kota%40lab.ntt.co.jp%22+OR+starredby%3A%22thiago%40redhat.com%22%29+OR+%28%28starredby%3A%22joel.wright%40sohonet.com%22+OR+starredby%3A%22tim.burke%40gmail.com%22%29+AND+project%3Aopenstack%2Fpython%2Dswiftclient%29%29+AND+status%3Aopen&Needs+Final+Approval+%28to+land+on+master%29=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+owner%3Aself+label%3ACode%2DReview%3E%3D2+%28NOT+label%3ACode%2DReview%2D1%2Cswift%2Dcore+OR+%28label%3ACode%2DReview%2D1%2Cswiftclient%2Dcore+AND+project%3Aopenstack%2Fpython%2Dswiftclient%29%29+NOT+project%3Aopenstack%2Fswift%2Dspecs+branch%3Amaster&In+the+gate+queue=label%3AWorkflow%3E%3D1+NOT+label%3ACode%2DReview%3C%3D%2D2&Needs+Reverify=label%3AVerified%3C%3D%2D1%2Cjenkins+NOT+project%3Aopenstack%2Fswift%2Dspecs+branch%3Amaster+NOT+label%3ACode%2DReview%3C%3D%2D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3ACode%2DReview%3C%3D%2D2&Possible+abandoned+changes=status%3Aopen+age%3A4week+%28label%3AVerified%3C%3D%2D1%2Cjenkins+OR+label%3ACode%2DReview%3C%3D%2D1%29+NOT+label%3AWorkflow%3C%3D%2D1

https://goo.gl/DV5vkD