Thursday, 2021-07-15

opendevreviewMerged openstack/swift master: reconciler: concurreny follow-up  https://review.opendev.org/c/openstack/swift/+/80084200:15
opendevreviewMerged openstack/swift master: sharder: add more validation checks on config  https://review.opendev.org/c/openstack/swift/+/79796101:29
timsstimburke: 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 if10:15
timssvery slowly. There's almost no data on the cluster and no users, so very low load. 10:15
*** mabrams1 is now known as mabrams11:09
opendevreviewTakashi Kajinami proposed openstack/swift master: Switch get(full)argspec function according to python version  https://review.opendev.org/c/openstack/swift/+/80092411:45
godoghello 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
godogI 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 though12:28
godogyeah 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 report13:07
godogok 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 cluster13:53
opendevreviewTakashi Kajinami proposed openstack/swift master: Switch get(full)argspec function according to python version  https://review.opendev.org/c/openstack/swift/+/80092414:51
opendevreviewClay Gerrard proposed openstack/swift master: container-reconciler: support multiple processes  https://review.opendev.org/c/openstack/swift/+/80085016:01
opendevreviewAlistair Coles proposed openstack/swift master: WIP diskfile: don't remove recently written non-durables  https://review.opendev.org/c/openstack/swift/+/80097417:10
opendevreviewMerged openstack/swift master: Switch get(full)argspec function according to python version  https://review.opendev.org/c/openstack/swift/+/80092418:22
timburkegodog, 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 like18:23
timburkemaybe also go checking your proxy-server logs for a line starting "Pipeline was modified. ..." to see how auto-insertion shook out18:24
timburkei should dust off https://review.opendev.org/c/openstack/swift/+/635040 ... see how if i can make a reasonable stand-alone validator18:26
opendevreviewAlistair Coles proposed openstack/swift master: WIP diskfile: don't remove recently written non-durables  https://review.opendev.org/c/openstack/swift/+/80097418:31
opendevreviewMerged openstack/swift master: Add some more arm64 non-voting tests  https://review.opendev.org/c/openstack/swift/+/79328019:27
claygI 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 vacation21:10
clayg〰️21:10
clayg👋21:10
timburkeman, 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
opendevreviewTim Burke proposed openstack/swift master: trivial: Replace readfp with read_file  https://review.opendev.org/c/openstack/swift/+/80100423:07
opendevreviewTim Burke proposed openstack/swift master: trivial: Replace assertRaisesRegexp with assertRaisesRegex  https://review.opendev.org/c/openstack/swift/+/80100523:07
opendevreviewTim Burke proposed openstack/swift master: trivial: Replace assertRegexpMatches with assertRegex  https://review.opendev.org/c/openstack/swift/+/80100623:07
opendevreviewTim Burke proposed openstack/swift master: trivial: Clean up a DeprecationWarning  https://review.opendev.org/c/openstack/swift/+/80100723:07

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!