On Wed, 2021-09-08 at 12:00 -0400, Brian Rosmaita wrote:

Hello operators,

Takashi Kajinami noticed that cinder's RateLimitingMiddleware class lives among the legacy v2 classes and should be moved to live with the v3 classes, but then it occurred to him that maybe it should simply be deprecated and removed [0].

There are a few reasons to believe that nobody is actually using this functionality, for example, it's not included in the default api-paste.ini file, this middleware no longer exists in Nova, it gives you inexact behavior with multiple API nodes, etc.

We discussed this topic at today's cinder meeting and decided to ask whether anyone is using the RateLimitingMiddleware with cinder before deprecating it for removal.

If it's going to be deprecated, we'd like to do it in Xena, so please reply to this email or leave a comment on the deprecation patch [1] (or preferably both) before 23:59 UTC on Tuesday 14 September 2021.

Thank you!

[0] https://bugs.launchpad.net/cinder/+bug/1942696 [1] https://review.opendev.org/c/openstack/cinder/+/807469

Not an ansewr, but I have a relatively old patch to document this [1]. If we're not deprecating this, we should probably merge the docs patch. If we are deprecating it, the doc patch can obviously be abandoned. Cheers, Stephen [1] https://review.opendev.org/c/openstack/cinder/+/782518