| acoles | timburke__: apologies, I won't be able to make today's meeting | 19:30 |
|---|---|---|
| opendevreview | Merged openstack/swift master: Add missing services to sample rsyslog.conf https://review.opendev.org/c/openstack/swift/+/841673 | 20:54 |
| kota | meeting? | 21:03 |
| mattoliver | I'll poke tim | 21:04 |
| kota | thx mattoliver | 21:05 |
| zaitcev | I didn't forget this time, meaning it's probably going to be jinxed with Tim's sick child or something. | 21:05 |
| kota | :/ | 21:06 |
| mattoliver | lol | 21:06 |
| mattoliver | Still no response from him | 21:08 |
| timburke__ | sorry, got distracted by an issue at home | 21:11 |
| timburke__ | #startmeeting swift | 21:11 |
| opendevmeet | timburke__: Error: Can't start another meeting, one is in progress. Use #endmeeting first. | 21:11 |
| mattoliver | that's a weird error | 21:11 |
| timburke__ | wha... | 21:11 |
| timburke__ | #endmeeting | 21:11 |
| opendevmeet | Meeting ended Wed May 18 21:11:45 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:11 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2022/swift.2022-05-11-21.00.html | 21:11 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2022/swift.2022-05-11-21.00.txt | 21:11 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2022/swift.2022-05-11-21.00.log.html | 21:11 |
| timburke__ | i guess i never ended it last week? oops... | 21:12 |
| timburke__ | #startmeeting swift | 21:12 |
| opendevmeet | Meeting started Wed May 18 21:12:05 2022 UTC and is due to finish in 60 minutes. The chair is timburke__. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:12 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:12 |
| opendevmeet | The meeting name has been set to 'swift' | 21:12 |
| mattoliver | how long has that been going for | 21:12 |
| mattoliver | lol | 21:12 |
| timburke__ | who's here for *this week's* swift meeting? | 21:12 |
| mattoliver | o/ | 21:12 |
| kota | o/ | 21:12 |
| kota | such a long meeting :D | 21:13 |
| timburke__ | as usual, the agenda's at | 21:13 |
| timburke__ | #link https://wiki.openstack.org/wiki/Meetings/Swift | 21:13 |
| mattoliver | I put in the first topic | 21:14 |
| timburke__ | and it looks like mattoliver added something to it :D | 21:14 |
| timburke__ | #topic tempurl and sha1 signatures | 21:14 |
| timburke__ | mattoliver, take it away :-) | 21:14 |
| mattoliver | Thanks | 21:14 |
| mattoliver | Got pinged in the channel during the week, there are still people interested in getting this landed. | 21:15 |
| mattoliver | Which makes sense :P | 21:15 |
| mattoliver | So just wanted to bring it to peoples attention. To get final reviews in and let's land at least the tempurl patch | 21:16 |
| timburke__ | i'll take a look for sure -- seems like there have been some changes since the last time i pushed anything up | 21:17 |
| mattoliver | Thanks, yeah, I think I pushed something up at some stage too. Can't remember :P I'm in the same boat. So I'll swing back to it myself. | 21:17 |
| mattoliver | People are waiting on it, so maybe we need to make it a priority. | 21:18 |
| mattoliver | That's all I have | 21:18 |
| timburke__ | then we'll check in on other work in progress :-) | 21:19 |
| timburke__ | #topic ring v2 | 21:19 |
| timburke__ | thanks for the review and follow-up mattoliver! i haven't had a chance to loop back on it yet, i'm afraid | 21:19 |
| mattoliver | that's ok. I went through the initial patch at attempted to address all the missing todos in the follow up | 21:20 |
| mattoliver | mostly a bunch of testing. It isn't extensive but wanted to add covereage | 21:20 |
| timburke__ | i'll try to get a fresh patch up by next week so we can land it soon-ish | 21:21 |
| timburke__ | #topic memcache errors | 21:22 |
| timburke__ | the first patch in the chain has already landed! https://review.opendev.org/c/openstack/swift/+/839448 | 21:22 |
| timburke__ | and i just approved the follow-up to make tempauth not hand out tokens that couldn't be set in memcache https://review.opendev.org/c/openstack/swift/+/840250 | 21:23 |
| timburke__ | there was one other follow-up to get the command name in more log messages, but the plumbing is a little awkward https://review.opendev.org/c/openstack/swift/+/839822 | 21:24 |
| timburke__ | we can live without it, but if anyone has some good ideas for a better way to carry some context around, feel free to push up a patch! | 21:25 |
| mattoliver | kk | 21:26 |
| timburke__ | #topic getting rid of logging locks | 21:26 |
| zaitcev | Wait, I could swear that I saw a memcache error propagation to client in proxy | 21:27 |
| zaitcev | Neither of the two linked above are it. | 21:27 |
| zaitcev | Must be dementia | 21:27 |
| zaitcev | Neither of the three | 21:28 |
| zaitcev | Oh well, logging locks. | 21:28 |
| timburke__ | zaitcev, what, like an uncaught error that made its way back out to eventlet, which formatted it for clients or something? i thought that was what catch_errors was for... | 21:29 |
| mattoliver | or another patch | 21:29 |
| zaitcev | timburke__, sorry. I must be confused about it. I thought I wanted to review that one and even saved it somewhere. But oh well. | 21:30 |
| zaitcev | This is what Joe Biden feels every day, I understand now. | 21:30 |
| timburke__ | funny, i've got this dim recollection as well... like you'd added yourself to the CC list, at least *shrug* | 21:32 |
| timburke__ | anyway, we've got https://review.opendev.org/c/openstack/swift/+/840232 in prod now, but still haven't turned it on. hopefully i'll actually have some data from prod next week? | 21:33 |
| timburke__ | i also think maybe mattoliver was going to try it out in our demo/staging environment | 21:34 |
| mattoliver | yeah, I plan too.. but keep getting distracted :( | 21:34 |
| mattoliver | Will try and get something done by next meeting (famous last words) :P | 21:34 |
| timburke__ | no worries! you're getting a lot done :-) | 21:35 |
| timburke__ | #topic backend ratelimiting | 21:35 |
| timburke__ | we've got the first couple patches in the chain landed! | 21:35 |
| timburke__ | there's the refactor https://review.opendev.org/c/openstack/swift/+/834960 | 21:36 |
| timburke__ | and a new option for some burstiness https://review.opendev.org/c/openstack/swift/+/835122 | 21:36 |
| timburke__ | next patch in the chain adds the new middleware itself https://review.opendev.org/c/openstack/swift/+/836046 | 21:37 |
| mattoliver | That last one looks good. I pushed back to make sure it was added to the SAIO pipelines so we'd start testing it more. Maybe that's more of a NIT? | 21:38 |
| mattoliver | Just don't want to forget about it and not realise until later | 21:39 |
| timburke__ | there's the risk it could cause some flakey probe tests -- but then, we usually have issues with things going too *slow* in probe tests (and popping timeouts), not too *fast* | 21:40 |
| timburke__ | and we could also just make sure it's configured to some reasonably high default | 21:41 |
| mattoliver | it should be defaulted to off. but alleast it'll be there to turn on | 21:41 |
| mattoliver | but true | 21:41 |
| timburke__ | all right, that's all i've got | 21:43 |
| timburke__ | #topic open discussion | 21:43 |
| timburke__ | anything else we should bring up this week? | 21:43 |
| timburke__ | i bugged temoto for an eventlet release recently, and now upper-constraints has been updated to use it. this should mean the timeouts in unit tests should go away | 21:45 |
| zaitcev | Cool | 21:45 |
| kota | nice | 21:46 |
| zaitcev | Digging through my dashboard just now during the meeting, I found that the conflict between object watcher and container sharding was not resolved. https://review.opendev.org/c/openstack/swift/+/787656 | 21:46 |
| zaitcev | September 2021 | 21:47 |
| zaitcev | Oh, december. Not as long as I thought... I'll try to find cycles to complete it, so it doesn't hang around. | 21:47 |
| timburke__ | that'll be great, thanks! sorry, i proposed the thing but then haven't really followed up on it | 21:48 |
| timburke__ | all right, i think i'll call it | 21:50 |
| timburke__ | thank you all for coming, and thank you for working on swift! | 21:50 |
| timburke__ | #endmeeting | 21:50 |
| opendevmeet | Meeting ended Wed May 18 21:50:52 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:50 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2022/swift.2022-05-18-21.12.html | 21:50 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2022/swift.2022-05-18-21.12.txt | 21:50 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2022/swift.2022-05-18-21.12.log.html | 21:50 |
| timburke__ | good! definitely ended this time ;-) | 21:51 |
| kota | :D | 21:52 |
| opendevreview | Merged openstack/swift master: Don't give clients made up tokens https://review.opendev.org/c/openstack/swift/+/840250 | 22:33 |
| opendevreview | Tim Burke proposed openstack/swift master: ceph-tests: Remove known-failure https://review.opendev.org/c/openstack/swift/+/842447 | 23:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!