)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"5c73661fb12bc7d3cadd0345adaa86d3d9a27e0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0bc98830_ad989bcf","updated":"2025-04-03 15:19:37.000000000","message":"@jay@jvf.cc sure, this was first round, fixes to tests coming up soon. We\u0027d need to mock the new method out, and extra test case for the new method that mocks \u0027open\u0027 from stdlib","commit_id":"a86bcea4b59415154d4abeeb3ebcce5be5d237ce"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"f16f082c29e68edd29fd086d93d699ac236fe4d2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b401da6_563f95cd","updated":"2025-04-03 15:02:16.000000000","message":"CI failures are real; likely need to update the associated mocks in the unit tests for this code.","commit_id":"a86bcea4b59415154d4abeeb3ebcce5be5d237ce"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"426de7038471d434f47d5f14ae68e5abcc386266","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"40bdc7cf_8563086e","updated":"2025-04-08 18:16:23.000000000","message":"This was looked at again as part of PTG review jam. Multiple cores expressed an interest in landing this change if the tests are fixed. \n\nThank you!","commit_id":"a86bcea4b59415154d4abeeb3ebcce5be5d237ce"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"3c99afdab83317b65838b09309c0e5aa1e0a6267","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"94bec24e_5f290b5a","in_reply_to":"4b401da6_563f95cd","updated":"2025-04-09 23:24:10.000000000","message":"Done","commit_id":"a86bcea4b59415154d4abeeb3ebcce5be5d237ce"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"efe2e05563f8c6188ecd18bd70a5f45904d41ae7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"76049d1d_2f043234","updated":"2025-04-10 13:14:17.000000000","message":"@jay@jvf.cc ready for review, tests fixed and added as needed (unlike that nova patch :-P)","commit_id":"9cfb693bebec64e97f7097e85531ac955eb17b9c"}],"ironic/tests/unit/common/test_images.py":[{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"21516caab35cd06ed3750bb9a61a9633e8b9ceda","unresolved":true,"context_lines":[{"line_number":714,"context_line":"        mock_move.assert_not_called()"},{"line_number":715,"context_line":"        mock_exec.assert_not_called()"},{"line_number":716,"context_line":""},{"line_number":717,"context_line":"    def test_detect_file_format(self):"},{"line_number":718,"context_line":"        self.skipTest(\"Not implemented\")"},{"line_number":719,"context_line":""},{"line_number":720,"context_line":"    def test_detect_file_format_fail(self):"},{"line_number":721,"context_line":"        self.skipTest(\"Not implemented\")"},{"line_number":722,"context_line":""},{"line_number":723,"context_line":"    def test_detect_file_format_iso_gpt(self):"},{"line_number":724,"context_line":"        self.skipTest(\"Not implemented\")"},{"line_number":725,"context_line":""},{"line_number":726,"context_line":""},{"line_number":727,"context_line":"class FsImageTestCase(base.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"ea1801ff_ca14fa62","line":724,"range":{"start_line":717,"start_character":0,"end_line":724,"end_character":40},"updated":"2025-04-09 17:07:57.000000000","message":"need to be implemented yet","commit_id":"e869f2724602af8f0fee38bda68ca913a72d5761"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"6e44bff2a6c380dc2d294cda15bbf92f17d4c4d1","unresolved":false,"context_lines":[{"line_number":714,"context_line":"        mock_move.assert_not_called()"},{"line_number":715,"context_line":"        mock_exec.assert_not_called()"},{"line_number":716,"context_line":""},{"line_number":717,"context_line":"    def test_detect_file_format(self):"},{"line_number":718,"context_line":"        self.skipTest(\"Not implemented\")"},{"line_number":719,"context_line":""},{"line_number":720,"context_line":"    def test_detect_file_format_fail(self):"},{"line_number":721,"context_line":"        self.skipTest(\"Not implemented\")"},{"line_number":722,"context_line":""},{"line_number":723,"context_line":"    def test_detect_file_format_iso_gpt(self):"},{"line_number":724,"context_line":"        self.skipTest(\"Not implemented\")"},{"line_number":725,"context_line":""},{"line_number":726,"context_line":""},{"line_number":727,"context_line":"class FsImageTestCase(base.TestCase):"}],"source_content_type":"text/x-python","patch_set":2,"id":"c748f3fd_259496d8","line":724,"range":{"start_line":717,"start_character":0,"end_line":724,"end_character":40},"in_reply_to":"ea1801ff_ca14fa62","updated":"2025-04-10 13:10:03.000000000","message":"Done","commit_id":"e869f2724602af8f0fee38bda68ca913a72d5761"}]}
