)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"056df20b77a01fae57c14fea0c9528653dc9ea0f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Revert \"Add support to configure token caching in keystone\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Changing cache/enabled\u003dFalse by default has dropped performance."},{"line_number":10,"context_line":"keystone local cache also got disabled with this."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This reverts commit 469d432195d1f5b5e15ce72ce1624d4ed4447e4e."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_a35ab8ee","line":10,"updated":"2019-10-15 19:20:25.000000000","message":"So the default behavior is to stand up caching by default, right?","commit_id":"63dd90aacc84545d9f171ee163b7e97452d0a537"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a2526b10848334ef74cfabf0155aa6697f0fb82c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Revert \"Add support to configure token caching in keystone\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Changing cache/enabled\u003dFalse by default has dropped performance."},{"line_number":10,"context_line":"keystone local cache also got disabled with this."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This reverts commit 469d432195d1f5b5e15ce72ce1624d4ed4447e4e."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_d50682ce","line":10,"in_reply_to":"3fa7e38b_a35ab8ee","updated":"2019-10-16 04:37:44.000000000","message":"@Lance yes looks like yes some local cache is enabled by default as per the test results we have seen till now. https://github.com/openstack/keystone/blob/master/keystone/conf/__init__.py#L188-L197 is setting enabled\u003dtrue overriding oslo.cache default.\n\nBut someone from keystone can confirm it, and also suggest the recommended configuration to move forward after the revert(revert was needed at this point to unblock promotion and remove the performance regression caused by the set of patches)","commit_id":"63dd90aacc84545d9f171ee163b7e97452d0a537"}]}
