)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36537,"name":"wanghongtao","display_name":"wanghongtao","email":"wanght26@lenovo.com","username":"wanght26"},"change_message_id":"9214bb39826de9de01ebaa071d39c0ad248fe155","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"adcf3d7b_f4e1c6f2","updated":"2026-06-11 07:07:42.000000000","message":"anthony.galica@hitachivantara.com","commit_id":"d42ec512cf2c08e5be7fd754c22c4a4e74042f5e"},{"author":{"_account_id":36537,"name":"wanghongtao","display_name":"wanghongtao","email":"wanght26@lenovo.com","username":"wanght26"},"change_message_id":"4fe07b40496dec33c45e98c633b17230b32fade8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bfa2fc30_a074ba39","updated":"2026-06-11 06:38:55.000000000","message":"recheck","commit_id":"d42ec512cf2c08e5be7fd754c22c4a4e74042f5e"}],"cinder/tests/unit/backup/drivers/test_backup_posix.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b0d07949e724a422d219006feb08fdbffc625718","unresolved":true,"context_lines":[{"line_number":107,"context_line":"    def test_put_container_disabled(self):"},{"line_number":108,"context_line":"        container \u003d f\"{FAKE_CONTAINER}_new\""},{"line_number":109,"context_line":"        self.override_config(\u0027backup_create_containers\u0027, False)"},{"line_number":110,"context_line":"        self.driver.backup_create_containers \u003d False"},{"line_number":111,"context_line":"        self.mock_object(os.path, \u0027exists\u0027, return_value\u003dFalse)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        self.assertRaises(OSError, self.driver.put_container, container)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f27eea8_b56032a9","line":110,"updated":"2026-06-11 13:56:10.000000000","message":"This should be done by line 109, why is this needed?","commit_id":"d42ec512cf2c08e5be7fd754c22c4a4e74042f5e"},{"author":{"_account_id":39181,"name":"huangjs3","display_name":"huangjs3","email":"huangjs3@lenovo.com","username":"huangjs3"},"change_message_id":"b28b686750ceaa014ea0d6ced5fa0e70cdf3ab2a","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    def test_put_container_disabled(self):"},{"line_number":108,"context_line":"        container \u003d f\"{FAKE_CONTAINER}_new\""},{"line_number":109,"context_line":"        self.override_config(\u0027backup_create_containers\u0027, False)"},{"line_number":110,"context_line":"        self.driver.backup_create_containers \u003d False"},{"line_number":111,"context_line":"        self.mock_object(os.path, \u0027exists\u0027, return_value\u003dFalse)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        self.assertRaises(OSError, self.driver.put_container, container)"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ed57d95a_acb995f9","line":111,"range":{"start_line":110,"start_character":8,"end_line":111,"end_character":63},"updated":"2026-06-11 08:24:07.000000000","message":"Marked as resolved.","commit_id":"d42ec512cf2c08e5be7fd754c22c4a4e74042f5e"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"b0d07949e724a422d219006feb08fdbffc625718","unresolved":true,"context_lines":[{"line_number":108,"context_line":"        container \u003d f\"{FAKE_CONTAINER}_new\""},{"line_number":109,"context_line":"        self.override_config(\u0027backup_create_containers\u0027, False)"},{"line_number":110,"context_line":"        self.driver.backup_create_containers \u003d False"},{"line_number":111,"context_line":"        self.mock_object(os.path, \u0027exists\u0027, return_value\u003dFalse)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        self.assertRaises(OSError, self.driver.put_container, container)"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"e86aa5da_cd627bf5","line":111,"updated":"2026-06-11 13:56:10.000000000","message":"Mocking os.path.exists directly can cause weird side effects. You\u0027ll need to mock it relative to the module where it is called.\n\nSee test_image_utils.py where we use\n\n    @mock.patch(\u0027cinder.image.image_utils.os.path.exists\u0027, return_value\u003dTrue)","commit_id":"d42ec512cf2c08e5be7fd754c22c4a4e74042f5e"}]}
