)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"71af5ab4462f3f641398dc4669010cd08983a037","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"47544fb1_7ddb260b","updated":"2024-11-15 10:34:19.000000000","message":"LGTM, thanks for fixing this","commit_id":"005be05ec3a8938c687e7d119b6737819d56236a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cde133b6741d964085bfc3278d9f9f6966e83c26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f227d8e1_ad4efb58","updated":"2024-11-15 17:30:29.000000000","message":"LGTM.","commit_id":"005be05ec3a8938c687e7d119b6737819d56236a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f78e0d2d3d0f468445b7491b0aea5cfb5aa85db3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"076ad9ba_7a55aad6","updated":"2024-10-12 14:01:18.000000000","message":"LGTM. Makes sense","commit_id":"005be05ec3a8938c687e7d119b6737819d56236a"}],"cinder/tests/unit/test_cmd.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"7d08dfbf5bbe02b912a6db287c2f713ab3e55307","unresolved":false,"context_lines":[{"line_number":133,"context_line":"            raise test.testtools.TestCase.skipException("},{"line_number":134,"context_line":"                \u0027requires more than 1 cpu (to set backup_workers \u003e1)\u0027)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"        CONF.set_override(\u0027backup_workers\u0027, 2)"},{"line_number":137,"context_line":"        mock_semaphore.side_effect \u003d [mock.sentinel.semaphore1,"},{"line_number":138,"context_line":"                                      mock.sentinel.semaphore2]"},{"line_number":139,"context_line":"        cinder_backup.main()"}],"source_content_type":"text/x-python","patch_set":1,"id":"78b8d2b2_65999088","line":136,"range":{"start_line":136,"start_character":26,"end_line":136,"end_character":45},"updated":"2024-11-15 10:35:24.000000000","message":"For reviewers,\n\nthis won\u0027t work for system with 1 CPU https://opendev.org/openstack/cinder/src/branch/master/cinder/cmd/backup.py#L65","commit_id":"005be05ec3a8938c687e7d119b6737819d56236a"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cde133b6741d964085bfc3278d9f9f6966e83c26","unresolved":false,"context_lines":[{"line_number":133,"context_line":"            raise test.testtools.TestCase.skipException("},{"line_number":134,"context_line":"                \u0027requires more than 1 cpu (to set backup_workers \u003e1)\u0027)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"        CONF.set_override(\u0027backup_workers\u0027, 2)"},{"line_number":137,"context_line":"        mock_semaphore.side_effect \u003d [mock.sentinel.semaphore1,"},{"line_number":138,"context_line":"                                      mock.sentinel.semaphore2]"},{"line_number":139,"context_line":"        cinder_backup.main()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9e913bab_d375ee64","line":136,"range":{"start_line":136,"start_character":26,"end_line":136,"end_character":45},"in_reply_to":"78b8d2b2_65999088","updated":"2024-11-15 17:30:29.000000000","message":"Thanks for the reference, makes it easy to review!","commit_id":"005be05ec3a8938c687e7d119b6737819d56236a"}]}