)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a6cabfa21133fa44880552954b8807613c70024e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8395f5ba_e52b271b","updated":"2026-08-12 12:19:23.000000000","message":"Typo noted inline.","commit_id":"065a1533b25e93e616458ad300f1de5e46c00381"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"4c09b51d83336a07f0dcfafbff3eca993135bf1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1f994a0e_503f87ab","updated":"2026-08-13 14:29:31.000000000","message":"recheck","commit_id":"d5ec9abe9202251c13757e312f54f8c0d5255a28"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"215f33b54a0031f048992b6e95b6b1cdcaa7bd7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1779ace4_3fe6f9ed","updated":"2026-08-19 11:57:01.000000000","message":"LGTM","commit_id":"c4d15ffeb7bf88b63fc269f69f2808b22657e2b2"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"0b503b3dded3c7b2bddb644a5bb666a7d4570029","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"484acf5d_e8c46a56","updated":"2026-08-14 13:24:06.000000000","message":"this seems fine to me.","commit_id":"c4d15ffeb7bf88b63fc269f69f2808b22657e2b2"}],"cinder/cmd/backup.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a6cabfa21133fa44880552954b8807613c70024e","unresolved":true,"context_lines":[{"line_number":52,"context_line":"backup_cmd_opts \u003d ["},{"line_number":53,"context_line":"    cfg.IntOpt(\u0027backup_workers\u0027,"},{"line_number":54,"context_line":"               default\u003d1, min\u003d1, max\u003dprocessutils.get_worker_count(),"},{"line_number":55,"context_line":"               max_default\u003d\u0027\u003cWORKER_COUNT\u003e\u0027,"},{"line_number":56,"context_line":"               sample_default\u003d8,"},{"line_number":57,"context_line":"               help\u003d\u0027Number of backup processes to launch. \u0027"},{"line_number":58,"context_line":"               \u0027Improves performance with concurrent backups.\u0027),"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a9c733f_374fafc5","line":55,"range":{"start_line":55,"start_character":15,"end_line":55,"end_character":26},"updated":"2026-08-12 12:19:23.000000000","message":"should be \u0027sample_max\u0027\nhttps://docs.openstack.org/releasenotes/oslo.config/unreleased.html#relnotes-10-7-0","commit_id":"065a1533b25e93e616458ad300f1de5e46c00381"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"fe3e7b31360ae97eefcb58125395991ebd497793","unresolved":false,"context_lines":[{"line_number":52,"context_line":"backup_cmd_opts \u003d ["},{"line_number":53,"context_line":"    cfg.IntOpt(\u0027backup_workers\u0027,"},{"line_number":54,"context_line":"               default\u003d1, min\u003d1, max\u003dprocessutils.get_worker_count(),"},{"line_number":55,"context_line":"               max_default\u003d\u0027\u003cWORKER_COUNT\u003e\u0027,"},{"line_number":56,"context_line":"               sample_default\u003d8,"},{"line_number":57,"context_line":"               help\u003d\u0027Number of backup processes to launch. \u0027"},{"line_number":58,"context_line":"               \u0027Improves performance with concurrent backups.\u0027),"}],"source_content_type":"text/x-python","patch_set":3,"id":"4ccf628d_1a431386","line":55,"range":{"start_line":55,"start_character":15,"end_line":55,"end_character":26},"in_reply_to":"1a9c733f_374fafc5","updated":"2026-08-12 12:36:10.000000000","message":"Ah, thanks ! :)","commit_id":"065a1533b25e93e616458ad300f1de5e46c00381"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f1525c943f6c32046c9be08dcb9b0ef7323413c8","unresolved":true,"context_lines":[{"line_number":52,"context_line":"backup_cmd_opts \u003d ["},{"line_number":53,"context_line":"    cfg.IntOpt(\u0027backup_workers\u0027,"},{"line_number":54,"context_line":"               default\u003d1, min\u003d1, max\u003dprocessutils.get_worker_count(),"},{"line_number":55,"context_line":"               sample_max\u003d\u0027\u003cWORKER_COUNT\u003e\u0027,"},{"line_number":56,"context_line":"               sample_default\u003d8,"},{"line_number":57,"context_line":"               help\u003d\u0027Number of backup processes to launch. \u0027"},{"line_number":58,"context_line":"               \u0027Improves performance with concurrent backups.\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"c0f4c836_2b79f441","line":55,"updated":"2026-08-12 13:00:24.000000000","message":"Is \"WORKER_COUNT\" a meaningful description for admins? I think this is just determined by CPU count on the machine, maybe saying that explicitly would be more clear?","commit_id":"d5ec9abe9202251c13757e312f54f8c0d5255a28"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"0ee6acc69e316ec0aaa713bab8693a0d26620d8c","unresolved":false,"context_lines":[{"line_number":52,"context_line":"backup_cmd_opts \u003d ["},{"line_number":53,"context_line":"    cfg.IntOpt(\u0027backup_workers\u0027,"},{"line_number":54,"context_line":"               default\u003d1, min\u003d1, max\u003dprocessutils.get_worker_count(),"},{"line_number":55,"context_line":"               sample_max\u003d\u0027\u003cWORKER_COUNT\u003e\u0027,"},{"line_number":56,"context_line":"               sample_default\u003d8,"},{"line_number":57,"context_line":"               help\u003d\u0027Number of backup processes to launch. \u0027"},{"line_number":58,"context_line":"               \u0027Improves performance with concurrent backups.\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"6155a9a6_4ad954d8","line":55,"in_reply_to":"c0f4c836_2b79f441","updated":"2026-08-13 14:30:52.000000000","message":"Wrote CPU_COUNT instead.","commit_id":"d5ec9abe9202251c13757e312f54f8c0d5255a28"}]}