| opendevreview | Matthew Oliver proposed openstack/swift master: Quarantine asyncs older then reclaim_age https://review.opendev.org/c/openstack/swift/+/527296 | 05:15 |
|---|---|---|
| -opendevstatus- NOTICE: The paste service at paste.opendev.org will have a short (15-20) minute outage momentarily to replace the underlying server. | 17:08 | |
| opendevreview | Merged openstack/swift master: Leave updater per-device stats in recon for debugging https://review.opendev.org/c/openstack/swift/+/938524 | 18:31 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: db-replicator: include container name in logs, not just db path https://review.opendev.org/c/openstack/swift/+/933379 | 20:34 |
| opendevreview | Tim Burke proposed openstack/swift master: versioning: 411 PUTs with neither content-length nor transfer-encoding https://review.opendev.org/c/openstack/swift/+/931325 | 20:52 |
| timburke | #startmeeting swift | 21:00 |
| opendevmeet | Meeting started Wed Jan 15 21:00:13 2025 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 |
| mattoliver | o/ | 21:00 |
| mattoliver | I think Jian said he was going to the dentist | 21:01 |
| timburke | well, we can get started :-) | 21:01 |
| acoles | o/ | 21:01 |
| timburke | i didn't put much on the agenda | 21:01 |
| timburke | #link https://wiki.openstack.org/wiki/Meetings/Swift | 21:01 |
| timburke | first up | 21:02 |
| timburke | #topic gate breakage | 21:02 |
| kota | o/ | 21:02 |
| timburke | a recent bandit release broke out pep8 job | 21:02 |
| acoles | boo | 21:02 |
| mattoliver | oh yay | 21:02 |
| timburke | fortunately the fix was simple and we merged it quickly (thanks acoles!) | 21:03 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/939184 | 21:03 |
| patch-bot | patch 939184 - swift - CI: Remove B320 and B410 bandit skips (MERGED) - 1 patch set | 21:03 |
| acoles | kinda mean to fail because we *excluded* a test that gets removed | 21:03 |
| timburke | yeah, i felt similarly :-/ | 21:03 |
| acoles | WDYT about my doc follow-up https://review.opendev.org/c/openstack/swift/+/939209 ? | 21:03 |
| patch-bot | patch 939209 - swift - CI: document bandit tests by reference - 1 patch set | 21:03 |
| timburke | but i didn't bother to submit a bug about it or anything | 21:03 |
| JayF | I'll also note a new eventlet landed in global requirements; in oslo.log for eventlet-patched things we had to fix PipeMutex; I have a draft patch for swift's version of PipeMutex here https://review.opendev.org/c/openstack/swift/+/935987 in case you all see any issues; I know swift is a little special so may not be impacted but wanted to surface it. | 21:04 |
| patch-bot | patch 935987 - swift - Use stdlib "os" in PipeMutex - 1 patch set | 21:04 |
| timburke | approved! yeah, better to just link out to their docs | 21:04 |
| timburke | JayF, i saw that! py313 support here we come! | 21:04 |
| acoles | OIC you already did +2, thanks | 21:04 |
| JayF | FWIW, I don't intend on revising 935987; so if you wanna take it over feelfree, I'll likely abandon it in a few months if it doesn't move :) | 21:05 |
| timburke | 👍 | 21:06 |
| timburke | it's still weird to me that it causes some failures. i should probably dust it off to try to understand more | 21:06 |
| timburke | but anyway, if anyone notices any other gate failures where it's just bandit complaining in the pep8 job, feel free to recheck | 21:06 |
| timburke | #topic py2 is dead! | 21:07 |
| timburke | it's a carryover from last week, but now we've got more folks around to celebrate ;-) | 21:07 |
| JayF | timburke: my hunch is that since it unconditionally patches eventlet as part of the new job, and swift only conditionally does it, that there may just need to be some tweaking | 21:07 |
| mattoliver | woo hoo! | 21:07 |
| JayF | congrats :) | 21:08 |
| timburke | the big patch to take out py2 code paths seems to be getting close, too | 21:08 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/853697 | 21:08 |
| patch-bot | patch 853697 - swift - Remove py2-only code paths - 18 patch sets | 21:08 |
| timburke | thanks zaitcev, acoles, and mattoliver! | 21:08 |
| timburke | there's still more that can be done, of course | 21:09 |
| mattoliver | yeah, it was a big one.. that must've been a tedious patch to write timburke | 21:09 |
| mattoliver | thanks for working on it. | 21:09 |
| timburke | simplify our super() calls, use subtests, use f-strings... | 21:10 |
| mattoliver | We know basically have 3 +2s on it now. So shall I just +A it? | 21:10 |
| acoles | I wanna get a `yield from` in somewhere :D | 21:10 |
| mattoliver | yeah, we can changes things as we touch files now, which is nice. | 21:10 |
| acoles | mattoliver: do it! | 21:10 |
| mattoliver | done! | 21:11 |
| timburke | i'm definitely looking forward to having a modern codebase | 21:11 |
| timburke | \o/ | 21:11 |
| acoles | I'm looking forward to not having to nursemaid a py2 dev environment anymore | 21:11 |
| timburke | swift isn't the only place we might want to drop some old python versions -- i also put together a pyeclib patch | 21:11 |
| timburke | #link https://review.opendev.org/c/openstack/pyeclib/+/938098 | 21:11 |
| patch-bot | patch 938098 - pyeclib - Drop support for Python 2.7, 3.5-3.8 - 4 patch sets | 21:11 |
| timburke | maybe it would've been better for me to split that into "drop py2" and "drop old py3s" -- oh well | 21:12 |
| timburke | but there are more patches following it to do some further cleanups in pyeclib, too | 21:13 |
| mattoliver | nah it's fine. I feel like its the start of the new year, time to clean some stuff up. We can always put things back. We're cleaning swift, let's clean pyeclib too! | 21:14 |
| timburke | the nice thing about dropping the old versions of py3 is that we can switch to using latest image tags when building wheels | 21:16 |
| timburke | well, that, and taking out half our unit test jobs | 21:17 |
| timburke | all right, last topic i've got | 21:17 |
| timburke | #topic swift release | 21:18 |
| timburke | i meant to get together an authors/changelog patch last week, but still haven't. hopefully i'll find time this week | 21:18 |
| mattoliver | kk, first release without py2! | 21:19 |
| timburke | (though i'm a little skeptical -- i've got a downstream release to get out, too, and i'm moving over the weekend) | 21:19 |
| mattoliver | well no rush | 21:19 |
| timburke | mattoliver, and i'm not even sure it deserves a release note! we announced the last stable release supporting py2 back in *yoga*! | 21:20 |
| acoles | it'll be done when it's done | 21:20 |
| timburke | #topic open discussion | 21:20 |
| timburke | anything else we should bring up this week? | 21:20 |
| mattoliver | device id size has become an major issue downstream for us. So looks like ringv2 is bubbling back up | 21:21 |
| mattoliver | So I've started reloading and reviewing it again. | 21:21 |
| mattoliver | maybe we can include it in the new release too ;) | 21:21 |
| timburke | oh yeah! makes me happy i've kept p 834261 reasonably up-to-date | 21:22 |
| patch-bot | https://review.opendev.org/c/openstack/swift/+/834261 - swift - ring: Introduce a v2 ring format - 21 patch sets | 21:22 |
| mattoliver | We have so many devices in our clusters, be kinda can't expand much more without the abilty to use bigger ints (crazy problem to have). | 21:23 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: db-replicator: include container name in logs, not just db path https://review.opendev.org/c/openstack/swift/+/933379 | 21:24 |
| mattoliver | I feel like it was pretty close to landed a year or 2 ago.. so hopefully it wont take too much effort to land it (esp thanks to Tim keeping it up to date). | 21:24 |
| timburke | i remember OVH having a similar problem years ago -- i think we've mostly managed to avoid it by buying to maximize TB/drive rather than minimize $/TB | 21:25 |
| mattoliver | although I assume me landing the py2 deprecation, we have alot of patches now in conflict :P | 21:25 |
| mattoliver | yeah, don't think that'll cut it anymore. So no py2 and soon new ring format! | 21:26 |
| mattoliver | 2025 is going to be great! | 21:26 |
| timburke | almost certainly :-) it'll be interesting to see what bothers to get rebased the fastest | 21:26 |
| mattoliver | that's all I've got. I've been on vacation | 21:28 |
| timburke | all right, i think i'll call it then | 21:29 |
| timburke | thank you all for coming, and thank you for working on swift! | 21:29 |
| timburke | #endmeeting | 21:29 |
| opendevmeet | Meeting ended Wed Jan 15 21:29:20 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:29 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2025/swift.2025-01-15-21.00.html | 21:29 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2025/swift.2025-01-15-21.00.txt | 21:29 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2025/swift.2025-01-15-21.00.log.html | 21:29 |
| opendevreview | Merged openstack/swift master: CI: document bandit tests by reference https://review.opendev.org/c/openstack/swift/+/939209 | 22:04 |
| opendevreview | Merged openstack/swift master: Remove py2-only code paths https://review.opendev.org/c/openstack/swift/+/853697 | 22:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!