)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c8bda94b119b9a6c4d669968f499a00e6f96efe4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ee13c8bf_314d79cf","updated":"2024-05-08 12:55:51.000000000","message":"Change LGTM.  Could probably use a test to prevent a regression in case someone refactors, but I won\u0027t hold this up.","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3e4c16a782f5ae9d56763d4350260756398f52a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"eb03abe3_6e201d3b","updated":"2024-05-08 14:28:43.000000000","message":"Hi Dan,\n\nAdded inline reply for your comment.\nThank you!","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"064557c010447b7da6af8beea9093b6247ee6edf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"32a60674_7a35dee5","updated":"2024-05-08 08:58:10.000000000","message":"LGTM Thanks !","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e10e8b307fef8e172bb7b71b5731c8098c8b5045","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d22acf4a_68516c92","updated":"2024-05-08 13:02:15.000000000","message":"Ninja-approving a small change.","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"e7eaea67b8ad5db137ee8d625f3bd5007856f6c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"55e8fe3a_a3877f5d","updated":"2024-05-08 08:22:08.000000000","message":"Thanks for the fix!","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"909b4a7a720b72f5c9005d4b2ab4811bd0972110","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"008e8d7c_948513e0","updated":"2024-05-08 05:33:16.000000000","message":"recheck tempest failure due to image service was unavailable","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a330a786d63d0e7bb109f8c402e3f6707c6d95fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"555fd379_54499f80","updated":"2024-05-08 15:33:04.000000000","message":"recheck tempest-integrated-storage - oom killed mysql leading to a bunch of identity errors (21 failures)","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"}],"glance/cmd/cache_prefetcher.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"e281fa06c4113f4fc2b3fb8ac89202d7513f5339","unresolved":true,"context_lines":[{"line_number":50,"context_line":"        config.parse_cache_args()"},{"line_number":51,"context_line":"        logging.setup(CONF, \u0027glance\u0027)"},{"line_number":52,"context_line":"        CONF.import_opt(\u0027enabled_backends\u0027, \u0027glance.common.wsgi\u0027)"},{"line_number":53,"context_line":"        glance.async_.set_threadpool_model(\u0027eventlet\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        if CONF.enabled_backends:"},{"line_number":56,"context_line":"            glance_store.register_store_opts(CONF)"}],"source_content_type":"text/x-python","patch_set":1,"id":"91dd5ccd_b41cc126","line":53,"updated":"2024-05-08 13:44:15.000000000","message":"Does this need to be eventlet? Just given the theme of things going on lately trying to not introduce any new eventlet dependencies, it might be better if this was native. Does it work that way?\nNot critical to hold up the fix though of course.","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3e4c16a782f5ae9d56763d4350260756398f52a7","unresolved":true,"context_lines":[{"line_number":50,"context_line":"        config.parse_cache_args()"},{"line_number":51,"context_line":"        logging.setup(CONF, \u0027glance\u0027)"},{"line_number":52,"context_line":"        CONF.import_opt(\u0027enabled_backends\u0027, \u0027glance.common.wsgi\u0027)"},{"line_number":53,"context_line":"        glance.async_.set_threadpool_model(\u0027eventlet\u0027)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        if CONF.enabled_backends:"},{"line_number":56,"context_line":"            glance_store.register_store_opts(CONF)"}],"source_content_type":"text/x-python","patch_set":1,"id":"53c7baed_112e01ee","line":53,"in_reply_to":"91dd5ccd_b41cc126","updated":"2024-05-08 14:28:43.000000000","message":"Nope, I didn\u0027t checked that. Chose to use eventlet because earlier prefetcher was using eventlet GreenPool only.","commit_id":"32ee57ccdeea90a86bce1e5e33d10fab115ed2d3"}]}