Difference between revisions of "Swift/PriorityReviews"

Latest revision as of 20:32, 20 January 2021

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

Open patches that are running in production somewhere

https://review.openstack.org/#/c/333331/ (rledisez)
https://review.openstack.org/#/c/337861/ (rledisez)
https://review.openstack.org/#/c/342857/ (timburke)

Swift Patches

regressions
- ~~https://review.opendev.org/c/openstack/swift/+/769848 - Resolve a regression with the patch for missing binaries~~
- ~~https://review.opendev.org/c/openstack/swift/+/770004 - s3api: Make UNSIGNED-PAYLOAD requests work again~~
py3 fixes
- https://review.opendev.org/c/openstack/swift/+/695781 - s3api: Fix prefix/delimiter/marker quoting
- ~~https://review.opendev.org/c/openstack/swift/+/759075 - Fix status comparison~~
- ~~https://review.opendev.org/c/openstack/swift/+/765204 - Fix non-ASCII multipart uploads~~
s3api improvements
- https://review.opendev.org/c/openstack/swift/+/765626 - Fix limited part listings with more than 9 parts
- https://review.opendev.org/c/openstack/swift/+/695780 - Fix blank delimiter handling
- https://review.opendev.org/c/openstack/swift/+/757636 - Transfer REMOTE_USER when using s3_acl
sharding
- https://review.opendev.org/c/openstack/swift/+/765623 - swift-manage-shard-ranges: add 'compact' command
- https://review.opendev.org/c/openstack/swift/+/765624 - swift-manage-shard-ranges: add repair and analyze commands
https://review.opendev.org/c/openstack/swift/+/742300 - Have object-server return 507 when chunked-transfer runs out of space
https://review.opendev.org/c/openstack/swift/+/758605 - Have recon only count asyncs once per IP

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

@@ Line 1: / Line 1: @@
-'''Swift Review Dashboard:''' https://goo.gl/eqeGwE
+'''Swift Review Dashboard:''' http://not.mn/reviews.html
-= Review Priorities =
+'''Open patches that are running in production somewhere'''
-FIX PROBETESTS!!!
+* https://review.openstack.org/#/c/333331/ (rledisez)
-    https://review.openstack.org/#/c/274840/ broke proble tests. There are two open patch that promise to fix it.
+* https://review.openstack.org/#/c/337861/ (rledisez)
-    Land one or both of
+* https://review.openstack.org/#/c/342857/ (timburke)
-      - https://review.openstack.org/#/c/313150/
-      - https://review.openstack.org/#/c/311899/
-Features/bug-fixes in one patch:
-    ionice https://review.openstack.org/#/c/238799/  <-- needs 1 more +2
-Container Sync:
+'''Swift Patches'''
-    process level concurrency https://review.openstack.org/#/c/210099/
-    thread level concurrency https://review.openstack.org/#/c/225338/
-Crypto :
+* regressions
-   These reviews are highest priority, in addition to the copy middleware review listed above:
+** <s>https://review.opendev.org/c/openstack/swift/+/769848 - Resolve a regression with the patch for missing binaries</s>
-    crypto - don't break conditional GETs on unencrypted objects  https://review.openstack.org/314740
+** <s>https://review.opendev.org/c/openstack/swift/+/770004 - s3api: Make UNSIGNED-PAYLOAD requests work again</s>
-    https://review.openstack.org/313640   crypto - wrap random body key
+* py3 fixes
+** https://review.opendev.org/c/openstack/swift/+/695781 - s3api: Fix prefix/delimiter/marker quoting
+** <s>https://review.opendev.org/c/openstack/swift/+/759075 - Fix status comparison</s>
+** <s>https://review.opendev.org/c/openstack/swift/+/765204 - Fix non-ASCII multipart uploads</s>
+* s3api improvements
+** https://review.opendev.org/c/openstack/swift/+/765626 - Fix limited part listings with more than 9 parts
+** https://review.opendev.org/c/openstack/swift/+/695780 - Fix blank delimiter handling
+** https://review.opendev.org/c/openstack/swift/+/757636 - Transfer REMOTE_USER when using s3_acl
+* sharding
+** https://review.opendev.org/c/openstack/swift/+/765623 - swift-manage-shard-ranges: add 'compact' command
+** https://review.opendev.org/c/openstack/swift/+/765624 - swift-manage-shard-ranges: add repair and analyze commands
+* https://review.opendev.org/c/openstack/swift/+/742300 - Have object-server return 507 when chunked-transfer runs out of space
+* https://review.opendev.org/c/openstack/swift/+/758605 - Have recon only count asyncs once per IP
-    Complete list of crypto reviews is here: https://review.openstack.org/#/q/status:open+project:openstack/swift+branch:feature/crypto
-= Bug Triage Links =
+'''PyECLib/LibEC Patches'''
-'''swiftclient bugs:''' https://goo.gl/uO4b7l  <-- bugs that are marked "New" and need triage
+* https://review.openstack.org/#/q/project:openstack/liberasurecode+is:open
-'''swift bugs:''' https://goo.gl/LMiiDc  <-- bugs that are marked "New" and need triage
+* 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]]