)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"58180f8d5b0f6060b4c3250db3c4fb4ba75f8f31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1a4b83e7_2144db40","updated":"2022-10-10 07:13:56.000000000","message":"Hi.\nThis patch worked for me, however, I had troubles to backport it for anything lower than ussuri (that is: train, stein and rocky, which I still maintain). Can someone come with such a backport of this patch?","commit_id":"30c9e1a90a4196422f00eb0e0678a71c4f29920a"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"eee6cb2caacec81d4264d63593f18756e0bf202b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"62451027_085dbc4d","updated":"2022-11-22 15:23:41.000000000","message":"I would expect that the cache can be cleared at any time, like with a Memcached restart, a server being down or due to eviction policy. It looks to me that we would still run into this issue, even after applying this fix. It will possibly manifest itself less frequently, but that could make it harder to pin down and correct.","commit_id":"30c9e1a90a4196422f00eb0e0678a71c4f29920a"},{"author":{"_account_id":7414,"name":"David Wilde","email":"dwilde@redhat.com","username":"d34dh0r53"},"change_message_id":"5f00125c0fc1e943ebfdc6737cc3e8761434cdf5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"496ecc43_504f5370","updated":"2022-12-06 21:28:55.000000000","message":"I\u0027ve added https://review.opendev.org/c/openstack/keystonemiddleware/+/866805 to unblock the pep8 gate.","commit_id":"30c9e1a90a4196422f00eb0e0678a71c4f29920a"},{"author":{"_account_id":7414,"name":"David Wilde","email":"dwilde@redhat.com","username":"d34dh0r53"},"change_message_id":"e1ff45e6e43b6bb09b1233e538196cb4b1e105cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f2beb12_0f634e18","updated":"2022-12-02 15:51:39.000000000","message":"LGTM","commit_id":"30c9e1a90a4196422f00eb0e0678a71c4f29920a"},{"author":{"_account_id":34275,"name":"Jorge Merlino","email":"jorge.merlino@canonical.com","username":"drencrom"},"change_message_id":"735751890985d0fa89bf6e5cd90d880827dfd38a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ce270163_9f3dabe7","in_reply_to":"62451027_085dbc4d","updated":"2022-11-22 15:48:08.000000000","message":"Thank you for the review. The problem is not that the cache is cleared. The problem is that that one specific token is marked as invalid in the cache and will never work again (until the cache is cleared)","commit_id":"30c9e1a90a4196422f00eb0e0678a71c4f29920a"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"0dccaa8644456aca3e10dd7a1a823af7fa5ba997","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"79529e92_994da87c","in_reply_to":"ce270163_9f3dabe7","updated":"2022-11-22 15:49:50.000000000","message":"Yes, sorry, on a second look, it\u0027s the _CACHE_INVALID_INDICATOR that is cached. I wonder what\u0027s the performance impact of not doing that, but it looks good to me.","commit_id":"30c9e1a90a4196422f00eb0e0678a71c4f29920a"}]}
