)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9a5a696302a0b5fd575d1e67c98f1ebe434d56e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8b745e14_69ea7c7f","updated":"2024-09-11 17:34:57.000000000","message":"ill push the update with the rest of the typo fixes in one go at the end.","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"01e2d165f970b03a925572e9dee02c5ea89e1385","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"2cceaea7_be091b9e","updated":"2024-09-11 13:30:14.000000000","message":"looks like the CI is happy so am I","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"70ed64308fadef4a1cc87166a01bffb6cc6ddc31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ed8244e1_a4a7956f","updated":"2024-09-12 16:58:01.000000000","message":"looks good to me.\n(the test is less functional than what I like but I\u0027m OK to have this as is for the current bug.)","commit_id":"e310049eab7dc3d05a25a37c68fec3b34d621c33"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"9bcfa37040410fd855a5f200b7daff0fe6318ac5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e531c109_dd3d8121","updated":"2024-09-16 07:31:02.000000000","message":"recheck","commit_id":"e310049eab7dc3d05a25a37c68fec3b34d621c33"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"ca4613441a2dbc54007e302df55d3e984b769182","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b3c9bcb6_9be616a6","updated":"2024-09-18 06:08:03.000000000","message":"recheck I don\u0027t get it.\n```\n2024-09-17 16:29:43.362127 | controller | Successfully installed python-openstackclient-7.1.1\n12401\t\n```\nbut still\n```\n2024-09-17 16:51:57.419339 | controller | Proxy.evacuate_server() got an unexpected keyword argument \u0027admin_password\u0027\n\n```\n\nhttps://zuul.opendev.org/t/openstack/build/23a7445d4829497586936a502eb5abd8/log/job-output.txt#12400\nhttps://zuul.opendev.org/t/openstack/build/23a7445d4829497586936a502eb5abd8/log/job-output.txt#34852","commit_id":"e310049eab7dc3d05a25a37c68fec3b34d621c33"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"8f0534b5a53b6c6ff8f1bd5bcaa1cc152b37b549","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"93ba0d13_e7bc002e","updated":"2024-09-17 16:03:20.000000000","message":"recheck osc 7.1.1 is bumped to uc","commit_id":"e310049eab7dc3d05a25a37c68fec3b34d621c33"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"4ff86588a7a89061b57ddb0cb5321ca135e97c7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c004b340_ad3ee249","updated":"2024-09-18 17:39:14.000000000","message":"recheck osc pin landed in uc","commit_id":"e310049eab7dc3d05a25a37c68fec3b34d621c33"}],"nova/tests/functional/regressions/test_bug_2079850.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"33fe0e1afa8f5b625c8d480e04ab5954877b4fc9","unresolved":true,"context_lines":[{"line_number":41,"context_line":"           self.skipTest(\"qemu-img not installed\")"},{"line_number":42,"context_line":"        if shutil.which(\u0027mkfs.vfat\u0027) is None:"},{"line_number":43,"context_line":"            subprocess.check_output(\u0027mkfs.vfat --help\u0027)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def test_create_file(self):"},{"line_number":46,"context_line":"        \"\"\"Test that files created files have partition tables"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"72b15c2b_fd5bd776","line":44,"updated":"2024-09-11 00:53:04.000000000","message":"oh i fixed this in the wrong patch thats fixed in the next patch in the serise","commit_id":"46916065d83e1c9a455ff33b87c37613edd1d5a8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f405613d746edb9fc635523f78e7c3a97a002cc2","unresolved":false,"context_lines":[{"line_number":41,"context_line":"           self.skipTest(\"qemu-img not installed\")"},{"line_number":42,"context_line":"        if shutil.which(\u0027mkfs.vfat\u0027) is None:"},{"line_number":43,"context_line":"            subprocess.check_output(\u0027mkfs.vfat --help\u0027)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def test_create_file(self):"},{"line_number":46,"context_line":"        \"\"\"Test that files created files have partition tables"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f0bd1d8_928d8d42","line":44,"in_reply_to":"72b15c2b_fd5bd776","updated":"2024-09-11 00:54:16.000000000","message":"Done","commit_id":"46916065d83e1c9a455ff33b87c37613edd1d5a8"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"01e2d165f970b03a925572e9dee02c5ea89e1385","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        if shutil.which(\"qemu-img\") is None:"},{"line_number":41,"context_line":"            self.skipTest(\"qemu-img not installed\")"},{"line_number":42,"context_line":"        if shutil.which(\"mkfs.vfat\") is None:"},{"line_number":43,"context_line":"            self.skipTest(\"mkfs.vfat not installed\")"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def test_create_file(self):"},{"line_number":46,"context_line":"        \"\"\"Test that files created files have partition tables"}],"source_content_type":"text/x-python","patch_set":7,"id":"d6033ee5_e436f29e","line":43,"updated":"2024-09-11 13:30:14.000000000","message":"++","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"01e2d165f970b03a925572e9dee02c5ea89e1385","unresolved":true,"context_lines":[{"line_number":46,"context_line":"        \"\"\"Test that files created files have partition tables"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        This test will create a file and then inspect it to ensure"},{"line_number":49,"context_line":"        that it has a partition table so that it can be used as a backing"},{"line_number":50,"context_line":"        \"\"\""},{"line_number":51,"context_line":"        file_path \u003d os.path.join(self.base_dir.path, \u0027test_file\u0027)"},{"line_number":52,"context_line":"        libvirt_utils.create_image(file_path, \u0027raw\u0027, \u002764M\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"b78523c4_ee8df9c7","line":49,"range":{"start_line":49,"start_character":66,"end_line":49,"end_character":73},"updated":"2024-09-11 13:30:14.000000000","message":"nit: as a backing *file*","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9a5a696302a0b5fd575d1e67c98f1ebe434d56e4","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        \"\"\"Test that files created files have partition tables"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"        This test will create a file and then inspect it to ensure"},{"line_number":49,"context_line":"        that it has a partition table so that it can be used as a backing"},{"line_number":50,"context_line":"        \"\"\""},{"line_number":51,"context_line":"        file_path \u003d os.path.join(self.base_dir.path, \u0027test_file\u0027)"},{"line_number":52,"context_line":"        libvirt_utils.create_image(file_path, \u0027raw\u0027, \u002764M\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"ee1a35fb_6e28e9d4","line":49,"range":{"start_line":49,"start_character":66,"end_line":49,"end_character":73},"in_reply_to":"b78523c4_ee8df9c7","updated":"2024-09-11 17:34:57.000000000","message":"Done","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"01e2d165f970b03a925572e9dee02c5ea89e1385","unresolved":false,"context_lines":[{"line_number":60,"context_line":"        # partition table so we should expect this to fail"},{"line_number":61,"context_line":"        self.assertRaises("},{"line_number":62,"context_line":"            format_inspector.ImageFormatError,"},{"line_number":63,"context_line":"            format_inspector.GPTInspector.from_file, file_path)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def test_cache_file(self):"},{"line_number":66,"context_line":"        \"\"\"Test that the qcow2 cache interaction for ephemeral disks"}],"source_content_type":"text/x-python","patch_set":7,"id":"ae87a9e3_76f5eca4","line":63,"updated":"2024-09-11 13:30:14.000000000","message":"++","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"01e2d165f970b03a925572e9dee02c5ea89e1385","unresolved":true,"context_lines":[{"line_number":63,"context_line":"            format_inspector.GPTInspector.from_file, file_path)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def test_cache_file(self):"},{"line_number":66,"context_line":"        \"\"\"Test that the qcow2 cache interaction for ephemeral disks"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        This test will create a file via the image backend cache function"},{"line_number":69,"context_line":"        and ensure that the backing file has a partition table"}],"source_content_type":"text/x-python","patch_set":7,"id":"b0ff75c7_9cd513eb","line":66,"range":{"start_line":66,"start_character":16,"end_line":66,"end_character":20},"updated":"2024-09-11 13:30:14.000000000","message":"nit: remove \"that\"","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9a5a696302a0b5fd575d1e67c98f1ebe434d56e4","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            format_inspector.GPTInspector.from_file, file_path)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def test_cache_file(self):"},{"line_number":66,"context_line":"        \"\"\"Test that the qcow2 cache interaction for ephemeral disks"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        This test will create a file via the image backend cache function"},{"line_number":69,"context_line":"        and ensure that the backing file has a partition table"}],"source_content_type":"text/x-python","patch_set":7,"id":"3d36e6d0_4a583d23","line":66,"range":{"start_line":66,"start_character":16,"end_line":66,"end_character":20},"in_reply_to":"b0ff75c7_9cd513eb","updated":"2024-09-11 17:34:57.000000000","message":"Done","commit_id":"c1c7a5c776e8d10fa6114a23daa1ce32d47c02b2"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"70ed64308fadef4a1cc87166a01bffb6cc6ddc31","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# SPDX-License-Identifier: Apache-2.0"},{"line_number":2,"context_line":"import functools"},{"line_number":3,"context_line":"import os"},{"line_number":4,"context_line":"import shutil"}],"source_content_type":"text/x-python","patch_set":8,"id":"a29514b8_73fd736e","line":1,"updated":"2024-09-12 16:58:01.000000000","message":"Is it OK from the Foundation perspective?","commit_id":"e310049eab7dc3d05a25a37c68fec3b34d621c33"}]}