Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

m (→‎Review Priorities)
 
(328 intermediate revisions by 12 users not shown)
Line 1: Line 1:
−
'''Swift Review Dashboard:''' https://goo.gl/eqeGwE
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
  
−
= Review Priorities =
 
  
−
Copy as Middleware:
+
'''Client Patches'''
−
    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:
+
* <s>https://review.opendev.org/c/openstack/python-swiftclient/+/909789 | CI: constrain deps for tests
−
    fallocate as a percent: https://review.openstack.org/#/c/288011/
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/903770 | Add transaction id to errors
−
    ionice https://review.openstack.org/#/c/238799/
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/848926 | shell: Print friendly account byte quotas
 +
* https://review.opendev.org/c/openstack/python-swiftclient/+/909498 | lint: Up-rev hacking</s>
 +
* https://review.opendev.org/c/openstack/python-swiftclient/+/833954 | Add formpost subcommand to generate signature
  
−
Container Sync:
+
'''Swift Patches'''
−
    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:
+
* <s>https://review.opendev.org/c/openstack/swift/+/810754 | staticweb: Work with prefix-based tempurls</s>
−
    <strike>https://review.openstack.org/237393  Support for http footers - Replication and EC</strike>
+
* https://review.opendev.org/c/openstack/swift/+/907523 | drive-full-checker
−
    https://review.openstack.org/303027  Change fetch keys to use callback namespace. <-- should be fast
+
* expirer grace period patches
−
    https://review.openstack.org/257603  Conditional GETs Fix
+
** https://review.opendev.org/c/openstack/swift/+/874806 | Add per account grace period to object expirer
−
    https://review.openstack.org/275492  Encrypt/decrypt etag in container listing
+
** https://review.opendev.org/c/openstack/swift/+/874710 | Add x-open-expired to recover expired objects
−
    https://review.openstack.org/304066  crypto: use cipher required iv length for random iv
+
* part-number patches
−
    https://review.openstack.org/304163 crypto - cleanup unit tests
+
** https://review.opendev.org/c/openstack/swift/+/894570 | slo: part-number=N query parameter support
−
    https://review.openstack.org/304806  crypto - cleanup decrypter exception handling
+
** https://review.opendev.org/c/openstack/swift/+/894580 | s3api: Support GET/HEAD request with ?partNumber
−
    https://review.openstack.org/298063  Resolve todos relying on real footers and transient-sysmeta cleanup
+
** https://review.opendev.org/c/openstack/swift/+/906906 | s3api: return 400 if partNumber and Range sent
−
    https://review.openstack.org/216974  Encrypt flag for probe tests
+
** https://review.opendev.org/c/openstack/swift/+/907751 | WIP s3api: add more MPU cross-compat tests
−
    https://review.openstack.org/220897  Probe tests for object encryption
+
<!--
−
    <strike>https://review.openstack.org/302868  Wrap and call existing update_footers callbacks</strike>
+
* aws-chunked patches
−
    <strike>https://review.openstack.org/274530  POST update to round-trip unit tests</strike>
+
** https://review.opendev.org/c/openstack/swift/+/909049 | s3api: Improve checksum-mismatch detection
 +
** https://review.opendev.org/c/openstack/swift/+/909800 | utils: Add crc32c function
 +
** https://review.opendev.org/c/openstack/swift/+/909801 | s3api: Add support for additional checksums
 +
** https://review.opendev.org/c/openstack/swift/+/909802 | s3api: Additional checksums for MPUs
 +
** https://review.opendev.org/c/openstack/swift/+/836755 | Add support of Sigv4-streaming
 +
-->
 +
* <s>https://review.opendev.org/c/openstack/swift/+/890919 | proxy-server: de-duplicate _get_next_response_part method
 +
** https://review.opendev.org/c/openstack/swift/+/908287 | proxy: don't use recoverable_node_timeout with x-newest</s>
 +
* https://review.opendev.org/c/openstack/swift/+/840531 | backend ratelimit: support reloadable config file
 +
* https://review.opendev.org/c/openstack/swift/+/839649 | replicator: Add sync_batches_per_revert option
 +
* https://review.opendev.org/c/openstack/swift/+/909637 | SIGUSR2 signals print process info on wsgi servers
  
−
= Bug Triage Links =
+
<!--
 +
* https://review.opendev.org/c/openstack/swift/+/800701 - Delete s3api MPU segments when the manifest expires
 +
* https://review.opendev.org/c/openstack/swift/+/834261 - Introduce a v2 ring format
 +
-->
  
−
'''swiftclient bugs:''' https://goo.gl/uO4b7l  <-- bugs that are marked "New" and need triage
+
'''PyECLib/LibEC Patches'''
−
'''swift bugs:''' https://goo.gl/LMiiDc  <-- bugs that are marked "New" and need triage
+
 
 +
* https://review.openstack.org/#/q/project:openstack/liberasurecode+is:open
 +
* https://review.openstack.org/#/q/project:openstack/pyeclib+is:open
  
 
----
 
----
  
−
=== Swift Review Dashboard ===
+
'''Bug triage worksheet:''' https://etherpad.openstack.org/p/swift-bug-triage-list
  
 +
'''swiftclient bugs:''' https://goo.gl/uO4b7l  <-- bugs that are marked "New" and need triage
  
−
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
+
'''swift bugs:''' https://goo.gl/LMiiDc  <-- bugs that are marked "New" and need triage
  
 +
----
  
−
https://goo.gl/mtEv1C
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
−
 
 
−
 
 
−
 
 
−
 
 
−
 
 
−
 
 
−
=== 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
 
  
 
----
 
----
  
 
[[Category:ObjectStorage]]
 
[[Category:ObjectStorage]]

Latest revision as of 22:32, 6 March 2024

Swift Review Dashboard: http://not.mn/reviews.html


Client Patches

Swift Patches


PyECLib/LibEC Patches

Bug triage worksheet: https://etherpad.openstack.org/p/swift-bug-triage-list

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: http://not.mn/reviews.html

Retrieved from "https://wiki.openstack.org/w/index.php?title=Swift/PriorityReviews&oldid=184473"