| opendevreview | Merged openstack/swift master: Return 503 for container listings when shards are deleted https://review.opendev.org/c/openstack/swift/+/793492 | 08:22 |
|---|---|---|
| opendevreview | Alistair Coles proposed openstack/swift master: reconstructor: Delay purging reverted non-durable datafiles https://review.opendev.org/c/openstack/swift/+/797515 | 18:11 |
| opendevreview | Alistair Coles proposed openstack/swift master: Capture logs when running custom daemons in probe tests https://review.opendev.org/c/openstack/swift/+/797735 | 18:11 |
| opendevreview | Tim Burke proposed openstack/swift master: Add some more arm64 non-voting tests https://review.opendev.org/c/openstack/swift/+/793280 | 20:37 |
| kota | good morning | 20:54 |
| kota | sorryk i'm absent for today because of some urgent staffs | 20:55 |
| seongsoocho | good morning ! | 20:55 |
| kota | seongsoocho: o/ | 20:55 |
| timburke | no worries kota | 20:58 |
| timburke | almost meeting time! | 20:59 |
| timburke | #startmeeting swift | 21:00 |
| opendevmeet | Meeting started Wed Jun 23 21:00:45 2021 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 |
| seongsoocho | o/ | 21:01 |
| acoles | o/ | 21:01 |
| clayg | o/ | 21:01 |
| mattoliver | o/ | 21:02 |
| timburke | as usualy, the agenda's at https://wiki.openstack.org/wiki/Meetings/Swift -- though it's basically just checking in on some of our longer-running threads | 21:02 |
| timburke | #topic sharding and shrinking | 21:03 |
| timburke | i meant to add the patches mentioned last week to the priority reviews page but i forgot :-( | 21:04 |
| timburke | https://review.opendev.org/c/openstack/swift/+/794582 - sharder: avoid small tail shards | 21:05 |
| timburke | we (NVIDIA) should be running with that soon -- but i'm still a little skeptical of the default for the new tunable | 21:06 |
| clayg | yeah, 100% agree - I don't think keeping "existing bad behavior" is ideal | 21:06 |
| clayg | @timburke what default do you think makes sense? something to do with the configured shrink size or something? | 21:07 |
| acoles | I'm unsure about linking it to shrinking - but happy to have a lrger absolute number as default | 21:07 |
| acoles | larger* | 21:07 |
| clayg | i forget what all of those vars are named - @mattoliver @acoles do you think a "helpful" default would be better? or because it's all decoupled with all the other values we have to do some validation? | 21:08 |
| acoles | even if it is same number as shrink_threshold, I just don't think the two should be couple | 21:08 |
| timburke | yeah, i was debating about that -- if we're pushing ops toward absolute values for these config opts anyway, 100k or something seems reasonable | 21:08 |
| acoles | +1 | 21:08 |
| clayg | @acoles are there some combinations that wouldn't make sense tho? like it can't be bigger than the rows_per_shard (but I think that's already enforced) | 21:08 |
| acoles | IIRC I enforced that yes | 21:09 |
| acoles | oh, BTW, @timburke yes that piece of code did have an eye towards adding more validations ;) | 21:09 |
| timburke | makes sense. do we want to respin it with the higher default before merging? | 21:10 |
| acoles | yep I'll do that tomorrow | 21:12 |
| timburke | 👍 | 21:12 |
| acoles | better to have the default start life as we intend | 21:12 |
| mattoliver | +1 | 21:12 |
| timburke | #topic dark data watcher | 21:13 |
| acoles | hmmm, anyone running with rows_per_shard < 100k will get burnt | 21:13 |
| timburke | i've still not gotten around to reviewing the patches, unfortunately | 21:13 |
| acoles | I think I need to catch up with a review too - sorry zaitcev , I've been on vacation | 21:14 |
| timburke | acoles, i feel like rows_per_shard = 100k is already pretty low -- though i guess we could make the default min(rows_per_shard, 100k) | 21:14 |
| acoles | ok | 21:14 |
| timburke | all right. acoles and i will try to get these suckers reviewed | 21:15 |
| timburke | #topic open discussion | 21:15 |
| timburke | (there wasn't really much to talk about with the relinker, i just forgot to take it off the agenda) | 21:16 |
| timburke | what else should we bring up this week? | 21:16 |
| clayg | the relinker and dark data were on the agenda? | 21:16 |
| timburke | i want to get a release together -- it's been a bit | 21:19 |
| timburke | and i've also been digging through our prod logs looking for tracebacks. turns out, we generate a lot of tracebacks | 21:19 |
| timburke | and i re-spun the last ARM job to lean in to the py38-ness | 21:21 |
| mattoliver | Yeah nice work on that. I've seen some patches in gerrit that address some of those I suspect :) (re tracebacks) | 21:22 |
| timburke | that's the idea :-) | 21:23 |
| timburke | all right, if nobody has anything else, we'll make it a short meeting | 21:23 |
| acoles | timburke: the arm jobs vote separately right? | 21:24 |
| timburke | yup; they'll drop a separate comment, and none of them are currently voting | 21:24 |
| acoles | does recheck trigger both? | 21:24 |
| acoles | ie arm and the fuller set | 21:25 |
| timburke | i figure, assuming everything is pretty stable, we can look at making them voting toward the end of the cycle | 21:25 |
| timburke | yup | 21:25 |
| acoles | ok | 21:25 |
| timburke | all right. thank you all for coming, and thank you for working on swift! | 21:26 |
| timburke | #endmeeting | 21:26 |
| opendevmeet | Meeting ended Wed Jun 23 21:26:06 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:26 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/swift/2021/swift.2021-06-23-21.00.html | 21:26 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/swift/2021/swift.2021-06-23-21.00.txt | 21:26 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/swift/2021/swift.2021-06-23-21.00.log.html | 21:26 |
| opendevreview | Tim Burke proposed openstack/liberasurecode master: Add FIPS jobs https://review.opendev.org/c/openstack/liberasurecode/+/797789 | 22:52 |
| opendevreview | Tim Burke proposed openstack/swift master: Be willing to send hashes as JSON https://review.opendev.org/c/openstack/swift/+/758638 | 23:27 |
| opendevreview | Tim Burke proposed openstack/swift master: Have expirer use IC's delete_object https://review.opendev.org/c/openstack/swift/+/762704 | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!