Difference between revisions of "Swift/PriorityReviews"

Latest revision as of 00:01, 27 February 2025

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

Swift Patches

https://review.opendev.org/c/openstack/swift/+/938631 | py3: Fix unicode-header-name handling
https://review.opendev.org/c/openstack/swift/+/942329 | tests: Fix versioning cross-compat tests
https://review.opendev.org/c/openstack/swift/+/942850 | Add config option for whether to run s3_acl-requiring tests
- https://review.opendev.org/c/openstack/swift/+/942133 | CI: run S3 cross-compat tests as part of DSVM jobs
https://review.opendev.org/c/openstack/swift/+/887908 | Stop using cgi.parse_header
cooperative tokens
- https://review.opendev.org/c/openstack/swift/+/890174 | common: add memcached based cooperative token mechanism
- https://review.opendev.org/c/openstack/swift/+/908969 | proxy: use cooperative tokens to coalesce updating shard range requests into backend

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
-= Liberty patches =
-{| class="wikitable"
+'''Swift Patches'''
-|-
-! Patch number !! Priority !! Patch owner !! Summary !! Status
-|-
-| https://review.openstack.org/#/c/227133/ || C || Christian || Enforce service role validation ||  Being reviewed by Matt and Alistair
-|-
-| https://review.openstack.org/#/c/222799 || C || Clay || Validate against duplicate part replica device assignment || reviewed by Sam and Christian; will have 1 or 2 follow-on patches
-|-
-| https://review.openstack.org/#/c/215276 || C || Alistair || Enable object server to return non-durable data || needs review and testing; Kota and Paul to review
-|-
-| https://review.openstack.org/#/c/186735 || C || Kota || Fix missing container update || Being reviewed by Matt and John
-|-
-| https://review.openstack.org/#/c/211338 || C || Paul || Better error handling for EC PUT path when client goes away || reviewed by Kota and Matt
-|-
-| https://review.openstack.org/#/c/220059 || C || Kota || Don't send commits for quorum BAD requests on EC || reviewed by Alistair and Sam
-|-
-| <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 =
+* https://review.opendev.org/c/openstack/swift/+/938631 | py3: Fix unicode-header-name handling
+* https://review.opendev.org/c/openstack/swift/+/942329 | tests: Fix versioning cross-compat tests
+* https://review.opendev.org/c/openstack/swift/+/942850 | Add config option for whether to run s3_acl-requiring tests
+** https://review.opendev.org/c/openstack/swift/+/942133 | CI: run S3 cross-compat tests as part of DSVM jobs
+* https://review.opendev.org/c/openstack/swift/+/887908 | Stop using cgi.parse_header
+* cooperative tokens
+** https://review.opendev.org/c/openstack/swift/+/890174 | common: add memcached based cooperative token mechanism
+** https://review.opendev.org/c/openstack/swift/+/908969 | proxy: use cooperative tokens to coalesce updating shard range requests into backend
-* EC "done" -- good enough to recommend for production use https://bugs.launchpad.net/swift/+bugs?field.tag=ec
+<!--
-* encryption -- Trello board at https://trello.com/b/63l5zQhq/swift-encryption
+* https://review.opendev.org/c/openstack/swift/+/800701 - Delete s3api MPU segments when the manifest expires
-* swiftclient improvements -- feature parity, parallel SLO downloads, discoverability, session management
+* https://review.opendev.org/c/openstack/swift/+/834261 - Introduce a v2 ring format
-* global clusters improvements -- eg container sync
+-->
-* container sharding -- https://review.openstack.org/#/c/139921/
-=== EC Bugs/Patches Prioritization for undecided/new (9/9/2015) ===
+'''PyECLib/LibEC Patches'''
-'''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 =
+* https://review.openstack.org/#/q/project:openstack/liberasurecode+is:open
+* https://review.openstack.org/#/q/project:openstack/pyeclib+is:open
-'''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 ===
+'''Bug triage worksheet:''' https://etherpad.openstack.org/p/swift-bug-triage-list
-https://goo.gl/eqeGwE
+'''swiftclient bugs:''' https://goo.gl/uO4b7l  <-- bugs that are marked "New" and need triage
-(this expands to)
+'''swift bugs:''' https://goo.gl/LMiiDc  <-- bugs that are marked "New" and need triage
-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
-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
+----
+'''Swift Review Dashboard:''' http://not.mn/reviews.html
 ----
 [[Category:ObjectStorage]]