| opendevreview | ASHWIN A NAIR proposed openstack/swift master: slo: part-number=N query paramater support https://review.opendev.org/c/openstack/swift/+/894570 | 00:30 |
|---|---|---|
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: slo: part-number=N query paramater support https://review.opendev.org/c/openstack/swift/+/894570 | 00:35 |
| opendevreview | Matthew Oliver proposed openstack/swift master: s3api: Add basic GET object-lock support https://review.opendev.org/c/openstack/swift/+/897108 | 03:14 |
| opendevreview | Matthew Oliver proposed openstack/swift master: s3api: Add basic GET object-lock support https://review.opendev.org/c/openstack/swift/+/897108 | 05:01 |
| opendevreview | kim nuri proposed openstack/swift master: WIP: Display host as both hostname and IP in swift-recon https://review.opendev.org/c/openstack/swift/+/893583 | 12:15 |
| opendevreview | Alistair Coles proposed openstack/swift master: proxy-server: de-duplicate _get_next_response_part method https://review.opendev.org/c/openstack/swift/+/890919 | 15:35 |
| opendevreview | Alistair Coles proposed openstack/swift master: DNM: proxy GET - close the parts iter https://review.opendev.org/c/openstack/swift/+/897343 | 15:35 |
| opendevreview | Alistair Coles proposed openstack/swift master: proxy: fix replicated GET ChunkReadTimeout handling https://review.opendev.org/c/openstack/swift/+/897236 | 15:36 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: s3api: Support GET/HEAD request with PartNumber https://review.opendev.org/c/openstack/swift/+/894580 | 18:38 |
| timburke | sorry i'm late! | 21:03 |
| timburke | anyone around for a swift meeting? | 21:03 |
| mattoliver | o/ | 21:03 |
| timburke | #startmeeting swift | 21:04 |
| opendevmeet | Meeting started Wed Oct 4 21:04:03 2023 UTC and is due to finish in 60 minutes. The chair is timburke. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:04 |
| opendevmeet | The meeting name has been set to 'swift' | 21:04 |
| timburke | i've just got a handful of follow-ups to last week's meeting | 21:04 |
| timburke | #topic vPTG | 21:05 |
| kota | o/ | 21:05 |
| timburke | we still need to list discussion topics at | 21:05 |
| timburke | #link https://etherpad.opendev.org/p/swift-ptg-caracal | 21:05 |
| uhyeongjo | o/ | 21:06 |
| mattoliver | Oh yeah, I was on vacation for the week and half. I'll do my best to seed that with some topics | 21:06 |
| timburke | and i still need to book meeting times -- if you haven't already (thanks kota and acoles!), please indicate preferred time slots at | 21:06 |
| timburke | #link https://framadate.org/liKzqQw5lZQW8Pvf | 21:06 |
| mattoliver | *last week and a half | 21:06 |
| mattoliver | Kk | 21:06 |
| acoles | o/ | 21:07 |
| timburke | PTG's just over two weeks away -- it'll be good to have some video time with each other :-) | 21:07 |
| timburke | #topic py312 | 21:08 |
| timburke | greenlet has shipped a 3.12-ready release now, at least -- eventlet is still likely to take a bit to get support | 21:09 |
| timburke | but zaitcev reviewed the whole chain to let us drop our use of distutils! | 21:10 |
| mattoliver | Oh cool | 21:10 |
| mattoliver | I hadnt seen that chain. (Still catching up) | 21:10 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/896473 | 21:11 |
| timburke | is the end of it | 21:11 |
| mattoliver | K | 21:11 |
| timburke | i might start looking at getting py312 gate jobs up for swiftclient and pyeclib -- my expectation is those should Just Work | 21:12 |
| timburke | last bit from last week: | 21:12 |
| timburke | #topic stable gates | 21:12 |
| timburke | xena and wallaby are now fixed, and the backport to wallaby that kicked this all off has landed | 21:13 |
| mattoliver | Thanks for always digging in an fixing the gates timburke | 21:14 |
| timburke | victoria was passing when i checked: https://review.opendev.org/c/openstack/swift/+/896858 | 21:14 |
| timburke | but usurri was not: https://review.opendev.org/c/openstack/swift/+/896859 | 21:14 |
| timburke | still haven't tried train, but i'd guess it's probably broken | 21:15 |
| timburke | (otoh, i would have said the same for victoria... i was rather surprised it passed) | 21:15 |
| timburke | stein and earlier have already been EOL'ed | 21:16 |
| timburke | looking at active patches... | 21:18 |
| timburke | #topic s3api partNumber API | 21:18 |
| timburke | there's been a good bit of activity on the chain ending with | 21:18 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/894580 | 21:19 |
| timburke | clayg has some nice refactoring in slo and its tests | 21:20 |
| timburke | then indianwhocodes builds from there, first adding a part-number query param to SLO, and letting s3api do the appropriate translations for S3 clients | 21:21 |
| acoles | +1 there's some great cleanup in the refactor | 21:21 |
| mattoliver | cool | 21:21 |
| timburke | while on the topic of s3 compat... | 21:22 |
| timburke | #topic object lock "support" | 21:22 |
| mattoliver | yeah this :) | 21:22 |
| mattoliver | So it turns out latest version of ansible that using the aws s3 module doesn't work with swift s3api | 21:23 |
| timburke | mattoliver pushed up a patch recently to at least let us respond to some lock-related requests | 21:23 |
| timburke | #link https://review.opendev.org/c/openstack/swift/+/897108 | 21:23 |
| mattoliver | and it's because they create a bucket and then attempt to pull down the object-lock-configuration | 21:23 |
| mattoliver | which we don't support.. so instead of returning the "not configured" error we treat it as a normal bucket request | 21:24 |
| timburke | fwiw, the patch is very similar to our initial "support" for various versioning-related APIs -- i'm all for it | 21:24 |
| mattoliver | so basically it was returning '{}' (empty dict) in boto | 21:24 |
| mattoliver | this causes ansble to die with a keyerror. Because they're code is brittle. | 21:25 |
| mattoliver | the patch, as timburke said basically just returns the expected not configured 404 that s3 returns | 21:25 |
| mattoliver | I need to retest the latest version with ansible (will do that today). But boto should now see the specific s3 error and hopefully now work | 21:26 |
| mattoliver | If it doesn't the ansible is simply broken because now we're behaving like s3 :P | 21:27 |
| timburke | sounds good -- i expect we'll probably get that landed this week | 21:27 |
| mattoliver | nice, thanks :) | 21:28 |
| mattoliver | I'm not the most familar with s3api side of swift, so been a good refresher :) | 21:28 |
| acoles | nice one mattoliver | 21:28 |
| mattoliver | thanks for your patience and support timburke | 21:29 |
| timburke | 👍 | 21:29 |
| timburke | all right, i think that's all i've got | 21:29 |
| timburke | #topic open discussion | 21:29 |
| timburke | anything else we should talk about this week? | 21:29 |
| clarkb | yes! did you see the PR I posted a few days ago? | 21:30 |
| clarkb | I don't know enough about erasure coding but want to make sure someone sees it since there is potential for data loss maybe? | 21:31 |
| clarkb | https://github.com/openstack/liberasurecode/pull/5 | 21:31 |
| timburke | yes! thanks clarkb! yeah, that looks like a solid catch, and sadly, not the first time that someone's tried to submit a PR about it... | 21:31 |
| clarkb | side note, you can subscribe to the github projects to get notifications by email for stuff like this. This is how I end up finding them | 21:32 |
| timburke | i've been trying to dust off my C to write some tests to go with it | 21:32 |
| clarkb | cool, i just wanted to make sure that didn't fall through the cracks | 21:32 |
| mattoliver | thanks clarkb and great idea, I'll make sure I'm subscribed | 21:32 |
| mattoliver | Like I mentioned earlier, I was on vacation for the last 1.5 weeks. So I dont' have much to say, still haven't had a chance to catch up. Will try and get back into reviews soon. I also want to catch up with the proxy side of the GET namespaces API.. But nothing to discuss here until I do (maybe next week) | 21:33 |
| timburke | 👍 subscribed | 21:33 |
| timburke | all right, i think i'll call it then | 21:36 |
| timburke | thank you all for coming, and thank you for working on swift! | 21:37 |
| timburke | #endmeeting | 21:37 |
| opendevmeet | Meeting ended Wed Oct 4 21:37:06 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2023/swift.2023-10-04-21.04.html | 21:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2023/swift.2023-10-04-21.04.txt | 21:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2023/swift.2023-10-04-21.04.log.html | 21:37 |
| opendevreview | Tim Burke proposed openstack/liberasurecode master: Fix bounds check in get_fragment_partition https://review.opendev.org/c/openstack/liberasurecode/+/897367 | 22:08 |
| opendevreview | Tim Burke proposed openstack/liberasurecode master: Add missing dependency to README https://review.opendev.org/c/openstack/liberasurecode/+/897368 | 22:13 |
| opendevreview | Tim Burke proposed openstack/swift master: Monkey-patch time https://review.opendev.org/c/openstack/swift/+/880766 | 22:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!