)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"f424d1e0ab35073da08bcd0a8bb7b1547bf55d5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0f5a83d6_b4d4d34a","updated":"2022-08-08 15:45:36.000000000","message":"Thanks for fixing this! Not sure it matters since the unit tests obviously passed, but just want to make sure we want these changes.","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"},{"author":{"_account_id":30566,"name":"Vanou Ishii","email":"ishii.vanou@fujitsu.com","username":"vanou"},"change_message_id":"242785ed52daba0eb1cd9981aad7ffb49671cf68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"10e6236e_5acc30f8","in_reply_to":"0f5a83d6_b4d4d34a","updated":"2022-08-09 06:38:42.000000000","message":"Thanks for review. As you pointed out, there was few mistakes. I rechecked changes and fix them.\nSorry for that.","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"d661be1c43d4bd23856bb30bf2430769b6248168","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"614f14d0_34eaf62a","updated":"2022-08-09 13:11:50.000000000","message":"Thank you, Vanou!\nSince is just unit tests change I\u0027m approving","commit_id":"3b28d0984d5eb82320c632dd5840cebf6d0762ad"},{"author":{"_account_id":30566,"name":"Vanou Ishii","email":"ishii.vanou@fujitsu.com","username":"vanou"},"change_message_id":"6f135efc9549478a4c302c52367d1f27c1abd831","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"180f3c90_5c7f4787","in_reply_to":"614f14d0_34eaf62a","updated":"2022-08-09 13:12:59.000000000","message":"Thanks!","commit_id":"3b28d0984d5eb82320c632dd5840cebf6d0762ad"}],"ironic/tests/unit/common/test_molds.py":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"eb0684bee948fbd356d687a8ae0887e3529fdee5","unresolved":true,"context_lines":[{"line_number":82,"context_line":"    @mock.patch.object(requests, \u0027put\u0027, autospec\u003dTrue)"},{"line_number":83,"context_line":"    def test_save_configuration_http_noauth(self, mock_put):"},{"line_number":84,"context_line":"        cfg.CONF.set_override(\u0027storage\u0027, \u0027http\u0027, \u0027molds\u0027)"},{"line_number":85,"context_line":"        cfg.CONF.set_override(\u0027user\u0027, \u0027user\u0027, \u0027molds\u0027)"},{"line_number":86,"context_line":"        cfg.CONF.set_override(\u0027password\u0027, None, \u0027molds\u0027)"},{"line_number":87,"context_line":"        url \u003d \u0027https://example.com/file1\u0027"},{"line_number":88,"context_line":"        data \u003d {\u0027key\u0027: \u0027value\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"7793694d_01249f60","line":85,"updated":"2022-08-08 15:55:32.000000000","message":"why isn\u0027t this (\u0027user\u0027, None, \u0027molds\u0027) ?","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"965d0d832393017517a09f74b46b7591dc82de3b","unresolved":true,"context_lines":[{"line_number":82,"context_line":"    @mock.patch.object(requests, \u0027put\u0027, autospec\u003dTrue)"},{"line_number":83,"context_line":"    def test_save_configuration_http_noauth(self, mock_put):"},{"line_number":84,"context_line":"        cfg.CONF.set_override(\u0027storage\u0027, \u0027http\u0027, \u0027molds\u0027)"},{"line_number":85,"context_line":"        cfg.CONF.set_override(\u0027user\u0027, \u0027user\u0027, \u0027molds\u0027)"},{"line_number":86,"context_line":"        cfg.CONF.set_override(\u0027password\u0027, None, \u0027molds\u0027)"},{"line_number":87,"context_line":"        url \u003d \u0027https://example.com/file1\u0027"},{"line_number":88,"context_line":"        data \u003d {\u0027key\u0027: \u0027value\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"7e9db9ea_71bbdf46","line":85,"in_reply_to":"7793694d_01249f60","updated":"2022-08-09 00:20:10.000000000","message":"Agree with Ruby, the old version has User as None","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"},{"author":{"_account_id":30566,"name":"Vanou Ishii","email":"ishii.vanou@fujitsu.com","username":"vanou"},"change_message_id":"242785ed52daba0eb1cd9981aad7ffb49671cf68","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    @mock.patch.object(requests, \u0027put\u0027, autospec\u003dTrue)"},{"line_number":83,"context_line":"    def test_save_configuration_http_noauth(self, mock_put):"},{"line_number":84,"context_line":"        cfg.CONF.set_override(\u0027storage\u0027, \u0027http\u0027, \u0027molds\u0027)"},{"line_number":85,"context_line":"        cfg.CONF.set_override(\u0027user\u0027, \u0027user\u0027, \u0027molds\u0027)"},{"line_number":86,"context_line":"        cfg.CONF.set_override(\u0027password\u0027, None, \u0027molds\u0027)"},{"line_number":87,"context_line":"        url \u003d \u0027https://example.com/file1\u0027"},{"line_number":88,"context_line":"        data \u003d {\u0027key\u0027: \u0027value\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"e0afdd5f_2b195ff8","line":85,"in_reply_to":"7e9db9ea_71bbdf46","updated":"2022-08-09 06:38:42.000000000","message":"Sorry. This is my mistake. Thanks","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"f424d1e0ab35073da08bcd0a8bb7b1547bf55d5b","unresolved":true,"context_lines":[{"line_number":216,"context_line":"    @mock.patch.object(requests, \u0027get\u0027, autospec\u003dTrue)"},{"line_number":217,"context_line":"    def test_get_configuration_http_noauth(self, mock_get):"},{"line_number":218,"context_line":"        cfg.CONF.set_override(\u0027storage\u0027, \u0027http\u0027, \u0027molds\u0027)"},{"line_number":219,"context_line":"        cfg.CONF.set_override(\u0027user\u0027, \u0027user\u0027, \u0027molds\u0027)"},{"line_number":220,"context_line":"        cfg.CONF.set_override(\u0027password\u0027, None, \u0027molds\u0027)"},{"line_number":221,"context_line":"        response \u003d mock.MagicMock()"},{"line_number":222,"context_line":"        response.status_code \u003d 200"}],"source_content_type":"text/x-python","patch_set":2,"id":"e24bd5b6_c3dfabf3","line":219,"updated":"2022-08-08 15:45:36.000000000","message":"ditto, \u0027user\u0027 -\u003e None ?","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"},{"author":{"_account_id":30566,"name":"Vanou Ishii","email":"ishii.vanou@fujitsu.com","username":"vanou"},"change_message_id":"242785ed52daba0eb1cd9981aad7ffb49671cf68","unresolved":false,"context_lines":[{"line_number":216,"context_line":"    @mock.patch.object(requests, \u0027get\u0027, autospec\u003dTrue)"},{"line_number":217,"context_line":"    def test_get_configuration_http_noauth(self, mock_get):"},{"line_number":218,"context_line":"        cfg.CONF.set_override(\u0027storage\u0027, \u0027http\u0027, \u0027molds\u0027)"},{"line_number":219,"context_line":"        cfg.CONF.set_override(\u0027user\u0027, \u0027user\u0027, \u0027molds\u0027)"},{"line_number":220,"context_line":"        cfg.CONF.set_override(\u0027password\u0027, None, \u0027molds\u0027)"},{"line_number":221,"context_line":"        response \u003d mock.MagicMock()"},{"line_number":222,"context_line":"        response.status_code \u003d 200"}],"source_content_type":"text/x-python","patch_set":2,"id":"06fa9ef3_aba019b0","line":219,"in_reply_to":"e24bd5b6_c3dfabf3","updated":"2022-08-09 06:38:42.000000000","message":"You\u0027re correct. My mistake. Thanks.","commit_id":"b1c593d159e32f2b41edfdc90e799496b1b4926d"}]}