Wednesday, 2022-03-23

opendevreviewMatthew Oliver proposed openstack/python-swiftclient master: Add formpost subcommand to generate signature  https://review.opendev.org/c/openstack/python-swiftclient/+/83395401:10
seongsoochoHi !  I checked the ptg schedule. Is the current schedule on the ptg website confirmed? 01:46
opendevreviewMatthew Oliver proposed openstack/swift master: db: shard up the DatabaseBroker pending files  https://review.opendev.org/c/openstack/swift/+/83055103:36
mattoliverseongsoocho:  Tim said it was 13 UTC start everyday. So quite late for us. Though I think less late for you :)03:42
seongsoochomattoliver:  oh I see. thanks. 13 UTC is 22pm to me :-)     03:44
kotamattoliver: do we find the official schedule link for the ptg schedule? It would be nice to have the schedule to me to get approval to shift my daily work time on the week.04:42
kotaUsually, it's at https://www.openstack.org/ptg/ but not found at this time.04:42
mattoliverkota:  official one doesn't seem to be up yet, but https://ptg.opendev.org/ptg.html seems to be correct.05:19
kotamattoliver: oh nice! it looks we have 3 days slot so we will need to up at night on Tue, Wed, Thu, right?05:22
mattoliver4 days I see, Mon, tues, wed, thurs in UTC so Those evening for us (unless my timezone fu is off) 05:50
mattoliveror rather over our nights05:51
opendevreviewTim Burke proposed openstack/swift master: CI: Run CORS tests under py3  https://review.opendev.org/c/openstack/swift/+/83436306:12
*** clarkb is now known as Guest10607:31
kotamattoliver: ok, thx09:23
* kota was missing Mom starts from 13:00 UTC on the leftside09:24
*** Guest106 is now known as clarkb15:24
opendevreviewAlistair Coles proposed openstack/swift master: WIP Refactor rate-limiting helper into a class  https://review.opendev.org/c/openstack/swift/+/83496018:32
opendevreviewMerged openstack/swift master: CI: Run CORS tests under py3  https://review.opendev.org/c/openstack/swift/+/83436319:15
timburke__almost meeting time!20:56
kotagood morning20:58
*** timburke__ is now known as timburke20:58
timburke#startmeeting swift21:00
opendevmeetMeeting started Wed Mar 23 21:00:02 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.21:00
opendevmeetThe meeting name has been set to 'swift'21:00
timburkewho's here for the swift meeting?21:00
kotao/21:00
mattolivero/21:00
timburkeas usual, the agenda's at https://wiki.openstack.org/wiki/Meetings/Swift21:02
timburkenot a whole lot for this week (which is good; i've got another meeting midway through this one :P)21:03
timburke#topic PTG21:03
timburkejust another reminder :-)21:03
timburke#link https://etherpad.opendev.org/p/swift-ptg-zed21:03
timburkehas development topics21:03
timburkeand if you haven't already, register at21:03
timburke#link https://openinfra-ptg.eventbrite.com/21:03
timburkemeeting times and rooms are on the etherpad now, too21:04
mattoliverI've convinced csmart to come and had a chat with some of our SREs and recommended they look at adding any topics they're interested in (and turn up for them). 21:04
timburkesorry again about the times kota and mattoliver -- i figure we ought to try to front-load the time slot so you guys can get to bed and not worry about missing much21:05
kotano problem21:05
timburkebut i also didn't want to invade your saturday21:06
mattoliveryeah no probs, I'll have an excuse for asking silly questions :P 21:06
timburke#topic py2 support in swiftclient21:06
timburkeit's gone!21:06
timburkesomeone's apparently been champing at the bit to clean up the cruft from a mixed py2/py3 codebase, too, so there's been a bunch of patches landing recently21:07
mattoliveryeah, that's been awesome21:08
timburkeif anyone wants to look at some more of that sort of thing, i pushed up https://review.opendev.org/c/openstack/python-swiftclient/+/834751 to address some things i noticed while reviewing the other patches21:08
mattoliverI have to learn to stop leaning on six as a python crutch now :) 21:09
timburkei also rebased a few old patches i had -- my hope is that dropping py2 support will make us a little more inclined to show the client some love21:10
mattoliverAlso meant I had to rebase https://review.opendev.org/c/openstack/python-swiftclient/+/833954 already, but that's a good thing, actaully makes it cleaner, no more getattr. 21:10
timburke\o/21:10
timburkeserver side, i expect we'll want to wait at least a few months before doing a py2 cleanup -- all that churn will make it really difficult to cherry-pick fixes to the stable/yoga branch if anything comes up21:11
timburke#topic ring v221:12
timburkejust a status update: i started tinkering with a new approach that brings in the index idea from last ptg21:13
timburke#link https://review.opendev.org/c/openstack/swift/+/83426121:13
mattoliverIt's an awesome implementation Tim. So easy to use.21:13
mattoliverI did a quick follow up that adding a few extra features, helper to write the version string/magic21:14
timburkemattoliver's done a great job building on top of it, too -- we'll keep working on making that chain great in time to talk about it at *this* ptg :-)21:14
mattoliverI've got a new last_part_table that stores index as well21:14
timburkethat's the important thing to keep in mind: what we *really want* is a way to track some ring history so we can make replication/reconstruction smarter21:15
mattoliverand almost completed a new version of the reconstructor that uses it. And does so much earlier so we don't waste ssync connections. Hopefully get the first version of that up today. 21:15
mattoliveryup +10021:15
mattoliverBecause I have the index at hand, I only need to check if one of the neighbours/partners happen to also be the same index. 21:16
mattoliverand do things only then. 21:16
mattoliverSo far it's coming out much nicer (famous last words) :P 21:17
timburkethere's definitely some sneaky bits to the new format -- i do a bunch of Z_FULL_FLUSH flushes as we're writing so we can come back and just start reading, and there's this one point where i swap out the decompressor for the GzipFile so i can force it to be uncompressed so i know exactly how many bytes i'm going to write out...21:18
timburkebut on the whole, i think i drew sane boundaries around it so we mostly don't have to think about it :-)21:18
timburke(speaking of famous last words...)21:18
mattoliverlol21:18
mattoliverwell just so see the (de)serialize_v2 in the follow up. I think it shows it's a winner :) 21:19
mattoliver*go21:19
timburkeyeah, i was about to say -- if anyone wants to start loading it into their head, that patch is a great place to start so you can see how the RingWriter api is actually used21:20
timburke#topic storage server ratelimiting21:20
timburkewe've been seeing issues where servers get overloaded (often trying to get one particular object) and it starts causing broader performance degradation21:22
timburkeclayg's been playing with getting some ratelimiting in there so we can just 503 before hitting disk21:23
opendevreviewMatthew Oliver proposed openstack/swift master: POC: Random hack to add some kind of disk stat based ratelimiting  https://review.opendev.org/c/openstack/swift/+/83409921:24
timburkethe idea being that we want to still have the iops to be able to serve the *other* bytes that are on this poor disk21:24
mattoliversorry, that got me thinking about that patch I was playing with, that I still had stuck on my laptop. 21:25
timburkeand mattoliver's been looking at how to make it so ops don't need to pick some number out of the air, but make it a little more adaptive21:25
timburkethis reminds me of issues we've had with overloaded container-servers, too -- i think the main take-away is that we need to (and are working towards) have backend servers be more defensive and shed load21:27
mattoliverYeah I should blow some dust of the adaptive code. 21:27
timburkethat's all i've got21:27
timburke#topic open discussion21:27
mattoliverspeaking of container servers I'd love to have someone take a look at: https://review.opendev.org/c/openstack/swift/+/83055121:27
mattoliverIt seems to be a big improvement to concurrent access to the same database. 21:28
timburkeyeah! i can try to take a look this week21:28
mattoliverat least in my vsaio.21:28
mattoliverthanks. 21:28
mattoliverWell I have another meeting to get t21:28
mattoliverto21:28
mattoliverSo that's all I have.. for now :P 21:28
kotano topics from me, thx timburke21:29
timburkeall right21:29
timburkethank you all for coming, and thank you for working on swift!21:29
timburke#endtopic21:29
timburke#endmeeting21:30
opendevmeetMeeting ended Wed Mar 23 21:30:12 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)21:30
opendevmeetMinutes:        https://meetings.opendev.org/meetings/swift/2022/swift.2022-03-23-21.00.html21:30
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/swift/2022/swift.2022-03-23-21.00.txt21:30
opendevmeetLog:            https://meetings.opendev.org/meetings/swift/2022/swift.2022-03-23-21.00.log.html21:30
csmartthanks mattoliverau, sorry my dog hurt his foot so was at the vet this morning..21:39
csmart/reads scrollback21:44
timburkecsmart, hope he gets better quickly!22:26
csmartThanks! Looks like soft tissue damage, they gave him an anti-inflammatory shot and we'll see how he goes...22:36
opendevreviewTim Burke proposed openstack/swift master: probe: Make sure we know what version of python we're testing in the gate  https://review.opendev.org/c/openstack/swift/+/79350322:37
opendevreviewTim Burke proposed openstack/swift master: utils: Make PipeMutex marginally more robust to "Too many open files" errors  https://review.opendev.org/c/openstack/swift/+/75339723:16

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!