Tuesday, 2023-06-27

opendevreviewOpenStack Proposal Bot proposed openstack/swift master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/swift/+/88669402:24
opendevreviewClay Gerrard proposed openstack/swift master: replicator: Add sync_batches_per_revert option  https://review.opendev.org/c/openstack/swift/+/83964904:40
opendevreviewMerged openstack/swift master: unify duplicate code in replication, EC GET paths  https://review.opendev.org/c/openstack/swift/+/88664504:42
opendevreviewMerged openstack/swift master: proxy: error limit the correct node when resuming GETs  https://review.opendev.org/c/openstack/swift/+/88686404:42
opendevreviewMatthew Oliver proposed openstack/swift master: internal_client: Add iter_{shard_ranges,namespaces} interfaces  https://review.opendev.org/c/openstack/swift/+/87758406:44
opendevreviewPhilippe SERAPHIN proposed openstack/swift master: Adjust column width  https://review.opendev.org/c/openstack/swift/+/88702108:07
opendevreviewMerged openstack/swift master: sharder: Fix completed metric emission  https://review.opendev.org/c/openstack/swift/+/88688313:10
zigotimburke: 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
opendevreviewAlistair Coles proposed openstack/swift master: proxy: encapsulate Getter resp, node and parts_iter  https://review.opendev.org/c/openstack/swift/+/88699414:21
opendevreviewAlistair Coles proposed openstack/swift master: proxy: remove client_chunk_size and skip_bytes from GetOrHeadHandler  https://review.opendev.org/c/openstack/swift/+/88682317:04
opendevreviewAlistair Coles proposed openstack/swift master: proxy: encapsulate Getter resp, node and parts_iter  https://review.opendev.org/c/openstack/swift/+/88699417:04
timburkezigo, 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
timburkeand 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/!