Difference between revisions of "Swift/PriorityReviews"
< Swift
Line 19: | Line 19: | ||
| https://review.openstack.org/#/c/220059 || C || Kota || Don't send commits for quorum BAD requests on EC || reviewed by Alistair and Sam | | https://review.openstack.org/#/c/220059 || C || Kota || Don't send commits for quorum BAD requests on EC || reviewed by Alistair and Sam | ||
|- | |- | ||
− | | https://review.openstack.org/#/c/211726/ || M || Tim || Add the ability to specify ranges for SLO segments || merged | + | | <strike>https://review.openstack.org/#/c/211726/</strike> || <strike>M</strike> || <strike>Tim</strike> || <strike>Add the ability to specify ranges for SLO segments</strike> || merged |
+ | |- | ||
+ | | https://review.openstack.org/#/c/227569/ || M || Tim || Fixes tests from SLORange patch || to be reviewed by Sam and Matt | ||
|- | |- | ||
| https://review.openstack.org/#/c/196848 || L || Minwoo || Handoff node to push existing fragment to the correct location || reviewed by Christian | | https://review.openstack.org/#/c/196848 || L || Minwoo || Handoff node to push existing fragment to the correct location || reviewed by Christian |
Revision as of 03:33, 25 September 2015
Swift Review Dashboard: https://goo.gl/eqeGwE
Contents
Liberty patches
Patch number | Priority | Patch owner | Summary | Status |
---|---|---|---|---|
https://review.openstack.org/#/c/227133/ | C | Christian | Enforce service role validation | Being reviewed by Matt and Alistair |
https://review.openstack.org/#/c/222799 | C | Clay | Validate against duplicate part replica device assignment | reviewed by Sam and Christian; will have 1 or 2 follow-on patches |
https://review.openstack.org/#/c/215276 | C | Alistair | Enable object server to return non-durable data | WIP -- needs help; Kota and Paul to review |
https://review.openstack.org/#/c/186735 | C | Kota | Fix missing container update | Being reviewed by Matt and John |
https://review.openstack.org/#/c/211338 | C | Paul | Better error handling for EC PUT path when client goes away | reviewed by Kota and Matt |
https://review.openstack.org/#/c/220059 | C | Kota | Don't send commits for quorum BAD requests on EC | reviewed by Alistair and Sam |
|
|
|
|
merged |
https://review.openstack.org/#/c/227569/ | M | Tim | Fixes tests from SLORange patch | to be reviewed by Sam and Matt |
https://review.openstack.org/#/c/196848 | L | Minwoo | Handoff node to push existing fragment to the correct location | reviewed by Christian |
Big important topics right now
- EC "done" -- good enough to recommend for production use https://bugs.launchpad.net/swift/+bugs?field.tag=ec
- encryption -- Trello board at https://trello.com/b/63l5zQhq/swift-encryption
- swiftclient improvements -- feature parity, parallel SLO downloads, discoverability, session management
- global clusters improvements -- eg container sync
- container sharding -- https://review.openstack.org/#/c/139921/
EC Bugs/Patches Prioritization for undecided/new (9/9/2015)
Updates to the logs should have low priority until post Liberty:
- Object server logs don't reflect reconstructor as source of query: https://bugs.launchpad.net/swift/+bug/1491871
- Reconstructor complains 404 is invalid responce when building objectL https://bugs.launchpad.net/swift/+bug/1491883
Should be higher than low:
- Proxy server ignores additional fragments on primary nodes: https://bugs.launchpad.net/swift/+bug/1484598
- Proxy server reports 202 even though EC backend didn't accept object (not just EC related?): https://bugs.launchpad.net/swift/+bug/1491908
This needs updates from the bug owners:
- ssync receiver get an odd SSYNC command and raise a exception: https://bugs.launchpad.net/swift/+bug/1468708
- Balance of EC ring seems odd compared to replica ring: https://bugs.launchpad.net/swift/+bug/1491669
Bug Triage Links
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 Dashboard Long URL
(this expands to)
Project overview dashboard: https://goo.gl/jL0byl
(this expands to)