)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6842527ee46d014ab5f7209edfbce606c9aa4b9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"808dcb21_494fe393","updated":"2022-10-18 14:12:53.000000000","message":"recheck unrelated nova-next failure","commit_id":"e4bf44148b30392d5982feb793ecfd64b2ecc355"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"c2c853a464f1f951943501c9e85a21cd5431a9db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6d4c58db_e1dc586b","updated":"2022-11-14 12:58:19.000000000","message":"lgtm","commit_id":"9b6aaff888bbe2c4923451cab774010b05a98058"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"0a4d5a0b4b88abd051e4d37204de6cb4ac04d424","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"390cddc5_166e7e27","updated":"2023-06-28 13:39:47.000000000","message":"Check that patch, it looks good to me.","commit_id":"465530fb2f0debd5739e3717c87c6b4b662788fa"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"071efc0cc5123259a03712d299672927413bd807","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"df4cc01c_81230c85","updated":"2023-11-14 22:19:41.000000000","message":"recheck get fresh results","commit_id":"2f706a31abfbc853b498e2b2ef2e336e5ddf8718"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a019458a4d0849ec02d29e75972a27b0c8d76f51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"05c6c80b_47f50cd3","updated":"2024-02-06 22:16:20.000000000","message":"recheck https://review.opendev.org/c/openstack/nova/+/908182 has merged","commit_id":"fab40d60fd4b9c11e76001df2d3c3b1d8edda97a"},{"author":{"_account_id":16207,"name":"ribaudr","display_name":"uggla","email":"rene.ribaud@gmail.com","username":"uggla","status":"Red Hat"},"change_message_id":"0831a5128b019a7e4a8f9fb92921066b4c44faed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"056d95f5_0b1e6c78","updated":"2024-02-14 10:16:00.000000000","message":"It was good before still ok for me. Especially now, you are a vertical backspace hunter. ;)","commit_id":"6854c8007eabaa2b42a77bb1858e3a714d3699ea"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ab61fb517852b648766e9cdad903f0bfde959ee0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"56d556f1_4e601fef","updated":"2024-02-19 01:56:54.000000000","message":"recheck","commit_id":"215869dbe084e22d568c2f21848bc94cbc4371ff"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"4e7e965fc41a18eba25fba768f1ab67c646bacf8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"f5de2635_9178cdcb","updated":"2024-02-26 06:53:50.000000000","message":"lgtm","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f139f2abf0f0e48489087bcc25c90501ef7da66e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"b61ec11f_e94915d6","updated":"2024-03-01 00:25:43.000000000","message":"recheck ceph meltdown\n\n2024-02-29T20:49:49.576+0000 7fb6360af700  1 mon.np0036919478@0(leader).osd e96 _set_new_cache_sizes cache_size:1020054731 inc_alloc: 348127232 full_alloc: 348127232 kv_alloc: 322961408\n2024-02-29T20:49:54.580+0000 7fb6360af700  0 log_channel(cluster) log [WRN] : Health check update: 12 slow ops, oldest one blocked for 1910 sec, osd.0 has slow ops (SLOW_OPS)","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"dbcb94e83d0f4a40d4e4c1154ab9196b29881be2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"db3b454e_1fb40426","updated":"2024-02-29 19:38:14.000000000","message":"recheck guest kernel crash","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"e903d5a44dd9137620c5650aea9db4362869603b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"f345bf52_8d20d029","updated":"2024-02-28 14:50:52.000000000","message":"recheck hmm same fail on the enc-lvm job.. hope it\u0027s not legit.","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"9a2bcd263430681c590c95ec350ee29fc9c1621e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"da81c679_a2a28a48","updated":"2024-03-01 06:03:39.000000000","message":"recheck tempest create_server urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host\u003d\u002710.209.38.190\u0027, port\u003d443): Read timed out. (read timeout\u003d60)","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"14bf06f594fa91018103d0d9bc700476450d791c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"af536c0b_757b1cab","updated":"2024-02-29 07:41:20.000000000","message":"recheck tempest create_server urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host\u003d\u002710.209.64.171\u0027, port\u003d443): Read timed out. (read timeout\u003d60)","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"b1d7abe46fcecad83aeaed90923c42920d33c81f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"ed772e1f_6f34b9ea","updated":"2024-02-29 15:44:56.000000000","message":"recheck timed out running (and progressing) in scenario tests","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3f17a145fce6150be62ac0413254362fd2313759","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"daca5d63_1a8ec64f","in_reply_to":"f345bf52_8d20d029","updated":"2024-02-28 16:24:14.000000000","message":"Do you mean barbican-tempest-plugin-simple-crypto? It looks like it\u0027s perma failing on the same tests and not only on the patches in this series:\n\nhttps://zuul.opendev.org/t/openstack/builds?job_name\u003dbarbican-tempest-plugin-simple-crypto\u0026project\u003dopenstack/nova\n\nThe failures look consistent though so I think something is likely legitimately broken in that job but I don\u0027t think it\u0027s related to patches in this series.","commit_id":"3391ac2656d794ebadd83540d0b4fd353f369d4a"}],"nova/tests/unit/virt/libvirt/test_imagebackend.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"627a2680ae99ba87079ea76b7ab7716b51e01fc8","unresolved":true,"context_lines":[{"line_number":260,"context_line":"        self.assertIsInstance("},{"line_number":261,"context_line":"            disk.encryption.secret,"},{"line_number":262,"context_line":"            vconfig.LibvirtConfigGuestDiskEncryptionSecret"},{"line_number":263,"context_line":"        )"},{"line_number":264,"context_line":"        self.assertEqual(\"passphrase\", disk.encryption.secret.type)"},{"line_number":265,"context_line":"        self.assertEqual(uuids.secret, disk.encryption.secret.uuid)"},{"line_number":266,"context_line":"        self.assertEqual(\"luks\", disk.encryption.format)"}],"source_content_type":"text/x-python","patch_set":21,"id":"4536929f_e47283f0","line":263,"updated":"2023-11-15 21:09:56.000000000","message":"If you respin, can I have my vertical space back please? :)","commit_id":"2f706a31abfbc853b498e2b2ef2e336e5ddf8718"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"45a0d8df8a5e7e1400f38bfea67637f061cde680","unresolved":false,"context_lines":[{"line_number":260,"context_line":"        self.assertIsInstance("},{"line_number":261,"context_line":"            disk.encryption.secret,"},{"line_number":262,"context_line":"            vconfig.LibvirtConfigGuestDiskEncryptionSecret"},{"line_number":263,"context_line":"        )"},{"line_number":264,"context_line":"        self.assertEqual(\"passphrase\", disk.encryption.secret.type)"},{"line_number":265,"context_line":"        self.assertEqual(uuids.secret, disk.encryption.secret.uuid)"},{"line_number":266,"context_line":"        self.assertEqual(\"luks\", disk.encryption.format)"}],"source_content_type":"text/x-python","patch_set":21,"id":"35b80b75_0cd625e8","line":263,"in_reply_to":"4536929f_e47283f0","updated":"2023-11-16 23:22:08.000000000","message":"Done 😛","commit_id":"2f706a31abfbc853b498e2b2ef2e336e5ddf8718"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"2438728acf97e2de477a0edfa9c4cbf0436ca31e","unresolved":true,"context_lines":[{"line_number":244,"context_line":"            classname \u003d type(image).__name__"},{"line_number":245,"context_line":"            self.skipTest("},{"line_number":246,"context_line":"                f\"LUKS encryption is not supported with {classname}\")"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"        disk \u003d image.libvirt_info("},{"line_number":249,"context_line":"            cache_mode\u003d\"none\", extra_specs\u003d{}, boot_order\u003d\"1\")"},{"line_number":250,"context_line":""}],"source_content_type":"text/x-python","patch_set":35,"id":"066178a8_0fef59cc","line":247,"updated":"2024-02-22 08:24:12.000000000","message":"how to test below from here.\nits get tested for below and skipped for all:\n`\u0027Qcow2TestCase , LvmTestCase , RbdTestCase , FlatTestCase , PloopTestCase , EncryptedLvmTestCase\u0027`\n\nand for others its not added,`PloopTestCase, UtimeWorkaroundTestCase`","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a23efb7cab83bfb3a52fa00ed54f13004701cee9","unresolved":true,"context_lines":[{"line_number":244,"context_line":"            classname \u003d type(image).__name__"},{"line_number":245,"context_line":"            self.skipTest("},{"line_number":246,"context_line":"                f\"LUKS encryption is not supported with {classname}\")"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"        disk \u003d image.libvirt_info("},{"line_number":249,"context_line":"            cache_mode\u003d\"none\", extra_specs\u003d{}, boot_order\u003d\"1\")"},{"line_number":250,"context_line":""}],"source_content_type":"text/x-python","patch_set":35,"id":"f521f1f5_d23f58a0","line":247,"in_reply_to":"066178a8_0fef59cc","updated":"2024-02-22 16:58:57.000000000","message":"The test is expected to not be in UtimeWorkaroundTestCase because the test is in the _ImageTestCase class and UtimeWorkaroundTestCase does not derive from _ImageTestCase.\n\nAs for the other skips, most of them will remain skips pretty much forever, but for Qcow2TestCase, RbdTestCase, and FlatTestCase the tests will become unskipped in the respective patches that set SUPPORTS_LUKS \u003d True for the backend.\n\nFor example, in the patch a few patches after this one [1], the test will start running for Qcow2TestCase. For FlatTestCase, it will start running as of [2]. For RbdTestCase it will start running as of [3].\n\nI could technically fake out SUPPORTS_LUKS in this patch to make one or more of them run, but it\u0027s a bit chicken and egg because libvirt_info for each image backend may not have the changes it needs until the patch that enables SUPPORTS_LUKS. So this patch is currently a foundational patch or a preparation patch for later patches, given the need to split up a large feature like this.\n\n[1] https://review.opendev.org/c/openstack/nova/+/870932/32/nova/virt/libvirt/imagebackend.py#674\n[2] https://review.opendev.org/c/openstack/nova/+/884313/37/nova/virt/libvirt/imagebackend.py#609\n[3] https://review.opendev.org/c/openstack/nova/+/889912/41/nova/virt/libvirt/imagebackend.py#1032","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5c5bacced7e881f0f659931f102eb86d705969b0","unresolved":false,"context_lines":[{"line_number":244,"context_line":"            classname \u003d type(image).__name__"},{"line_number":245,"context_line":"            self.skipTest("},{"line_number":246,"context_line":"                f\"LUKS encryption is not supported with {classname}\")"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"        disk \u003d image.libvirt_info("},{"line_number":249,"context_line":"            cache_mode\u003d\"none\", extra_specs\u003d{}, boot_order\u003d\"1\")"},{"line_number":250,"context_line":""}],"source_content_type":"text/x-python","patch_set":35,"id":"52ecf02d_f5a06bfe","line":247,"in_reply_to":"df904ecf_48873118","updated":"2024-02-27 09:15:54.000000000","message":"Acknowledged","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"4e7e965fc41a18eba25fba768f1ab67c646bacf8","unresolved":true,"context_lines":[{"line_number":244,"context_line":"            classname \u003d type(image).__name__"},{"line_number":245,"context_line":"            self.skipTest("},{"line_number":246,"context_line":"                f\"LUKS encryption is not supported with {classname}\")"},{"line_number":247,"context_line":""},{"line_number":248,"context_line":"        disk \u003d image.libvirt_info("},{"line_number":249,"context_line":"            cache_mode\u003d\"none\", extra_specs\u003d{}, boot_order\u003d\"1\")"},{"line_number":250,"context_line":""}],"source_content_type":"text/x-python","patch_set":35,"id":"df904ecf_48873118","line":247,"in_reply_to":"f521f1f5_d23f58a0","updated":"2024-02-26 06:53:50.000000000","message":"Acknowledged","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"}],"nova/virt/libvirt/imagebackend.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"d4def9a407b1c6a4f55b8728d4c84b808f09e9e8","unresolved":true,"context_lines":[{"line_number":185,"context_line":"        info.source_path \u003d self.path"},{"line_number":186,"context_line":"        info.boot_order \u003d boot_order"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"        if (self.SUPPORTS_LUKS and"},{"line_number":189,"context_line":"            self.disk_info_mapping and"},{"line_number":190,"context_line":"            self.disk_info_mapping.get(\u0027encrypted\u0027) and"},{"line_number":191,"context_line":"            self.disk_info_mapping.get(\u0027encryption_format\u0027) \u003d\u003d \u0027luks\u0027"}],"source_content_type":"text/x-python","patch_set":20,"id":"566ab388_69d787d7","line":188,"updated":"2023-08-08 20:56:16.000000000","message":"This is per type... I assume we\u0027re well within the libvirt support envelope for this always being available? Do we not need to check that the host claims support, like if libvirt was compiled without support or something? My point is I\u0027m guessing the host capabilities in libvirt expose an equivalent of \"SUPPORTS_LUKS\". I\u0027m not sure what our usual practice is here, I can just see it not being available, as security-sensitive people *love* compiling out everything they don\u0027t use :)","commit_id":"42eef2afe2afbd7932ae1a4d19e15a77216281ff"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"2bdedc81a0a4f6257063d5c8d0f8f22f563c208c","unresolved":true,"context_lines":[{"line_number":185,"context_line":"        info.source_path \u003d self.path"},{"line_number":186,"context_line":"        info.boot_order \u003d boot_order"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"        if (self.SUPPORTS_LUKS and"},{"line_number":189,"context_line":"            self.disk_info_mapping and"},{"line_number":190,"context_line":"            self.disk_info_mapping.get(\u0027encrypted\u0027) and"},{"line_number":191,"context_line":"            self.disk_info_mapping.get(\u0027encryption_format\u0027) \u003d\u003d \u0027luks\u0027"}],"source_content_type":"text/x-python","patch_set":20,"id":"99b2516b_997effa7","line":188,"in_reply_to":"566ab388_69d787d7","updated":"2023-08-09 03:29:46.000000000","message":"I would assume so as well but I don\u0027t know for sure ... I\u0027ll check around and see if there is a way to check for it at a libvirt level for additional safety.","commit_id":"42eef2afe2afbd7932ae1a4d19e15a77216281ff"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"7b68c6c80090c26d83615f1289fef26385a75112","unresolved":true,"context_lines":[{"line_number":185,"context_line":"        info.source_path \u003d self.path"},{"line_number":186,"context_line":"        info.boot_order \u003d boot_order"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"        if (self.SUPPORTS_LUKS and"},{"line_number":189,"context_line":"            self.disk_info_mapping and"},{"line_number":190,"context_line":"            self.disk_info_mapping.get(\u0027encrypted\u0027) and"},{"line_number":191,"context_line":"            self.disk_info_mapping.get(\u0027encryption_format\u0027) \u003d\u003d \u0027luks\u0027"}],"source_content_type":"text/x-python","patch_set":20,"id":"e217860c_8feeaf66","line":188,"in_reply_to":"95f118d4_44a647cc","updated":"2023-08-10 13:38:20.000000000","message":"Okay, I looked in capabilities and pool-capabilities and I don\u0027t see anything about it either, so I guess there\u0027s nothing to key off of.\n\nAnyway, all good, just wanted to make sure we weren\u0027t missing an opportunity to reject something before it fails, like we do for other feature type things.","commit_id":"42eef2afe2afbd7932ae1a4d19e15a77216281ff"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"41f87d6b4739cad31bf5953cd6256006f5d9add2","unresolved":true,"context_lines":[{"line_number":185,"context_line":"        info.source_path \u003d self.path"},{"line_number":186,"context_line":"        info.boot_order \u003d boot_order"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"        if (self.SUPPORTS_LUKS and"},{"line_number":189,"context_line":"            self.disk_info_mapping and"},{"line_number":190,"context_line":"            self.disk_info_mapping.get(\u0027encrypted\u0027) and"},{"line_number":191,"context_line":"            self.disk_info_mapping.get(\u0027encryption_format\u0027) \u003d\u003d \u0027luks\u0027"}],"source_content_type":"text/x-python","patch_set":20,"id":"95f118d4_44a647cc","line":188,"in_reply_to":"99b2516b_997effa7","updated":"2023-08-10 01:47:50.000000000","message":"Stealing from a commit about supporting LUKS encrypted volumes [1], you just need QEMU 2.6 and Libvirt 2.2.0 to use it. (And our minimums are 4.2 and 6.0.0)\n\nI didn\u0027t see anything related to LUKS or encryption in the host or domain capabilities [2][3] nor the libvirt or QEMU build options [4][5].\n\nIf you know of another place to look, let me know.\n\n[1] https://opendev.org/openstack/nova/commit/f8e24c3\n[2] https://libvirt.org/formatcaps.html\n[3] https://libvirt.org/formatdomaincaps.html\n[4] https://github.com/libvirt/libvirt/blob/831b5a6/meson_options.txt\n[5] https://github.com/qemu/qemu/blob/64d3be9/meson_options.txt","commit_id":"42eef2afe2afbd7932ae1a4d19e15a77216281ff"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6560bf254a6082d9dee3c4327739596dd0461bd1","unresolved":false,"context_lines":[{"line_number":185,"context_line":"        info.source_path \u003d self.path"},{"line_number":186,"context_line":"        info.boot_order \u003d boot_order"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"        if (self.SUPPORTS_LUKS and"},{"line_number":189,"context_line":"            self.disk_info_mapping and"},{"line_number":190,"context_line":"            self.disk_info_mapping.get(\u0027encrypted\u0027) and"},{"line_number":191,"context_line":"            self.disk_info_mapping.get(\u0027encryption_format\u0027) \u003d\u003d \u0027luks\u0027"}],"source_content_type":"text/x-python","patch_set":20,"id":"b7b04bad_59325dbf","line":188,"in_reply_to":"e217860c_8feeaf66","updated":"2024-01-25 12:03:50.000000000","message":"Acknowledged","commit_id":"42eef2afe2afbd7932ae1a4d19e15a77216281ff"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"2438728acf97e2de477a0edfa9c4cbf0436ca31e","unresolved":true,"context_lines":[{"line_number":197,"context_line":"            encryption.secret \u003d secret"},{"line_number":198,"context_line":"            encryption.format \u003d self.disk_info_mapping.get(\u0027encryption_format\u0027)"},{"line_number":199,"context_line":"            info.encryption \u003d encryption"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"        if disk_bus \u003d\u003d \u0027scsi\u0027:"},{"line_number":202,"context_line":"            self.disk_scsi(info, disk_unit)"},{"line_number":203,"context_line":""}],"source_content_type":"text/x-python","patch_set":35,"id":"47914355_81066d6f","line":200,"updated":"2024-02-22 08:24:12.000000000","message":"commented added code and ran cover, no test failed.","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"4e7e965fc41a18eba25fba768f1ab67c646bacf8","unresolved":false,"context_lines":[{"line_number":197,"context_line":"            encryption.secret \u003d secret"},{"line_number":198,"context_line":"            encryption.format \u003d self.disk_info_mapping.get(\u0027encryption_format\u0027)"},{"line_number":199,"context_line":"            info.encryption \u003d encryption"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"        if disk_bus \u003d\u003d \u0027scsi\u0027:"},{"line_number":202,"context_line":"            self.disk_scsi(info, disk_unit)"},{"line_number":203,"context_line":""}],"source_content_type":"text/x-python","patch_set":35,"id":"1947bb54_cac9e005","line":200,"in_reply_to":"47914355_81066d6f","updated":"2024-02-26 06:53:50.000000000","message":"Done","commit_id":"c6408859d3cf97ce657a6581f282d4343a644372"}]}