opendevreview | Merged openstack/swift master: reconciler: concurreny follow-up https://review.opendev.org/c/openstack/swift/+/800842 | 00:15 |
---|---|---|
opendevreview | Merged openstack/swift master: sharder: add more validation checks on config https://review.opendev.org/c/openstack/swift/+/797961 | 01:29 |
timss | timburke: New deployment, but a couple of years old HW: 8 object nodes, 96 GB RAM each, 64 drives (HBA), not EC. Been getting the error on multiple devices without any clear pattern that I've seen so far. Will look into SMR, but don't think they are. Increasing the node_timeout to see if that resolves it could be an idea, if nothing else to see if it's able to complete a request even if | 10:15 |
timss | very slowly. There's almost no data on the cluster and no users, so very low load. | 10:15 |
*** mabrams1 is now known as mabrams | 11:09 | |
opendevreview | Takashi Kajinami proposed openstack/swift master: Switch get(full)argspec function according to python version https://review.opendev.org/c/openstack/swift/+/800924 | 11:45 |
godog | hello folks, I've just updated one proxy host to 2.26 in a 2.19.1 cluster and getting this error from s3api (the clients haven't changed), have you seen this before? I think I may be holding it wrong https://paste.debian.net/hidden/26a93a11/ | 12:22 |
godog | (running Debian) | 12:24 |
godog | I did inspect the body and for some reason it is \n-delimited, so it seems for some reason the listing isn't requested (or less likely sent back) as json is my hunch, not sure why though | 12:28 |
godog | yeah it looks like in my case BucketController._parse_request_options doesn't return format: json in the query for BucketController.GET to use, I'll dig a bit further and report | 13:07 |
godog | ok I can't find anything obvious to my untrained eye, happy to be directed further and debug why I'm running into this bug in a 2.19.1 / 2.26 mixed cluster | 13:53 |
opendevreview | Takashi Kajinami proposed openstack/swift master: Switch get(full)argspec function according to python version https://review.opendev.org/c/openstack/swift/+/800924 | 14:51 |
opendevreview | Clay Gerrard proposed openstack/swift master: container-reconciler: support multiple processes https://review.opendev.org/c/openstack/swift/+/800850 | 16:01 |
opendevreview | Alistair Coles proposed openstack/swift master: WIP diskfile: don't remove recently written non-durables https://review.opendev.org/c/openstack/swift/+/800974 | 17:10 |
opendevreview | Merged openstack/swift master: Switch get(full)argspec function according to python version https://review.opendev.org/c/openstack/swift/+/800924 | 18:22 |
timburke | godog, what does your proxy server pipeline look like? i know we've seen issues before where the listing_formats middleware isn't placed where we'd like | 18:23 |
timburke | maybe also go checking your proxy-server logs for a line starting "Pipeline was modified. ..." to see how auto-insertion shook out | 18:24 |
timburke | i should dust off https://review.opendev.org/c/openstack/swift/+/635040 ... see how if i can make a reasonable stand-alone validator | 18:26 |
opendevreview | Alistair Coles proposed openstack/swift master: WIP diskfile: don't remove recently written non-durables https://review.opendev.org/c/openstack/swift/+/800974 | 18:31 |
opendevreview | Merged openstack/swift master: Add some more arm64 non-voting tests https://review.opendev.org/c/openstack/swift/+/793280 | 19:27 |
clayg | I think acoles is right we should write an upstream bug for p 800974 - but i'm sure y'all will get that all squared away by the time I'm back from vacation | 21:10 |
clayg | 〰️ | 21:10 |
clayg | 👋 | 21:10 |
timburke | man, https://github.com/openstack/swift/blob/master/swift/obj/diskfile.py#L2532-L2533 feels weird -- like, why can't we just `return self.open()`? 🤔 | 22:41 |
opendevreview | Tim Burke proposed openstack/swift master: trivial: Replace readfp with read_file https://review.opendev.org/c/openstack/swift/+/801004 | 23:07 |
opendevreview | Tim Burke proposed openstack/swift master: trivial: Replace assertRaisesRegexp with assertRaisesRegex https://review.opendev.org/c/openstack/swift/+/801005 | 23:07 |
opendevreview | Tim Burke proposed openstack/swift master: trivial: Replace assertRegexpMatches with assertRegex https://review.opendev.org/c/openstack/swift/+/801006 | 23:07 |
opendevreview | Tim Burke proposed openstack/swift master: trivial: Clean up a DeprecationWarning https://review.opendev.org/c/openstack/swift/+/801007 | 23:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!