Jump to: navigation, search

Difference between revisions of "Swift/PriorityReviews"

 
(169 intermediate revisions by 8 users not shown)
Line 1: Line 1:
'''Swift Review Dashboard:''' https://goo.gl/eqeGwE
+
'''Swift Review Dashboard:''' http://not.mn/reviews.html
  
  
'''Priority reviews/bugs'''
+
'''Client Patches'''
  
* Multi-key KMIP (in order)
+
* <s>https://review.opendev.org/c/openstack/python-swiftclient/+/909789 | CI: constrain deps for tests
** <s>https://review.openstack.org/#/c/586901/ - Move keymaster_config_path parsing out of _get_root_secret</s>
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/903770 | Add transaction id to errors
** https://review.openstack.org/#/c/577874/ - multiple secrets
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/848926 | shell: Print friendly account byte quotas
** https://review.openstack.org/#/c/586455/ - multi-key kmip
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/909498 | lint: Up-rev hacking</s>
** https://review.openstack.org/#/c/586902/ - multiple keymasters (nice-to-have)
+
* https://review.opendev.org/c/openstack/python-swiftclient/+/833954 | Add formpost subcommand to generate signature
** https://review.openstack.org/#/c/586903/ - Display crypto data/metadata details in swift-object-info (nice-to-have)
 
* Pending s3api patches (in order)
 
** <s>https://review.openstack.org/#/c/570477/ - Stop holding on to sys.exc_info tuples quite so much - simple cleanup</s>
 
** https://review.openstack.org/#/c/575860/ - Include '-' in multipart ETags - matches S3 and required for many 3rd party tools to prevent etag validation
 
** <s>https://review.openstack.org/#/c/577941/ - s3_acl: Require swift_owner authz to create buckets</s>
 
** https://review.openstack.org/#/c/575808/ - Add debugging info to SignatureDoesNotMatch responses
 
** <s>https://review.openstack.org/#/c/575842/ - Fix the deletion of non-existent keys</s>
 
** https://review.openstack.org/#/c/575838/ - Listing of versioned objects when versioning is not enabled
 
* PUT + POST for EC and encryption
 
** https://review.openstack.org/#/c/427911/
 
* Undelete accounts
 
** https://review.openstack.org/#/c/507808
 
* LOSF pre-requirements
 
** https://review.openstack.org/#/c/447129/ Configure diskfile per storage policy
 
** https://review.openstack.org/#/c/548034/ Add optional parameters to common/utils fallocate()
 
  
* https://review.openstack.org/#/c/573355/ - Use python3 for a bunch of tox jobs ('''python-swiftclient''')
+
'''Swift Patches'''
** At the very least, it'd be good to have ''at least one'' functional test job that runs with python3. There are legit test failures, though -- probably need something like https://review.openstack.org/#/c/562059/ though the approach may need more thought.
 
* https://review.openstack.org/#/c/577584/ - Stop leaking quite so many connections ('''python-swiftclient''')
 
** Has some good ideas, but tests need cleaning up
 
  
* Bugs
+
* <s>https://review.opendev.org/c/openstack/swift/+/810754 | staticweb: Work with prefix-based tempurls</s>
** Client may hold socket open after ChunkWriteTimeout
+
* https://review.opendev.org/c/openstack/swift/+/907523 | drive-full-checker
*** https://bugs.launchpad.net/swift/+bug/1572719 (high)
+
* expirer grace period patches
** Connection between client and proxy service does not close
+
** https://review.opendev.org/c/openstack/swift/+/874806 | Add per account grace period to object expirer
*** https://bugs.launchpad.net/swift/+bug/1568650 (high)
+
** https://review.opendev.org/c/openstack/swift/+/874710 | Add x-open-expired to recover expired objects
** hard-coded socket.TCP_KEEPIDLE = 600 causes problems for some users (low hanging fruit)
+
* part-number patches
*** https://bugs.launchpad.net/swift/+bug/1759606
+
** 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'''
 +
 
 +
* https://review.openstack.org/#/q/project:openstack/liberasurecode+is:open
 +
* https://review.openstack.org/#/q/project:openstack/pyeclib+is: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