opendevreview | OpenStack Proposal Bot proposed openstack/swift master: Imported Translations from Zanata https://review.opendev.org/c/openstack/swift/+/886694 | 02:24 |
---|---|---|
opendevreview | Clay Gerrard proposed openstack/swift master: replicator: Add sync_batches_per_revert option https://review.opendev.org/c/openstack/swift/+/839649 | 04:40 |
opendevreview | Merged openstack/swift master: unify duplicate code in replication, EC GET paths https://review.opendev.org/c/openstack/swift/+/886645 | 04:42 |
opendevreview | Merged openstack/swift master: proxy: error limit the correct node when resuming GETs https://review.opendev.org/c/openstack/swift/+/886864 | 04:42 |
opendevreview | Matthew Oliver proposed openstack/swift master: internal_client: Add iter_{shard_ranges,namespaces} interfaces https://review.opendev.org/c/openstack/swift/+/877584 | 06:44 |
opendevreview | Philippe SERAPHIN proposed openstack/swift master: Adjust column width https://review.opendev.org/c/openstack/swift/+/887021 | 08:07 |
opendevreview | Merged openstack/swift master: sharder: Fix completed metric emission https://review.opendev.org/c/openstack/swift/+/886883 | 13:10 |
zigo | timburke: Heya Tim! Could you have a look at https://review.opendev.org/c/openstack/swift/+/887021 from my colleague? Only the first half of the patch is important, what's below is to reduce to 79 chars only (otherwise pep8 fails). | 14:01 |
opendevreview | Alistair Coles proposed openstack/swift master: proxy: encapsulate Getter resp, node and parts_iter https://review.opendev.org/c/openstack/swift/+/886994 | 14:21 |
opendevreview | Alistair Coles proposed openstack/swift master: proxy: remove client_chunk_size and skip_bytes from GetOrHeadHandler https://review.opendev.org/c/openstack/swift/+/886823 | 17:04 |
opendevreview | Alistair Coles proposed openstack/swift master: proxy: encapsulate Getter resp, node and parts_iter https://review.opendev.org/c/openstack/swift/+/886994 | 17:04 |
timburke | zigo, thanks for calling it out! left a review; core idea seems fairly reasonable. i wonder, though -- if we seem to get better signal looking in /dev/block, should we *just* look there? | 17:20 |
timburke | and sorry about the pep8 failures -- strange that we don't have it causing failures for other patches (at least, i *thought* our style checks ran on all files, not just the ones changed in the patch...) | 17:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!