Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

(Review Priorities)
Line 21: Line 21:
  
 
Crypto:
 
Crypto:
     <strike>https://review.openstack.org/291458 Changes crypto to use transient-sysmeta for crypto-meta  <-- NEXT TO MERGE</strike>
+
     <strike>https://review.openstack.org/237393/  Support for http footers - Replication and EC</strike>
     https://review.openstack.org/237393Support for http footers - Replication and EC <---- needs feedback - clean-up work to be done
+
     https://review.openstack.org/303027/ Change fetch keys to use callback namespace. <-- should be fast
     https://review.openstack.org/257603/  Conditional GETs Fix   <---- depends on having real footers
+
    (need to rebase most of the below on latest - I believe there are no more dependencies)
     https://review.openstack.org/275492/  Encrypt/decrypt etag in container listing <---- depends on real footers
+
     https://review.openstack.org/257603/  Conditional GETs Fix
     https://review.openstack.org/298063/  Resolve todos relying on real footers and transient-sysmeta cleanup   <-- depends on real footers
+
     https://review.openstack.org/275492/  Encrypt/decrypt etag in container listing
 +
     https://review.openstack.org/298063/  Resolve todos relying on real footers and transient-sysmeta cleanup
 
     https://review.openstack.org/216974/  Encrypt flag for probe tests
 
     https://review.openstack.org/216974/  Encrypt flag for probe tests
 
     https://review.openstack.org/220897/  Probe tests for object encryption
 
     https://review.openstack.org/220897/  Probe tests for object encryption
 +
    https://review.openstack.org/302868/  Wrap and call existing update_footers callbacks
 
     https://review.openstack.org/274530/  POST update to round-trip unit tests
 
     https://review.openstack.org/274530/  POST update to round-trip unit tests
     https://review.openstack.org/277103/  Protect against incomplete read <---- depends on real footers
+
     https://review.openstack.org/277103/  Protect against incomplete read
  
 
= Bug Triage Links =
 
= Bug Triage Links =

Revision as of 19:01, 7 April 2016

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

Review Priorities

Copy as Middleware:

   These two patches block crypto work and make swift simpler, to boot. The first one makes sure that the versioning
   feature doesn't rely on COPY functionality, and the second one moves the COPY implementation from a proxy
   controller and into middleware.
   
   - https://review.openstack.org/260179 Decouple versioned writes from COPY
   - https://review.openstack.org/156923 Refactor server side copy as middleware

Features/bug-fixes in one patch:

   fallocate as a percent: https://review.openstack.org/#/c/288011/
   ionice https://review.openstack.org/#/c/238799/

Container Sync:

   per container stat report https://review.openstack.org/#/c/281814/
   process level concurrency https://review.openstack.org/#/c/210099/
   thread level concurrency https://review.openstack.org/#/c/225338/

Crypto:

   https://review.openstack.org/237393/  Support for http footers - Replication and EC
   https://review.openstack.org/303027/ Change fetch keys to use callback namespace.  <-- should be fast
   (need to rebase most of the below on latest - I believe there are no more dependencies)
   https://review.openstack.org/257603/  Conditional GETs Fix
   https://review.openstack.org/275492/  Encrypt/decrypt etag in container listing
   https://review.openstack.org/298063/  Resolve todos relying on real footers and transient-sysmeta cleanup
   https://review.openstack.org/216974/  Encrypt flag for probe tests
   https://review.openstack.org/220897/  Probe tests for object encryption
   https://review.openstack.org/302868/  Wrap and call existing update_footers callbacks
   https://review.openstack.org/274530/  POST update to round-trip unit tests
   https://review.openstack.org/277103/  Protect against incomplete read

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