Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

 
(559 intermediate revisions by 18 users not shown)
Line 1: Line 1:
= Swift Review Dashboard =
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
Possibly abandoned changes dash:
 
  
http://bit.ly/UHcm1d
 
  
Older version:
+
'''Client Patches'''
  
http://bit.ly/1iVBigF
+
* <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
  
Please take a look at the "Needs Approval" section of the dashboard so we can get pending stuff landed quickly.
+
'''Swift Patches'''
  
== Bugs that need to be addressed ==
+
* <s>https://review.opendev.org/c/openstack/swift/+/810754 | staticweb: Work with prefix-based tempurls</s>
 +
* 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
 +
<!--
 +
* 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
  
* affinity + DELETEs https://bugs.launchpad.net/swift/+bug/1318375
+
<!--
 +
* 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
 +
-->
  
== General ==
+
'''PyECLib/LibEC Patches'''
  
* migration middleware https://review.openstack.org/#/c/64430/
+
* https://review.openstack.org/#/q/project:openstack/liberasurecode+is:open
* global cluster replication improvements https://review.openstack.org/#/c/99824/
+
* https://review.openstack.org/#/q/project:openstack/pyeclib+is:open
* SwiftService (makes swiftclient way more useful as a library) https://review.openstack.org/#/c/85453/
 
  
== keystone v3 related reviews ==
+
----
  
* https://review.openstack.org/#/c/91788/ - swiftclient support
+
'''Bug triage worksheet:''' https://etherpad.openstack.org/p/swift-bug-triage-list
* https://review.openstack.org/#/c/86430/ - fixing the ACL issue
 
  
See https://wiki.openstack.org/wiki/Swift/DevstackSetupForKeystoneV3 for how to test these with devstack (more notes at https://gist.github.com/alistairncoles/ae9d5f92063b58afeb88)
+
'''swiftclient bugs:''' https://goo.gl/uO4b7l  <-- bugs that are marked "New" and need triage
  
== Performance ==
+
'''swift bugs:''' https://goo.gl/LMiiDc  <-- bugs that are marked "New" and need triage
* https://review.openstack.org/#/c/91133/ - make chunked PUTs faster (on par with non-chunked PUTs)
 
* Improve the reconciler: https://review.openstack.org/#/q/status:open+project:openstack/swift+branch:master+topic:multi-reconciler,n,z
 
* object-updater: https://review.openstack.org/#/c/75941/
 
* zero-copy object server GET: https://review.openstack.org/#/c/102609/
 
* buffer DiskFile writes https://review.openstack.org/#/c/91133/
 
  
= Swift Dashboard =
+
----
Possible abandoned changes version:
 
  
http://bit.ly/UHcm1d
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
  
(this expands to)
+
----
  
https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fswift+OR+project%3Aopenstack%2Fpython-swiftclient+OR+project%3Aopenstack%2Fswift-python-agent+OR+project%3Aopenstack%2Fswift-bench+OR+project%3Aopenstack%2Fswift-specs%29+status%3Aopen+NOT+label%3ACode-Review%3C%3D-2&title=Swift+Review+Dashboard&Swift+Specs=project%3Aopenstack%2Fswift-specs+NOT+owner%3Aself&My+Patches=owner%3Aself+status%3Aopen&Needs+Approval=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%252cjenkins+NOT+owner%3Aself+label%3ACode-Review%3E%3D2+NOT+label%3ACode-Review-1+NOT+project%3Aopenstack%2Fswift-specs+branch%3Amaster&Needs+Reverify=label%3AVerified%3C%3D-1%252cjenkins+NOT+project%3Aopenstack%2Fswift-specs+branch%3Amaster+NOT+label%3ACode-Review%3C%3D-1+NOT+label%3AWorkflow%3C%3D-1&5+Days+Without+Feedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%252cjenkins+NOT+owner%3Aself+NOT+label%3ACode-Review%3C%3D2+age%3A5d+NOT+project%3Aopenstack%2Fswift-specs+branch%3Amaster&No+Negative+Feedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%252cjenkins+NOT+owner%3Aself+NOT+label%3ACode-Review%3C%3D-1+NOT+label%3ACode-Review%3E%3D2+NOT+project%3Aopenstack%2Fswift-specs+limit%3A50+branch%3Amaster&With+Negative+Feedback=NOT+label%3AWorkflow%3C%3D-1+NOT+label%3AVerified%3C%3D-1%252cjenkins+NOT+owner%3Aself+label%3ACode-Review-1+NOT+project%3Aopenstack%2Fswift-specs+limit%3A20+branch%3Amaster&Work+In+Progress=NOT+label%3AWorkflow%3E%3D1+NOT+owner%3Aself+label%3AWorkflow%3C%3D-1+limit%3A20+NOT+project%3Aopenstack%2Fswift-specs+branch%3Amaster&Erasure+Code+work=NOT+project%3Aopenstack%2Fswift-specs+branch%3Afeature%2Fec+status%3Aopen&Possible+abandoned+changes=status%3Aopen+age%3A4week+%28label%3AVerified%3C%3D-1%252cjenkins+OR+label%3ACode-Review%3C%3D-1%29+NOT+label%3AWorkflow%3C%3D-1
+
[[Category:ObjectStorage]]
 
 
Older version:
 
http://bit.ly/1iVBigF
 
 
 
(this expands to)
 
 
 
https://review.openstack.org/#/dashboard/?foreach=(project:openstack/swift+OR+project:openstack/python-swiftclient+OR+project:openstack/swift-python-agent+OR+project:openstack/swift-bench+OR+project:openstack/swift-specs)+status:open+NOT+label:Code-Review%3C=-2&title=Swift+Review+Dashboard&Swift+Specs=project:openstack/swift-specs+NOT+owner:self&My+Patches=owner:self+status:open&Needs+Approval=NOT+label:Workflow%3E=1+NOT+label:Workflow%3C=-1+label:Verified%3E=1%252cjenkins+NOT+owner:self+label:Code-Review%3E=2+NOT+label:Code-Review-1+NOT+project:openstack/swift-specs+branch:master&Needs+Reverify=label:Verified%3C=-1%252cjenkins+NOT+project:openstack/swift-specs+branch:master+NOT+label:Code-Review%3C=-1+NOT+label:Workflow%3C=-1&5+Days+Without+Feedback=NOT+label:Workflow%3E=1+NOT+label:Workflow%3C=-1+label:Verified%3E=1%252cjenkins+NOT+owner:self+NOT+label:Code-Review%3C=2+age:5d+NOT+project:openstack/swift-specs+branch:master&No+Negative+Feedback=NOT+label:Workflow%3E=1+NOT+label:Workflow%3C=-1+label:Verified%3E=1%252cjenkins+NOT+owner:self+NOT+label:Code-Review%3C=-1+NOT+label:Code-Review%3E=2+NOT+project:openstack/swift-specs+limit:50+branch:master&With+Negative+Feedback=NOT+label:Workflow%3C=-1+NOT+label:Verified%3C=-1%252cjenkins+NOT+owner:self+label:Code-Review-1+NOT+project:openstack/swift-specs+limit:20+branch:master&Work+In+Progress=NOT+label:Workflow%3E=1+NOT+owner:self+label:Workflow%3C=-1+limit:20+NOT+project:openstack/swift-specs+branch:master&Erasure+Code+work=NOT+project:openstack/swift-specs+branch:feature/ec+status:open
 

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