opendevreview | Jianjian Huo proposed openstack/swift master: s3api: more test cases for conditional writes. https://review.opendev.org/c/openstack/swift/+/950371 | 01:07 |
---|---|---|
mattoliver | I'm here this time. | 07:00 |
cschwede | I'm around as well :) | 07:00 |
mattoliver | Great! Sorry about last time, seems I got the meeting wrong by a 24 hours :P Epic TZ file : P | 07:01 |
mattoliver | Let's start the meeting then! | 07:02 |
mattoliver | #startmeeting swift | 07:02 |
opendevmeet | Meeting started Wed May 21 07:02:16 2025 UTC and is due to finish in 60 minutes. The chair is mattoliver. Information about MeetBot at http://wiki.debian.org/MeetBot. | 07:02 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 07:02 |
opendevmeet | The meeting name has been set to 'swift' | 07:02 |
mattoliver | Who's here for the swift meeting? | 07:02 |
kota | o/ | 07:02 |
cschwede | o/ | 07:02 |
mattoliver | \o/ first 0700 meeting in years | 07:03 |
mattoliver | Agenda is at: | 07:03 |
mattoliver | #link https://wiki.openstack.org/wiki/Meetings/Swift | 07:03 |
kota | good. it's healthy time for APAC | 07:03 |
mattoliver | kota: yeah, much easier for us too ;) | 07:03 |
cschwede | Same for EMEA, which is nice as well :) | 07:03 |
mattoliver | I've seeded some topics, and stole some from Tim more as an FYI | 07:04 |
mattoliver | First up one I made for cschwede :P | 07:04 |
mattoliver | #topic Bug triage | 07:04 |
mattoliver | Hows the triage going cschwede ? we probably should all be helping you tho. | 07:05 |
cschwede | On it. A bit slower than expected, but I'm running over bugs in NEW state atm. Updated ~10 earlier today, and will update our wiki page at https://wiki.openstack.org/wiki/Swift/BugTriage later | 07:05 |
mattoliver | nice one! | 07:06 |
mattoliver | #link https://wiki.openstack.org/wiki/Swift/BugTriage | 07:06 |
mattoliver | (so it links properly in logs) | 07:06 |
cschwede | Noticed a few very easy ones (like one-liner doc changes) that I might simply propose. Tagged as low-hanging fruit for now | 07:06 |
mattoliver | cool, yeah, if there too easy then even newbies might not be that interested in them. | 07:07 |
mattoliver | Thanks so much for working through these each fortnight! | 07:08 |
cschwede | Right, at the same time i don't want to close them simply without ack'ing the time of the reporter | 07:08 |
mattoliver | yeah fair enough | 07:08 |
mattoliver | I see all the extra emails in my inbox. Anything we need to do on our end? Can I help by taking a look. Though sounds like your marking them as expected | 07:10 |
cschwede | I run over a couple of bugs that need more investigation. I think I will create a list of things that need a second look and share that as well | 07:11 |
mattoliver | cool, sounds like a plan. Happy to double check anything your need. | 07:12 |
mattoliver | next topic then | 07:12 |
cschwede | Thanks Matt! | 07:12 |
mattoliver | #topic eventlet removal POC update | 07:12 |
mattoliver | This is just a place holder topic, no stress on how it's going.. but curious | 07:13 |
cschwede | Working on it, and also looked into cheroot mentioned earlier by timburke_. As expected it's a rabbit hole, but I hope to have a POC by the end of June | 07:13 |
cschwede | s/hope/plan | 07:13 |
mattoliver | lol, yeah huge rabbit hole | 07:14 |
mattoliver | Nice tho, I haven't looked at cheroot, so sounds like I might get nerdsniped :P | 07:14 |
mattoliver | I've let my manager know eventlet removal will be something that will have to happen in the future for us too, so thanks for giving at least a POC a crack so we can start moving in the right direction! | 07:15 |
cschwede | That's great news, very much appreciated! | 07:16 |
mattoliver | Want me to leave it in the meeting agenda to check in, or should I just add in every month or so? | 07:16 |
cschwede | I won't have an update in 2 weeks, but let's keep it on the agenda and we check in 4 weeks on the state. | 07:16 |
mattoliver | kk | 07:17 |
mattoliver | #topic SO_TIMESTAMP/SO_TIMESTAMPING patch | 07:17 |
mattoliver | #link https://review.opendev.org/c/openstack/swift/+/944103 | 07:17 |
mattoliver | I bring this up because its one of mine :P | 07:17 |
mattoliver | I think its coming along nicely, timburke_ has been reviewing it, so starting to look good. | 07:18 |
mattoliver | This patch will allow us to track the time the request hits the server, not just when we accept it. Also the time it waited on the accept queue can also be logged. | 07:19 |
cschwede | That's the one we discussed at the PTG, right? Is there any issue with swift-bench that needs a fix (unrelated to this patch), I see Tim mentioned that in the comments? | 07:19 |
mattoliver | yup | 07:19 |
mattoliver | if the client is doing pipelined requests, then the fields in the logs (if defined) will just be '-' because we already have the connection open. | 07:19 |
mattoliver | Although, I guess I better make a test for that or something :) | 07:20 |
mattoliver | We want to alert of time spent in accept queues because this is a sign of overloading. Maybe it's interesting to others too. | 07:20 |
mattoliver | especally when a client's time a request took and differ then our logs because of the wait. | 07:21 |
cschwede | Yes, definetely useful | 07:21 |
mattoliver | Otherwise, not too much to report there. If anyone wants to take it for a spin or review great. :) | 07:21 |
mattoliver | next topic | 07:22 |
mattoliver | #topic ringv2 | 07:22 |
mattoliver | We talked about this at the PTG too.. | 07:22 |
mattoliver | in fact alot of the serialisation design work happened with everyone at PTGs past | 07:23 |
cschwede | Looks like it got some updates very recently | 07:23 |
mattoliver | But it still hasn't landed :( | 07:23 |
mattoliver | But good news is, we've been running it prod (defaulting to v1). | 07:23 |
cschwede | That's the case for quite some time already, isn't it? | 07:24 |
mattoliver | Soak tested a few racks by running 4-byte devid rings in our biggest cluster. | 07:24 |
mattoliver | yup | 07:24 |
mattoliver | which went well. | 07:24 |
mattoliver | And just today we've enabled ringv2 in some of our smaller clusters | 07:24 |
mattoliver | *small prod clusters | 07:25 |
cschwede | That's great news, sounds like it is working as expected then | 07:25 |
mattoliver | timburke_: responded and cleaned up some stuff, so I hope we can finally land it. | 07:25 |
mattoliver | Once it lands I want to blow the dust of my last_part_table patch, so reconstructors can find missing frags without rebuilding after a rebalance (hopefully remove the need for handoffs_only option) | 07:27 |
mattoliver | Well remove the need after a rebalance at least | 07:27 |
mattoliver | The last few topics are really just FYIs, as they're topics in the other timed meeting. | 07:28 |
mattoliver | #topic aws-chunked | 07:28 |
mattoliver | So I went on the landing rampage a few days ago, and went and landed the first 2 patches in the chain. | 07:29 |
mattoliver | They work great and figured people can do any more polishing in follow up patches. | 07:29 |
mattoliver | This means we finally have support for aws-chunked so latest boto should now work. | 07:30 |
mattoliver | The next patch in the chain is important tho | 07:30 |
mattoliver | #link https://review.opendev.org/c/openstack/swift/+/944073 | 07:30 |
mattoliver | As it actually means we actually start checking the crcs users give us :P | 07:31 |
mattoliver | So want it to land before the next actual swift release | 07:31 |
cschwede | Supporting latest boto release and crc checking sound like very useful fixes | 07:31 |
mattoliver | There are some test requirements causing issues, so couldn't land that one yet. | 07:31 |
mattoliver | yeah! | 07:32 |
mattoliver | And I hope we can start using more of the crcs in swift too. | 07:32 |
mattoliver | Moving on | 07:32 |
mattoliver | #topic swiftclient release | 07:33 |
mattoliver | timburke_ needs to cut a new swiftclient release soon. | 07:33 |
cschwede | Nice, ptetera and his fix now mentioned as author in the 4.8.0 release | 07:33 |
mattoliver | \o/ | 07:33 |
mattoliver | Final topic I have | 07:34 |
mattoliver | #topic conditional PUTs for s3api | 07:34 |
mattoliver | #link https://review.opendev.org/c/openstack/swift/+/949680 | 07:35 |
mattoliver | This one we found by a user using some s3 client (as normal). | 07:35 |
mattoliver | AWS has finally added If-Match and If-None-Match support now. We've always had If-None-Match: in swift | 07:36 |
mattoliver | So that patch starting using it in s3api | 07:37 |
mattoliver | oh and it's already merged, so that makes life eaier | 07:37 |
cschwede | :) | 07:37 |
mattoliver | There is more work to do on it, but at least we could easily support part of it. | 07:38 |
mattoliver | Anyway, that's all I got for the agenda so.. | 07:38 |
mattoliver | #topic open discussion | 07:38 |
mattoliver | Anything else anyone wants to bring up the meeting? | 07:39 |
mattoliver | hows ptetera going? | 07:39 |
cschwede | He's working on more patches - will meet with him later today | 07:40 |
mattoliver | cool! | 07:40 |
cschwede | Nothing else to bring up today on my side, thx | 07:41 |
mattoliver | We're working on base OS migration stuff. Which means tracing is finally coming onto our upcoming roadmap. So tracing might finally land soon! I've been cleaning it up. Will push the rebased version (which is always a pain) tomorrow probably. | 07:43 |
mattoliver | I should probably go a rebase the vsaio branch too. | 07:43 |
mattoliver | Anyway, that's all I have | 07:43 |
cschwede | Are you using tracing to detect differences between OS releases? | 07:43 |
mattoliver | I could. but more the fact so I can actually build modern python packages :P | 07:44 |
mattoliver | and therefore start actaully using otel :P | 07:44 |
cschwede | :) | 07:44 |
mattoliver | Cool, well I'll call it early. I might go start making dinner for the fam :) | 07:45 |
mattoliver | Thanks for coming and thanks for working on swift! | 07:45 |
mattoliver | #endmeeting | 07:45 |
opendevmeet | Meeting ended Wed May 21 07:45:49 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 07:45 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2025/swift.2025-05-21-07.02.html | 07:45 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2025/swift.2025-05-21-07.02.txt | 07:45 |
opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2025/swift.2025-05-21-07.02.log.html | 07:45 |
cschwede | Thanks Matt, have a nice evening! | 07:45 |
mattoliver | Great to be able to meet and chat with you more regularly cschwede ! Have a great day! | 07:46 |
kota | Thanks mattoliver! | 07:46 |
opendevreview | Alistair Coles proposed openstack/swift master: s3api: more test cases for conditional writes. https://review.opendev.org/c/openstack/swift/+/950371 | 09:28 |
opendevreview | Merged openstack/swift master: s3api: more test cases for conditional writes. https://review.opendev.org/c/openstack/swift/+/950371 | 14:46 |
opendevreview | Clay Gerrard proposed openstack/swift master: Make part_listdir_percent configurable https://review.opendev.org/c/openstack/swift/+/949953 | 16:05 |
opendevreview | Merged openstack/swift master: Add labeled metrics to proxy-logging https://review.opendev.org/c/openstack/swift/+/917711 | 17:14 |
-opendevstatus- NOTICE: Gerrit is being updated to the latest 3.10 bugfix release as part of early prep work for an eventual 3.11 upgrade. Gerrit will be offline momentarily while it restarts on the new version. | 17:35 | |
opendevreview | Tim Burke proposed openstack/swift master: s3api: Validate additional checksums on upload https://review.opendev.org/c/openstack/swift/+/944073 | 20:03 |
opendevreview | Tim Burke proposed openstack/swift master: s3api: Add support for crc64nvme checksum calculation https://review.opendev.org/c/openstack/swift/+/946141 | 20:03 |
opendevreview | Tim Burke proposed openstack/swift master: Add labeled metrics to s3api https://review.opendev.org/c/openstack/swift/+/939481 | 20:03 |
opendevreview | Shreeya Deshpande proposed openstack/swift master: Provide some s3 helper methods for other middlewares to use. https://review.opendev.org/c/openstack/swift/+/940791 | 20:27 |
opendevreview | Shreeya Deshpande proposed openstack/swift master: proxy-logging: Add real-time transfer bytes counters https://review.opendev.org/c/openstack/swift/+/930918 | 20:45 |
opendevreview | Tim Burke proposed openstack/swift master: Revert "Remove unused MANIFEST.in" https://review.opendev.org/c/openstack/swift/+/950615 | 23:49 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!