Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

 
(250 intermediate revisions by 11 users not shown)
Line 1: Line 1:
−
'''Swift Review Dashboard:''' https://goo.gl/eqeGwE
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
  
−
= Review Priorities =
 
  
−
<strike>composite ring (for global ec)
+
'''Client Patches'''
−
* https://review.openstack.org/#/c/441921/</strike>
 
−
* https://review.openstack.org/#/c/465184/ (docs follow up)
 
  
−
per-policy affinity (for global ec)
+
* <s>https://review.opendev.org/c/openstack/python-swiftclient/+/909789 | CI: constrain deps for tests
−
<s>* https://review.openstack.org/#/c/462619/</s>
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/903770 | Add transaction id to errors
−
<s>* https://review.openstack.org/#/c/448240/</s>
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/848926 | shell: Print friendly account byte quotas
−
Follow-ups:
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/909498 | lint: Up-rev hacking</s>
−
  - <s>https://review.openstack.org/#/c/466952/</s>
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/833954 | Add formpost subcommand to generate signature
−
  - https://review.openstack.org/#/c/467920/
 
−
  - <s>https://review.openstack.org/#/c/467994/</s>
 
−
  - https://review.openstack.org/#/c/468011/
 
  
−
PUT+POST change unblocks golang object server work. The idea is to make the proxy-to-object protocol more standard so it's possible to implement it apart from eventlet
+
'''Swift Patches'''
−
* https://review.openstack.org/#/c/427911/
 
  
 +
* <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
  
 +
<!--
 +
* 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
 +
-->
  
−
----
+
'''PyECLib/LibEC Patches'''
−
 
 
−
== Bugs ==
 
−
 
 
−
'''High'''
 
−
 
 
−
* <s>ssync fails to replicate an object that had x-delete-at removed - https://bugs.launchpad.net/swift/+bug/1683689 (https://review.openstack.org/#/c/456921/)</s>
 
−
  - Note: merging this ^^ this does not fix https://bugs.launchpad.net/swift/+bug/1652323  “ssync syncs an expired object as a tombstone...”
 
−
* DB replicators loose metadata - https://bugs.launchpad.net/swift/+bug/1570118 (https://review.openstack.org/#/c/302494/)
 
  
−
'''Medium'''
+
* https://review.openstack.org/#/q/project:openstack/liberasurecode+is:open
−
 
+
* https://review.openstack.org/#/q/project:openstack/pyeclib+is:open
−
* StaticWeb & DomainRemap - https://bugs.launchpad.net/swift/+bug/1190625 (https://review.openstack.org/#/c/435771)
 
−
* Does everyone tune EVENTLET_THREADPOOL_SIZE by hand?  https://bugs.launchpad.net/swift/+bug/1554233 (https://review.openstack.org/#/c/289664/)
 
−
* Optimize reconstructor tombstone only revert https://bugs.launchpad.net/swift/+bug/1668857 (https://review.openstack.org/#/c/439572/)
 
−
* Drop translation that keeps causing bugs https://bugs.launchpad.net/swift/+bug/1580678
 
−
* Return less stale reads https://bugs.launchpad.net/swift/+bug/1560574 (https://review.openstack.org/#/c/371150/)
 
  
 
----
 
----
  
−
=== Triage Links ===
+
'''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
 
'''swiftclient bugs:''' https://goo.gl/uO4b7l  <-- bugs that are marked "New" and need triage
Line 49: Line 56:
 
----
 
----
  
−
=== Swift Review Dashboard ===
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
−
 
 
−
http://not.mn/reviews.html
 
  
 
----
 
----
  
 
[[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"