opendevreview | likui proposed openstack/swift master: Replace deprecated assertNotRegexpMatches https://review.opendev.org/c/openstack/swift/+/817664 | 01:53 |
---|---|---|
opendevreview | wushiming proposed openstack/swift master: Replace deprecated assertRegexpMatches https://review.opendev.org/c/openstack/swift/+/817698 | 05:59 |
opendevreview | Matthew Oliver proposed openstack/swift master: WIP: First attempt at a ArributeRateLimit Iterator https://review.opendev.org/c/openstack/swift/+/817699 | 06:07 |
mattoliver | ^ Here is a very rough AttributeRateLimtedIterator that could be used to ratelimit object-updater by containers. It's very very rough but does have an ascii art diagram. | 06:10 |
mattoliver | Not sure it's the right solution, more of a discussion starting point. | 06:10 |
mattoliver | In theory we should be able to: RateLimitedIterator(AttributeRateLimitedIterator(.., <container_path>, per_container_ratelimit,...), ..., max_objects_per_second, ..) | 06:13 |
mattoliver | and get 2 tunables, per item and then per container. | 06:14 |
mattoliver | Haven't fully sold myself on the idea.. but wanted to come up with at least one approach to start discussions :) | 06:14 |
opendevreview | wushiming proposed openstack/swift master: Replace deprecated assertRegexpMatches https://review.opendev.org/c/openstack/swift/+/817710 | 07:43 |
zigo | Hi. Where should I open bugs against pyeclib ? | 10:10 |
zigo | timburke: It has unit test failures under Python 3.10. See https://bugs.debian.org/cgi-bin/999391 | 10:10 |
zigo | BTW, thanks a lot Tim, for your Eventlet Python 3.10 compat patches ! (I used them and uploaded it to unstable) | 12:03 |
timburke | zigo, https://bugs.launchpad.net/pyeclib -- was it something about needing to use Py_ssize_t? we probably just need a fresh release since https://review.opendev.org/c/openstack/pyeclib/+/790537 landed | 16:47 |
timburke | (the debian bugs link 404s, btw) | 16:48 |
timburke | i guess we want https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=999391 ? and yeah, i think the "Invalid arguments" errors are likely all fixed by that Py_ssize_t patch | 16:51 |
opendevreview | Alistair Coles proposed openstack/swift master: Make cmp_policy_info agree with the API behaviour https://review.opendev.org/c/openstack/swift/+/816731 | 17:10 |
opendevreview | Alistair Coles proposed openstack/swift master: Improve storage policy index reconciliation unit tests https://review.opendev.org/c/openstack/swift/+/816892 | 17:10 |
zigo | timburke: Yeah, it was about #999391 thanks for the patch, I'll apply it on top of the current package and see how it goes. | 18:08 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!