Bug #6078
closed
- Status changed from 12 to Fix Under Review
Pushed a bunch of changes to wip-6078.
- Assignee changed from Yehuda Sadeh to Josh Durgin
- Assignee changed from Josh Durgin to Yehuda Sadeh
Left a couple comments on github. It'd be nice to try to break up some of the changes and explicitly explain why changes are made more in future commits. I.e. https://github.com/ceph/ceph/commit/7bdcc79124504f3d924714ddfa0d60feb7c7209d could be split into method extraction, reordering error checks (I can infer why, but it won't be so clear without context months from now), and finally adding generic cors response.
- Status changed from Fix Under Review to Pending Backport
- Status changed from Pending Backport to Resolved
Done. Backported to dumpling, merged at commit:a304016fa01b02efd500135c00b9bf3407a9999c. Also, commit:670db7e80ddc9c26c43a4f66907a5996ce207c4d.
Also available in: Atom
PDF