Difference between revisions of "Swift/PriorityReviews"

Latest revision as of 22:32, 6 March 2024

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

Client Patches

~~https://review.opendev.org/c/openstack/python-swiftclient/+/909789 | CI: constrain deps for tests~~

https://review.opendev.org/c/openstack/python-swiftclient/+/903770 | Add transaction id to errors

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~~
https://review.opendev.org/c/openstack/python-swiftclient/+/833954 | Add formpost subcommand to generate signature

Swift Patches

~~https://review.opendev.org/c/openstack/swift/+/810754 | staticweb: Work with prefix-based tempurls~~
https://review.opendev.org/c/openstack/swift/+/907523 | drive-full-checker
expirer grace period patches
- https://review.opendev.org/c/openstack/swift/+/874806 | Add per account grace period to object expirer
- https://review.opendev.org/c/openstack/swift/+/874710 | Add x-open-expired to recover expired objects
part-number patches
- https://review.opendev.org/c/openstack/swift/+/894570 | slo: part-number=N query parameter support
- https://review.opendev.org/c/openstack/swift/+/894580 | s3api: Support GET/HEAD request with ?partNumber
- https://review.opendev.org/c/openstack/swift/+/906906 | s3api: return 400 if partNumber and Range sent
- https://review.opendev.org/c/openstack/swift/+/907751 | WIP s3api: add more MPU cross-compat tests
~~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~~
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

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: Done! =
-{| class="wikitable"
+'''Client Patches'''
-|-
-! Patch number !! Priority !! Patch owner !! Summary !! Status
-|-
-| <strike>https://review.openstack.org/#/c/227133/</strike> || <strike>C</strike> || <strike>Christian</strike> || <strike>Enforce service role validation</strike> ||  merged
-|-
-| https://review.openstack.org/#/c/222799 || <strike>C</strike> || <strike>Clay</strike> || <strike>Validate against duplicate part replica device assignment</strike> || merged
-|-
-| <strike>https://review.openstack.org/#/c/138498/</strike> || <strike>C</strike> || <strike>Alistair</strike> || <strike>Add POST capability to ssync for .meta files</strike> || merged
-|-
-| <strike>https://review.openstack.org/#/c/215276</strike> || <strike>C</strike> || <strike>Alistair</strike> || <strike>Enable object server to return non-durable data</strike> || has been (is being) triaged with other work. will not be in liberty
-|-
-| <strike>https://review.openstack.org/#/c/186735</strike> || <strike>C</strike> || <strike>Kota</strike> || <strike>Fix missing container update</strike> || merged
-|-
-| <strike>https://review.openstack.org/#/c/211338</strike> || <strike>C</strike> || <strike>Paul</strike> || <strike>Better error handling for EC PUT path when client goes away</strike> || merged
-|-
-| <strike>https://review.openstack.org/#/c/220059</strike> || <strike>C</strike> || <strike>Kota</strike> || <strike>Don't send commits for quorum BAD requests on EC</strike> || merged
-|-
-| <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 =
+* <s>https://review.opendev.org/c/openstack/python-swiftclient/+/909789 | CI: constrain deps for tests
+* https://review.opendev.org/c/openstack/python-swiftclient/+/903770 | Add transaction id to errors
+* 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
-* EC "done" -- good enough to recommend for production use https://bugs.launchpad.net/swift/+bugs?field.tag=ec
+'''Swift Patches'''
-* encryption -- Trello board at https://trello.com/b/63l5zQhq/swift-encryption
-* swiftclient improvements -- feature parity, parallel SLO downloads, discoverability, session management
-* global clusters improvements -- eg container sync (https://review.openstack.org/#/q/status:open+owner:%22Eran+Rom+%253Ceranr%2540il.ibm.com%253E%22+project:openstack/swift,n,z)
-* container sharding -- https://review.openstack.org/#/c/218738/
-=== EC Bugs/Patches Prioritization for undecided/new (9/9/2015) ===
+* <s>https://review.opendev.org/c/openstack/swift/+/810754 | staticweb: Work with prefix-based tempurls</s>
-'''Updates to the logs should have '''low''' priority until post Liberty:'''
+* https://review.opendev.org/c/openstack/swift/+/907523 | drive-full-checker
-* Object server logs don't reflect reconstructor as source of query: https://bugs.launchpad.net/swift/+bug/1491871
+* expirer grace period patches
-* Reconstructor complains 404 is invalid responce when building objectL https://bugs.launchpad.net/swift/+bug/1491883
+** https://review.opendev.org/c/openstack/swift/+/874806 | Add per account grace period to object expirer
-<br />
+** https://review.opendev.org/c/openstack/swift/+/874710 | Add x-open-expired to recover expired objects
-'''Should be higher than low:'''
+* part-number patches
-*Proxy server ignores additional fragments on primary nodes: https://bugs.launchpad.net/swift/+bug/1484598
+** https://review.opendev.org/c/openstack/swift/+/894570 | slo: part-number=N query parameter support
-*Proxy server reports 202 even though EC backend didn't accept object (not just EC related?): https://bugs.launchpad.net/swift/+bug/1491908
+** https://review.opendev.org/c/openstack/swift/+/894580 | s3api: Support GET/HEAD request with ?partNumber
-<br />
+** https://review.opendev.org/c/openstack/swift/+/906906 | s3api: return 400 if partNumber and Range sent
-'''This needs updates from the bug owners:'''
+** https://review.opendev.org/c/openstack/swift/+/907751 | WIP s3api: add more MPU cross-compat  tests
-*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
+* aws-chunked patches
+** 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 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]]