| opendevreview | Tim Burke proposed openstack/swift master: Fix unicode-header-name handling https://review.opendev.org/c/openstack/swift/+/938631 | 01:44 |
|---|---|---|
| opendevreview | Alistair Coles proposed openstack/swift master: Improve get_logger tests re. statsd prefix https://review.opendev.org/c/openstack/swift/+/936406 | 09:16 |
| opendevreview | Alistair Coles proposed openstack/swift master: Improve get_logger tests re. statsd prefix https://review.opendev.org/c/openstack/swift/+/936406 | 09:42 |
| opendevreview | Merged openstack/swift master: tests: use a method to test a method https://review.opendev.org/c/openstack/swift/+/938630 | 10:56 |
| opendevreview | Alistair Coles proposed openstack/swift feature/mpu: mpu: check for manifest in versions container during parts audit https://review.opendev.org/c/openstack/swift/+/933122 | 11:02 |
| opendevreview | Alistair Coles proposed openstack/swift feature/mpu: mpu: append link to dependant mpu resource to override etag https://review.opendev.org/c/openstack/swift/+/938189 | 11:02 |
| opendevreview | Alistair Coles proposed openstack/swift feature/mpu: WIP mpu: make object server discover potential orphans https://review.opendev.org/c/openstack/swift/+/938141 | 11:02 |
| opendevreview | Alistair Coles proposed openstack/swift feature/mpu: mpu: make ContainerBroker discover potential orphans https://review.opendev.org/c/openstack/swift/+/937279 | 11:17 |
| opendevreview | Merged openstack/swift feature/mpu: mpu: check for manifest in versions container during parts audit https://review.opendev.org/c/openstack/swift/+/933122 | 15:10 |
| *** cschwede_ is now known as cschwede | 15:21 | |
| opendevreview | Alistair Coles proposed openstack/swift master: WIP: s3 compat test for null versions https://review.opendev.org/c/openstack/swift/+/938678 | 16:27 |
| opendevreview | Merged openstack/swift master: Improve get_logger tests re. statsd prefix https://review.opendev.org/c/openstack/swift/+/936406 | 17:18 |
| opendevreview | Tim Burke proposed openstack/swift master: Remove py2-only code paths https://review.opendev.org/c/openstack/swift/+/853697 | 18:15 |
| opendevreview | Tim Burke proposed openstack/swift master: obj: Add option to tune down etag validation in object-server https://review.opendev.org/c/openstack/swift/+/927857 | 18:21 |
| opendevreview | Merged openstack/swift master: obj: Add option to tune down etag validation in object-server https://review.opendev.org/c/openstack/swift/+/927857 | 20:59 |
| timburke | #startmeeting swift | 21:00 |
| opendevmeet | Meeting started Wed Jan 8 21:00:08 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 |
| jianjian | o/ | 21:01 |
| timburke | i know mattoliver is still out -- kota was around earlier this week, but doesn't seem logged in now | 21:02 |
| timburke | well, may as well start :-) | 21:03 |
| timburke | i actually updated the agenda! | 21:03 |
| timburke | #link https://wiki.openstack.org/wiki/Meetings/Swift | 21:03 |
| timburke | first up | 21:03 |
| timburke | #topic welcome jianjian to the core team | 21:03 |
| timburke | the cores had all been talking about jianjian's good reviews, and wanted to extend an invitation to be on the upstream core team | 21:04 |
| timburke | yesterday i reached out, he accepted, and i see we've already merged some code because of it! | 21:05 |
| timburke | 🎉 | 21:05 |
| jianjian | Thanks, Tim! 😇 I am very excited. | 21:05 |
| timburke | glad to have you with us doing more and more upstream work :-) | 21:05 |
| timburke | i think this may be the first time i've had occasion to update the core list, so i'm excited too :D | 21:06 |
| jianjian | after working on swift for more than 2.5 years, I feel the more I work on, the more I found out I need to learn more. will use +2 wisely and look forward to contributing more into swift universe 💪 | 21:06 |
| timburke | next up | 21:07 |
| timburke | #topic py2 is dead! | 21:07 |
| jianjian | thanks for your kind word, Tim, and thanks other core devs for helping me out all the time! | 21:07 |
| timburke | coming up on the new year, i was a little impulsive and went ahead and merged p 853590 | 21:08 |
| patch-bot | https://review.opendev.org/c/openstack/swift/+/853590 - swift - Drop py2 support (MERGED) - 23 patch sets | 21:08 |
| jianjian | yeah, let's use python 3! | 21:08 |
| timburke | it's been a long time coming, i think we're finally ready, and i'm so excited to not have that albatross around our neck for another year | 21:09 |
| jianjian | 🎉 | 21:09 |
| timburke | that patch just updated packaging and gate jobs | 21:09 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/853697 | 21:09 |
| patch-bot | patch 853697 - swift - Remove py2-only code paths - 13 patch sets | 21:09 |
| timburke | does a lot to get rid of the old py2-only code paths | 21:10 |
| timburke | (including removing our dependency on six) | 21:10 |
| timburke | there's other clean up that can be done, but i figured a net-reduction of ~1400 LoC is a good place to start | 21:11 |
| timburke | though it looks like i need to fix a pep8 error :-/ | 21:12 |
| jianjian | lol, it's a huge patch | 21:12 |
| timburke | yeah... sorry everybody! but zaitcev has already +2ed it, and i know acoles is interested in taking a look sooner rather than later, too | 21:12 |
| timburke | (i'm guessing so he can deal with the fallout on feature/mpu sooner rather than later) | 21:13 |
| jianjian | seems there is a small flake8 check error, https://zuul.opendev.org/t/openstack/build/125a50c965da48679b126872236a66d2 | 21:13 |
| timburke | but there's at least one other project we might want to look at, too! | 21:14 |
| jianjian | yeah, we will be happy to rebase our patches on top of it | 21:14 |
| timburke | #link https://review.opendev.org/c/openstack/pyeclib/+/938098 | 21:14 |
| patch-bot | patch 938098 - pyeclib - Drop support for Python 2.7, 3.5-3.8 - 4 patch sets | 21:14 |
| timburke | that one also drops some older py3 versions, mainly just because they're EOLed | 21:15 |
| jianjian | nice, so 3.8 is also an old version already | 21:16 |
| timburke | but i feel pretty OK about it, especially given our infrequent updates to pyeclib -- i think anyone still on old python versions will be content to remain on 1.6.4 for a while | 21:16 |
| timburke | py39 will EOL this October | 21:17 |
| jianjian | yes, "Python 3.8 reached its end-of-life (EOL) on October 7, 2024" | 21:17 |
| jianjian | woww | 21:17 |
| timburke | there's a reason we really need to look at upgrading our py3 again already, despite having only *just* finished the last transition | 21:18 |
| jianjian | ack, they are EOL already | 21:18 |
| timburke | there are some other follow-on pyeclib patches stacked on top of that to clear out some cruft from the old versions | 21:19 |
| timburke | big one is p 938101 -- get to take out a whole module that was only included for the sake of py2! | 21:20 |
| patch-bot | https://review.opendev.org/c/openstack/pyeclib/+/938101 - pyeclib - Get rid of custom enum module - 3 patch sets | 21:20 |
| timburke | next up | 21:21 |
| timburke | #topic swift release | 21:21 |
| timburke | i've been thinking that it's about time to get a new one out! | 21:21 |
| timburke | especially since we merged https://review.opendev.org/c/openstack/swift/+/937493 | 21:22 |
| patch-bot | patch 937493 - swift - xprofile: Stop using eval() (MERGED) - 1 patch set | 21:22 |
| timburke | i'll try to get some release notes together this week, so keep an eye out for the patch! | 21:23 |
| jianjian | Nice! | 21:23 |
| timburke | that's all i've got for this week | 21:23 |
| timburke | #topic open discussion | 21:24 |
| timburke | anything else we should bring up? or i can let you go a little early ;-) | 21:24 |
| jianjian | big thanks to Tim for working on those py2/py3 upstream patches, it's a lot of work! | 21:24 |
| timburke | i've been itching to do it for *so long* | 21:24 |
| timburke | literally the whole time i've been PTL has been a long slog of trying to make this transition happen, so i'm so glad it's nearly there | 21:26 |
| jianjian | would love not to worry about py2 anymore, I will also look at those patches | 21:26 |
| jianjian | no other topic from me | 21:27 |
| timburke | oh! but speaking of! i finally figured out a probe test issue that lead me to do this: https://github.com/openstack/swift/blob/master/test/probe/test_reconstructor_rebuild.py#L529-L537 | 21:27 |
| timburke | and in the process discovered another long-running py3 http parsing bug :-( | 21:28 |
| timburke | but at least i've got a fix! https://review.opendev.org/c/openstack/swift/+/938631 | 21:28 |
| patch-bot | patch 938631 - swift - Fix unicode-header-name handling - 1 patch set | 21:28 |
| jianjian | a long-running bug... wow | 21:29 |
| timburke | if you get a chance, i'd appreciate some eyes on it | 21:29 |
| jianjian | yeah, interested to look into it | 21:30 |
| timburke | it's https://bugs.python.org/issue37093 all over again, just that i didn't work around it completely before | 21:30 |
| timburke | thanks! | 21:30 |
| timburke | all right, that's all i've got, too | 21:30 |
| timburke | happy new year! it's already looking like the best year yet for swift :-) | 21:31 |
| timburke | thank you for coming, and thank you for working on swift! | 21:31 |
| timburke | #endmeeting | 21:31 |
| opendevmeet | Meeting ended Wed Jan 8 21:31:29 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2025/swift.2025-01-08-21.00.html | 21:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2025/swift.2025-01-08-21.00.txt | 21:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2025/swift.2025-01-08-21.00.log.html | 21:31 |
| jianjian | happy new year to all! | 21:31 |
| opendevreview | Tim Burke proposed openstack/swift master: Remove py2-only code paths https://review.opendev.org/c/openstack/swift/+/853697 | 22:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!