| opendevreview | Tim Burke proposed openstack/python-swiftclient master: Add option to skip container PUT during upload https://review.opendev.org/c/openstack/python-swiftclient/+/824281 | 05:13 |
|---|---|---|
| opendevreview | Tim Burke proposed openstack/python-swiftclient master: Detect and retry SLO delete partial failures https://review.opendev.org/c/openstack/python-swiftclient/+/814820 | 05:16 |
| opendevreview | Tim Burke proposed openstack/python-swiftclient master: Translate '\'-delimited paths on Windows to be '/'-delimited https://review.opendev.org/c/openstack/python-swiftclient/+/241064 | 05:24 |
| opendevreview | Mickael Asseline proposed openstack/python-swiftclient master: Fix copy.py example. https://review.opendev.org/c/openstack/python-swiftclient/+/824361 | 09:20 |
| opendevreview | Alistair Coles proposed openstack/swift master: Modify log_name in internal clients' pipeline configs https://review.opendev.org/c/openstack/swift/+/820539 | 11:07 |
| opendevreview | Clay Gerrard proposed openstack/swift master: Get rid of pipeline_property (harder) https://review.opendev.org/c/openstack/swift/+/824471 | 17:34 |
| opendevreview | Tim Burke proposed openstack/swift master: Allow ring-builder CLI users to specify device ID https://review.opendev.org/c/openstack/swift/+/808532 | 19:11 |
| *** timburke__ is now known as timburke | 19:12 | |
| timburke | just popping ^^^ off the ring-v2 chain, since it's pretty independent | 19:13 |
| opendevreview | Merged openstack/python-swiftclient master: Fix copy.py example. https://review.opendev.org/c/openstack/python-swiftclient/+/824361 | 19:32 |
| timburke | #startmeeting swift | 21:00 |
| opendevmeet | Meeting started Wed Jan 12 21:00:22 2022 UTC and is due to finish in 60 minutes. The chair is timburke. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:00 |
| opendevmeet | The meeting name has been set to 'swift' | 21:00 |
| timburke | who's here for the swift meeting? | 21:00 |
| acoles | o/ | 21:01 |
| mattoliver | o/ | 21:01 |
| mattoliver | (Tho taking care of kids at the same time so a little distracted) | 21:02 |
| timburke | as usual, the agenda's at https://wiki.openstack.org/wiki/Meetings/Swift | 21:02 |
| timburke | #topic object updater fairness | 21:02 |
| kota | hi | 21:02 |
| timburke | looks like the chain related to all this has collapsed down to just the two changes | 21:03 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/820962 | 21:03 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/821736 | 21:04 |
| timburke | how are we feeling about them? do we think they need more review? each has a +2 | 21:04 |
| acoles | I think we distilled things down to a reasonable balance of simple vs fair | 21:06 |
| acoles | I'm not sure we've yet seen the deferrals kick in in prod yet | 21:06 |
| timburke | so maybe we merge the first and wait on the second for another week? | 21:07 |
| acoles | yes, just in case something shows up in prod, that seems prudent | 21:08 |
| mattoliver | +1 | 21:08 |
| timburke | sounds good | 21:09 |
| timburke | #topic memcache skipping | 21:09 |
| timburke | just a quick update: we recently got the shard range patch into prod; still need to actually try it out, though (the skip behavior is off by default) | 21:10 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/736802 | 21:10 |
| mattoliver | I saw the shard range one. Opened it last night to review, but didn't get far. Will check these out today. | 21:11 |
| mattoliver | It's a great idea. Keeping the cache warm is a great idea | 21:11 |
| timburke | the account/container info patch is likely still a bit out -- but clayg left a review and diff for the pre-req to get the final app onto every middleware in the pipeline | 21:12 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/821920 | 21:12 |
| timburke | (get rid of pipeline_property) | 21:12 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/824471 | 21:12 |
| timburke | (get rid of pipeline property harder, which i'll squash in) | 21:13 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/821921 | 21:13 |
| timburke | (chance to skip memcache when looking for account/container info) | 21:13 |
| timburke | #topic tempurl signatures in logs | 21:14 |
| timburke | #link https://bugs.launchpad.net/swift/+bug/1685798 | 21:14 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/822585 | 21:14 |
| timburke | mattoliver seemed on board with my plan to follow the register_swift_info pattern, so i squashed that down. i really like how that patch is shaping up! | 21:15 |
| timburke | i do think it'd be good to have someone besides matt and i review it, though | 21:16 |
| mattoliver | Yeah, there was no reason to build up an obscure map for every request. So moving it global at filter factory was a great move IMHO | 21:16 |
| timburke | anyone think they'll have some review bandwidth for it? | 21:16 |
| acoles | I can try | 21:18 |
| timburke | i'll try to poke clayg about it, too :-) | 21:18 |
| timburke | next up, i was going to have clayg tell us a bit about MPU expiration... but he doesn't seem to be around right now | 21:19 |
| timburke | so i'll check in on mattoliver instead :-) | 21:19 |
| timburke | #topic request tracing | 21:19 |
| timburke | mattoliver, it going? | 21:20 |
| mattoliver | Conference presentation is on Saturday. Think I'm mostly ready for it | 21:20 |
| mattoliver | Pushed up a new patch to instrument container and account update. | 21:21 |
| mattoliver | So now we can watch the object server send updates to the container on a put or delete. | 21:21 |
| timburke | nice! blocked on anything? just need reviews? | 21:22 |
| mattoliver | So now tracking more of the request | 21:22 |
| mattoliver | I don't want to touch anything else before the talk. | 21:22 |
| acoles | good luck with the presentation mattoliver | 21:23 |
| timburke | makes sense :-) | 21:23 |
| mattoliver | Afterwards, I'm thinking about squashing it all down into 1 or 2 patches | 21:23 |
| timburke | yeah, GL! | 21:23 |
| mattoliver | Then it should be easier to review | 21:23 |
| timburke | we can check back in next week, then | 21:24 |
| timburke | #topic open discussion | 21:24 |
| timburke | anything else we should bring up this week? | 21:24 |
| mattoliver | On the ring serialisation v2 etherpad I wrote up what it might look post ptg discussion. So discussions/comments on that would be good. | 21:26 |
| mattoliver | Don't have the link handy as I'm on my phone. | 21:26 |
| timburke | and i'm finally circling back around to that :-) | 21:26 |
| mattoliver | Cool | 21:27 |
| timburke | #link https://etherpad.opendev.org/p/swift-ring-v2-serialization | 21:27 |
| mattoliver | Thanks | 21:27 |
| timburke | all right, let's call it early | 21:29 |
| mattoliver | I also have a patch chain to add global error limiting.. but probably need to loop back and rebase that.. so maybe something to link for next week's meeting :) | 21:29 |
| timburke | always feel free to update the agenda :-) | 21:29 |
| mattoliver | Kk | 21:29 |
| timburke | thank you all for coming, and thank you for working on swift! | 21:30 |
| timburke | #endmeeting | 21:30 |
| opendevmeet | Meeting ended Wed Jan 12 21:30:12 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2022/swift.2022-01-12-21.00.html | 21:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2022/swift.2022-01-12-21.00.txt | 21:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2022/swift.2022-01-12-21.00.log.html | 21:30 |
| opendevreview | Merged openstack/swift master: Finer grained ratelimit for update https://review.opendev.org/c/openstack/swift/+/820962 | 22:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!