Wednesday, 2023-10-18

opendevreviewJianjian Huo proposed openstack/swift master: Container-server: add request timing metrics for PUT object/shard/container.  https://review.opendev.org/c/openstack/swift/+/89870005:15
opendevreviewAlistair Coles proposed openstack/swift master: s3api: Stop propagating storage policy to sub-requests  https://review.opendev.org/c/openstack/swift/+/89786409:53
opendevreviewAlistair Coles proposed openstack/swift master: Delete s3api MPU segments when the manifest expires  https://review.opendev.org/c/openstack/swift/+/80070109:53
opendevreviewAlistair Coles proposed openstack/swift master: s3api: Stop propagating storage policy to sub-requests  https://review.opendev.org/c/openstack/swift/+/89786410:27
opendevreviewAlistair Coles proposed openstack/swift master: Delete s3api MPU segments when the manifest expires  https://review.opendev.org/c/openstack/swift/+/80070110:27
opendevreviewAlistair Coles proposed openstack/swift master: sharder: reduce rate of 'Reclaimable db...' warnings  https://review.opendev.org/c/openstack/swift/+/89874214:54
opendevreviewAlistair Coles proposed openstack/swift master: sharder: reduce rate of 'Reclaimable db...' warnings  https://review.opendev.org/c/openstack/swift/+/89874215:02
opendevreviewTim Burke proposed openstack/swift master: Move InputProxy back to proxy-logging  https://review.opendev.org/c/openstack/swift/+/89874817:00
opendevreviewClay Gerrard proposed openstack/swift master: WIP: Refactor SLO size/etag sysmeta tests  https://review.opendev.org/c/openstack/swift/+/89646620:06
opendevreviewClay Gerrard proposed openstack/swift master: WIP: slo: refactor GET/HEAD response handling  https://review.opendev.org/c/openstack/swift/+/89357820:06
kotamorning20:57
mattolivermorning o/ 21:01
timburke#startmeeting swift21:03
opendevmeetMeeting started Wed Oct 18 21:03:27 2023 UTC and is due to finish in 60 minutes.  The chair is timburke. Information about MeetBot at http://wiki.debian.org/MeetBot.21:03
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.21:03
opendevmeetThe meeting name has been set to 'swift'21:03
timburkesorry i'm late21:03
timburkewho's here for the swift meeting?21:03
mattolivero/21:04
kotay/21:04
timburkebig topic for this week -- PTG's next week!21:05
timburkethanks everyone for voting for times on https://framadate.org/liKzqQw5lZQW8Pvf21:05
timburkelooks like while the block starting around this time would work fairly well for Monday, the rest of the days 1300-1800 UTC might work better21:06
mattoliveryeah, well having the ops feed back session at busy time always makes sense. 21:07
kotasounds ok21:08
timburkemattoliver and kota, am i right to think it'd be better to not try to shift the times around? i feel like that'd make for a really long first day for you guys21:08
mattoliverBut I'm easy.. I just wont be as productive during work during normal hours :P21:08
kotaany of things is ok because i'm at home those days for some reasons21:09
timburkewe can also aim for somewhat shorter sessions -- i don't really *want* to keep mattoliver up until 5am21:09
kotai got conjunctivitis in the last weeks and i have to stay home until the end of this month.21:09
mattoliver1800UTC is getting a little late, that'll be 5am21:09
kotaso there is no plan to go outside.21:10
timburkeoh no! hope you feel better soon kota21:10
mattoliveroh yeah, obviously my irc is lagging :P 21:10
mattoliveroh no kota :(21:10
timburkeseems like all the more reason to make sure you can get a good night's sleep21:10
kota:)21:10
timburkei'll start booking rooms after this meeting21:12
mattoliverkk21:12
indianwhocodeso/21:13
timburkeand i'll try to flesh out a few of my topics some more on https://etherpad.opendev.org/p/swift-ptg-caracal21:13
timburkeif you have anything else you'd like to bring up at the PTG, please add it to the etherpad!21:14
mattoliveryeah I just added a new one based off discussions with SRE yesterday21:14
mattoliverwill keep adding as we think of things21:14
timburkei saw that! i think i like it21:14
timburkei was having some other thoughts around recon recently, but somewhat going the other way -- have recon-cron emit gauges that could get picked up by something like statsd_exporter21:16
timburke#link https://review.opendev.org/c/openstack/swift/+/89573721:16
mattoliveroh yeah, well seems like a good topic to talk about at the ptg then! 21:16
timburkeit also got me thinking about two directions we could take on recon-cron -- (1) fold it into the object-updater, making it the only place that crawls async dirs, or (2) expand it to also do other monitoring, like counting primary/handoff parts and dropping that in recon/stats, too21:19
mattoliveroh, interesting idea. both have merits. I like less things walking the dirs.. but having the primary, handoff, deep handoff stats is so useful I think upstream should get that data21:21
timburkeabsolutely -- though it could also be done in some extra process/thread the replicator/reconstructor spins off21:22
mattolivercough audit watcher cough :P21:23
timburkethe biggest hesitation i've got about folding it into the updater is losing visibility when you've got your updater paused for whatever reason21:23
mattoliveryeah21:23
mattoliverthat's not great21:23
mattoliverrandom thoughts, link into the kernels inotify and look for asyncs that way, get triggered to run rather then walking?21:24
timburkeoh, now *there's* a thought... wouldn't help with the async counting today... but maybe it'd be reasonable to have it attempt some kind of audit for those, too?21:25
mattoliverie when a new async it written or removed21:25
mattoliversame could we said for an alert to quarantine21:25
timburkeit also seems to have a lot of overlap with some of clayg's overseer ideas21:26
timburkeanyway, we can discuss this more at the PTG :-)21:27
mattoliverI feel a nerd-snipe coming one.. good thing PTG is soon ;) 21:28
mattoliver*on21:28
timburkethat's the only thing i wanted to bring up, and to thank everyone for being so flexible on scheduling21:29
mattoliverasync counting, when started could walk to get current count. Insert a inotify to the kernel and then start counting/decrementing based on events in the async folder to keep the count up to date.21:29
mattoliverperiodic walk to keep in line (if we don't trust it). 21:29
mattoliverOh yeah, we're so close to PTG, no need to bring many more topics up in this meeting today :) 21:30
timburkeheh21:30
timburke#topic open discussion21:30
timburkeanything else we should discuss this week?21:30
timburkeseems like there's a good number of patches all making pretty good progress21:30
timburkemattoliver, just as a heads-up i proposed (and merged) https://review.opendev.org/c/openstack/swift/+/898440 as a follow-up to https://review.opendev.org/c/openstack/swift/+/897108 (silly yaml)21:32
indianwhocodesI was wondering if this patch was in our radar  https://review.opendev.org/c/openstack/swift/+/79986621:32
mattoliverkk, thanks timburke 21:33
indianwhocodes799866: Let WSGI apps signal that a client connection should be closed currently seems to have a merge conflict which I attempted fixing21:33
timburkeindianwhocodes, it hasn't been, but probably should be!21:33
mattoliveryeah, timburke (not that I've looked) but maybe we should start using priority reviews page again21:34
mattoliverI keep getting side tracked with other work. But slowly working through timburke swift-reload patch chain. We've been running it for a while in prod and it's pretty great.  21:35
timburkepriority reviews page is a great idea -- i've been remiss in tending it21:36
mattoliverchain starting:21:36
mattoliver#link https://review.opendev.org/c/openstack/swift/+/833174 21:36
mattoliverwe all have.. but maybe we should start using it. There are alot of patches out there that could do with some love.21:37
indianwhocodes<321:37
mattoliverI myself need to get better at reviewing again.. seems I'm not good at multi-tasking :P 21:37
indianwhocodespart-num api is wrapping up, so recalibrating my swift-priorities21:38
indianwhocodessame here, can expect more reviews from me now.21:38
timburke👍21:40
mattoliverI got nothing else seeing as the PTG starts next week, can't wait to talk and see everyone again!21:41
mattoliverjust wish it was in person21:41
timburkeat least we did a pretty good job of merging what was on the priority reviews page -- 7 of the ten patches that were listed when i last updated it have now merged21:41
mattoliver\o/21:41
indianwhocodes+221:41
timburkeditto -- i'd much prefer being jet-lagged but all together to having nearly everyone awake at odd hours while still also needing to do normal life things21:42
mattoliver^ 100x this21:43
kotaagree21:43
timburkeotoh, i suppose it's nice that we'll get some kota time! idk that we would otherwise, with the conjunctivitis ;-)21:44
timburkeall right, i think i'll end the meeting a little early21:46
timburkethank you all for coming, thank you for working on swift, and see you all next week!21:46
timburke#endmeeting21:46
opendevmeetMeeting ended Wed Oct 18 21:46:35 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)21:46
opendevmeetMinutes:        https://meetings.opendev.org/meetings/swift/2023/swift.2023-10-18-21.03.html21:46
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/swift/2023/swift.2023-10-18-21.03.txt21:46
opendevmeetLog:            https://meetings.opendev.org/meetings/swift/2023/swift.2023-10-18-21.03.log.html21:46
kotathanks21:46
timburkefeel better soon kota!21:46
opendevreviewASHWIN A NAIR proposed openstack/swift master: slo: part-number=N query paramater support  https://review.opendev.org/c/openstack/swift/+/89457022:08
opendevreviewASHWIN A NAIR proposed openstack/swift master: s3api: Support GET/HEAD request with PartNumber  https://review.opendev.org/c/openstack/swift/+/89458022:08
opendevreviewASHWIN A NAIR proposed openstack/swift master: proxy: handle ChunkReadTimeout in replicated GET path  https://review.opendev.org/c/openstack/swift/+/89723622:25
opendevreviewASHWIN A NAIR proposed openstack/swift master: proxy: stop logging backend timeouts as client disconnects  https://review.opendev.org/c/openstack/swift/+/89734322:25
opendevreviewASHWIN A NAIR proposed openstack/swift master: proxy: don't send multi-part terminator when no parts sent  https://review.opendev.org/c/openstack/swift/+/89756622:25
opendevreviewASHWIN A NAIR proposed openstack/swift master: proxy-server: de-duplicate _get_next_response_part method  https://review.opendev.org/c/openstack/swift/+/89091922:25
opendevreviewTim Burke proposed openstack/swift master: object: Return storage policy in headers  https://review.opendev.org/c/openstack/swift/+/89878322:33
opendevreviewTim Burke proposed openstack/swift master: proxy: Move object storage policy to expected header  https://review.opendev.org/c/openstack/swift/+/89878422:33

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