)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"947aecfb842e22cfc4ac20dbfeecd93a3e900eaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3de40d04_a0cd08d4","updated":"2022-02-14 14:07:42.000000000","message":"@Chhavi: response/question for you inline.","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"0b2229ada1837c4c4fc71418c63ce56370da3811","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"95702803_1633a62a","updated":"2022-02-02 14:03:13.000000000","message":"No ideas with the document, but if we could get any remaining users of this to use external rate limiting, getting rid of it would be a good idea (as Nova have done)","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0dd8c49a91f9dc82d77368c4d70347a6f58eb726","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f120d03f_6685cdc6","updated":"2022-02-02 13:33:36.000000000","message":"Resolved the merge conflict.","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e3e0e7fdb9cc033d561f1fefe0a3530ac22c4f31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d80f758a_e940e00b","updated":"2022-02-02 14:34:33.000000000","message":"Thanks for getting this documented - it was long overdue but I never had the time to figure it out.","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"edc06c32a3bf60868a1950009143ad32bb8a58f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2056277b_23d64218","updated":"2022-02-17 22:52:07.000000000","message":"Thanks, Stephen.","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"}],"doc/source/admin/api-configuration.rst":[{"author":{"_account_id":14806,"name":"Chhavi Agarwal","email":"chhagarw@in.ibm.com","username":"chhagarw"},"change_message_id":"75f4ca0a5e0ebd00087a1630b979d7afa6ffc0a0","unresolved":true,"context_lines":[{"line_number":12,"context_line":".. warning::"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"   This is legacy functionality that is poorly tested and may be removed in the"},{"line_number":15,"context_line":"   future. You may wish to enforce rate limiting through a proxy server"},{"line_number":16,"context_line":"   instead."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Cinder supports admin-configured API limits. These are disabled by default but"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3daa6878_ad45e60f","line":15,"updated":"2022-02-03 07:29:06.000000000","message":"Does it means it\u0027s not recommended to use this Ratelimiter, are there any open issues which we should be aware of.","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"61232cc64edd7cb6388d274852bebf7dfa81201c","unresolved":false,"context_lines":[{"line_number":12,"context_line":".. warning::"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"   This is legacy functionality that is poorly tested and may be removed in the"},{"line_number":15,"context_line":"   future. You may wish to enforce rate limiting through a proxy server"},{"line_number":16,"context_line":"   instead."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Cinder supports admin-configured API limits. These are disabled by default but"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b911eac2_690f740d","line":15,"in_reply_to":"3daa6878_ad45e60f","updated":"2022-02-03 12:23:08.000000000","message":"That\u0027s a question for the cinder maintainers. The lack of documentation and testing suggested to me that this is not in widespread use and should probably be avoided, but then again it could Just Work (TM). idk","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"947aecfb842e22cfc4ac20dbfeecd93a3e900eaf","unresolved":false,"context_lines":[{"line_number":12,"context_line":".. warning::"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"   This is legacy functionality that is poorly tested and may be removed in the"},{"line_number":15,"context_line":"   future. You may wish to enforce rate limiting through a proxy server"},{"line_number":16,"context_line":"   instead."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Cinder supports admin-configured API limits. These are disabled by default but"}],"source_content_type":"text/x-rst","patch_set":2,"id":"d9970bb9_f4d38ab2","line":15,"in_reply_to":"b911eac2_690f740d","updated":"2022-02-14 14:07:42.000000000","message":"@Chhavi: basically, what stephenfin said.  We don\u0027t test it, and it\u0027s legacy technology copied out of nova (which the Nova team has in the meantime completely removed from nova).  When I floated a proposal to remove it on the mailing list, one operator responded on the gerrit review that it was in-use in their deployment.  So we decided to leave it in.\n\nIn short, no issues have been reported, but then since we don\u0027t believe it\u0027s in wide use, so it may \"just work\" for the particular configurations where it\u0027s being used, and that doesn\u0027t mean someone else won\u0027t encounter issues.\n\nOur recommendation is exactly what Stephen says in the \"Warning\" box.  Is that clear enough, or do you think we need to rewrite the warning?","commit_id":"e6859ded3170580b2be692a3fbf320bb8d1eb567"}]}