Difference between revisions of "Swift/PriorityReviews"

Revision as of 22:17, 26 September 2018

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

Priority reviews/bugs

Keymaster nice-to-haves
- https://review.openstack.org/#/c/586902/ - multiple keymasters
- https://review.openstack.org/#/c/586903/ - Display crypto data/metadata details in swift-object-info
General task queue work
- https://review.openstack.org/#/c/601950/
Pending s3api patches
- https://review.openstack.org/#/c/592231/ - s3api: Include '-' in S3 ETags of normal SLOs - when possible, do the same for SLOs/multipart-uploads *not* written via an updated s3api
- https://review.openstack.org/#/c/575838/ - Listing of versioned objects when versioning is not enabled
Undelete accounts
- https://review.openstack.org/#/c/507808
python-swiftclient
- https://review.openstack.org/#/c/573355/ - Use python3 for a bunch of tox jobs
  - 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
  - Has some good ideas, but tests need cleaning up

'Bugs
- Client may hold socket open after ChunkWriteTimeout
  - https://bugs.launchpad.net/swift/+bug/1572719 (high)
- Connection between client and proxy service does not close
  - https://bugs.launchpad.net/swift/+bug/1568650 (high)

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 4: / Line 4: @@
 '''Priority reviews/bugs'''
-* Multi-key KMIP (in order)
+* '''Keymaster''' nice-to-haves
-** <s>https://review.openstack.org/#/c/586901/ - Move keymaster_config_path parsing out of _get_root_secret</s>
+** https://review.openstack.org/#/c/586902/ - multiple keymasters
-** <s>https://review.openstack.org/#/c/577874/ - multiple secrets</s>
+** https://review.openstack.org/#/c/586903/ - Display crypto data/metadata details in swift-object-info
-** <s>https://review.openstack.org/#/c/586455/ - multi-key kmip</s>
+* '''General task queue''' work
-** https://review.openstack.org/#/c/586902/ - multiple keymasters (nice-to-have)
+** https://review.openstack.org/#/c/601950/
-** https://review.openstack.org/#/c/586903/ - Display crypto data/metadata details in swift-object-info (nice-to-have)
+* Pending '''s3api''' patches
-* 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
 ** https://review.openstack.org/#/c/592231/ - s3api: Include '-' in S3 ETags of normal SLOs - when possible, do the same for SLOs/multipart-uploads *not* written via an updated s3api
-** <s>https://review.openstack.org/#/c/577941/ - s3_acl: Require swift_owner authz to create buckets</s>
-** <s>https://review.openstack.org/#/c/575808/ - Add debugging info to SignatureDoesNotMatch responses</s>
-** <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
+* '''python-swiftclient'''
-** https://review.openstack.org/#/c/447129/ Configure diskfile per storage policy
+** https://review.openstack.org/#/c/573355/ - Use python3 for a bunch of tox jobs
-** https://review.openstack.org/#/c/548034/ Add optional parameters to common/utils fallocate()
+*** 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
+*** Has some good ideas, but tests need cleaning up
-* https://review.openstack.org/#/c/573355/ - Use python3 for a bunch of tox jobs ('''python-swiftclient''')
+* ''''Bugs'''
-** 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
 ** Client may hold socket open after ChunkWriteTimeout
 *** https://bugs.launchpad.net/swift/+bug/1572719 (high)
 ** Connection between client and proxy service does not close
 *** https://bugs.launchpad.net/swift/+bug/1568650 (high)
-** hard-coded socket.TCP_KEEPIDLE = 600 causes problems for some users (low hanging fruit)
-*** https://bugs.launchpad.net/swift/+bug/1759606
 ----