Difference between revisions of "Swift/PriorityReviews"

Revision as of 19:38, 25 July 2018

Swift Review Dashboard: https://goo.gl/eqeGwE

Priority reviews/bugs

python-swiftclient release

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

Swift

PUT + POST for EC and encryption
- https://review.openstack.org/#/c/427911/
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()

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'''
-* Next python-swiftclient release:
+'''python-swiftclient release'''
-** <s>https://review.openstack.org/#/c/579037/ - Add bash_completion to swiftclient</s>
+* https://review.openstack.org/#/c/573355/ - Use python3 for a bunch of tox jobs
-** <s>https://review.openstack.org/#/c/581374/ - Add ability to generate a temporary URL with an IP range restriction
+** 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.
-*** Follow-up to swift patch https://review.openstack.org/#/c/569404/</s>
+* https://review.openstack.org/#/c/577584/ - Stop leaking quite so many connections
-** <s>https://review.openstack.org/#/c/538349/ - Treat 404 as success when cleaning up segments</s>
+** Has some good ideas, but tests need cleaning up
-** <s>https://review.openstack.org/#/c/498069/ - Stop mutating header dicts</s>
-** https://review.openstack.org/#/c/568914/ - Back out some dependency version bumps
-*** https://review.openstack.org/#/c/556192/ dragged our dependencies on requests, six, and python-keystoneclient forward a good bit (more than timburke would've liked). If we want to support `pip install`ing on platforms like xenial and CentOS 6/7 with their system packages, we ought to make it work now, before we have a release with the new deps.
-** 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
-** <s>https://review.openstack.org/#/c/549901/ - Add close() to _RetryBody</s>
-* Open Reviews
+'''Swift'''
-** PUT + POST for EC and encryption
+* PUT + POST for EC and encryption
-*** https://review.openstack.org/#/c/427911/
+** https://review.openstack.org/#/c/427911/
-** LOSF pre-requirements
+* LOSF pre-requirements
-*** https://review.openstack.org/#/c/447129/ Configure diskfile per storage policy
+** 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/548034/ Add optional parameters to common/utils fallocate()
 * Bugs