)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"50034d24d9b6c2426f6b5433f01c565e005964bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"aece58ca_bc3c6b46","updated":"2026-07-27 15:53:39.000000000","message":"recheck","commit_id":"05112d9b20b1e857b228f71d688bdfe22d568c2d"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"3077e5843b49b7cdddc0342098244bb1dd71c35a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"91e6c6f5_b75e7822","updated":"2026-08-14 18:12:07.000000000","message":"@gouthampravi@gmail.com Thank you for a solid baseline and partial blueprint coverage. See inline comments for enhancement suggestions.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"65d2560d24fa63d8ddac891107f42fef33c3944e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"b88a8cef_94a589be","updated":"2026-08-25 12:26:00.000000000","message":"Thank you for working on these tests. Looks good, only have minor suggestions for using constants instead of rewriting the strings all the time","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"1395cc5d46a566e68b4b82ec1c56b722125be438","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"1fb14e55_65bd99ec","updated":"2026-08-28 11:07:46.000000000","message":"Looks good, thanks Goutham.","commit_id":"2aa8df38b2f48cb6a1c3a9a5d1b16971eb25bc12"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"d5d9c6160b24530cf90d937320df976ba061db5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"fa00fbe4_e0d95f34","updated":"2026-08-31 13:26:30.000000000","message":"Thank you for working on this change and addressing the feedback we shared. It does need the core feature in Nova to merge before we can actually merge this in our tempest plugin.","commit_id":"2aa8df38b2f48cb6a1c3a9a5d1b16971eb25bc12"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0753c782596db611fbb77445ca93498f04f1b373","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"380a6574_deb3b003","updated":"2026-08-27 00:24:27.000000000","message":"Thanks for the reviews @vhariria@redhat.com @ces.eduardo98@gmail.com","commit_id":"2aa8df38b2f48cb6a1c3a9a5d1b16971eb25bc12"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5efa2e9a71ccfabd7250ff3a4612c96b2b072ed4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"42721492_d8d83901","updated":"2026-09-03 06:06:15.000000000","message":"@vhariria@redhat.com @ces.eduardo98@gmail.com: The nova code changed, and there isn\u0027t a microversion bump anymore. So this patch had to be changed","commit_id":"a1fdaab508cda45e032ed3ef805d401427f2a8d4"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"a5556efef9f1c1dfff13abea11142063dfa975ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"2cec20a1_c75f3c70","updated":"2026-09-15 13:07:35.000000000","message":"LGTM, thank you Goutham!","commit_id":"a1fdaab508cda45e032ed3ef805d401427f2a8d4"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"4b976e8f8e04183ef8102e061ef2fc3d33ec6577","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"b762ab51_ca7a8945","updated":"2026-09-16 12:05:20.000000000","message":"LGTM, thank you.","commit_id":"a1fdaab508cda45e032ed3ef805d401427f2a8d4"}],"manila_tempest_tests/tests/scenario/manager_share.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"65d2560d24fa63d8ddac891107f42fef33c3944e","unresolved":true,"context_lines":[{"line_number":868,"context_line":"            server \u003d self.servers_client.show_server(server_id)[\u0027server\u0027]"},{"line_number":869,"context_line":"            server_status \u003d server[\u0027status\u0027]"},{"line_number":870,"context_line":""},{"line_number":871,"context_line":"            if server_status \u003d\u003d \u0027VERIFY_RESIZE\u0027:"},{"line_number":872,"context_line":"                self.servers_client.confirm_resize_server(server_id)"},{"line_number":873,"context_line":"                waiters.wait_for_server_status("},{"line_number":874,"context_line":"                    self.servers_client, server_id,"}],"source_content_type":"text/x-python","patch_set":25,"id":"521b0e7c_16ca33e5","line":871,"range":{"start_line":871,"start_character":32,"end_line":871,"end_character":47},"updated":"2026-08-25 12:26:00.000000000","message":"can we have this as a variable in constants?","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"d5d9c6160b24530cf90d937320df976ba061db5f","unresolved":false,"context_lines":[{"line_number":868,"context_line":"            server \u003d self.servers_client.show_server(server_id)[\u0027server\u0027]"},{"line_number":869,"context_line":"            server_status \u003d server[\u0027status\u0027]"},{"line_number":870,"context_line":""},{"line_number":871,"context_line":"            if server_status \u003d\u003d \u0027VERIFY_RESIZE\u0027:"},{"line_number":872,"context_line":"                self.servers_client.confirm_resize_server(server_id)"},{"line_number":873,"context_line":"                waiters.wait_for_server_status("},{"line_number":874,"context_line":"                    self.servers_client, server_id,"}],"source_content_type":"text/x-python","patch_set":25,"id":"f4664f9d_ebd5ad25","line":871,"range":{"start_line":871,"start_character":32,"end_line":871,"end_character":47},"in_reply_to":"521b0e7c_16ca33e5","updated":"2026-08-31 13:26:30.000000000","message":"Done","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"}],"manila_tempest_tests/tests/scenario/test_share_attachment.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"fc8f21566feb8715c07309696f4dc710a7af74a1","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import testtools"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":"from tempest.common import waiters as tempest_waiters"},{"line_number":19,"context_line":"from tempest import config"},{"line_number":20,"context_line":"from tempest.lib import decorators"}],"source_content_type":"text/x-python","patch_set":4,"id":"03663ab5_474e8510","line":17,"updated":"2026-07-19 21:28:30.000000000","message":"pep8: H306: imports not in alphabetical order (testtools, oslo_log.log)","commit_id":"6fb3de0d57193692faf59b23963dcdd96deb1bc5"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"65d2560d24fa63d8ddac891107f42fef33c3944e","unresolved":true,"context_lines":[{"line_number":409,"context_line":"        \"\"\"Cold migrate a server and wait for VERIFY_RESIZE.\"\"\""},{"line_number":410,"context_line":"        self.os_admin.servers_client.migrate_server(server_id)"},{"line_number":411,"context_line":"        tempest_waiters.wait_for_server_status("},{"line_number":412,"context_line":"            self.servers_client, server_id, \u0027VERIFY_RESIZE\u0027)"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"    def resize_server(self, server_id, flavor_ref):"},{"line_number":415,"context_line":"        \"\"\"Resize a server and wait for VERIFY_RESIZE.\"\"\""}],"source_content_type":"text/x-python","patch_set":25,"id":"26cdb430_4bffd10d","line":412,"range":{"start_line":412,"start_character":45,"end_line":412,"end_character":58},"updated":"2026-08-25 12:26:00.000000000","message":"same about the constant :)","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"bd3237d1ea9f24a4286cf9d8e153638cb0bbcff3","unresolved":false,"context_lines":[{"line_number":409,"context_line":"        \"\"\"Cold migrate a server and wait for VERIFY_RESIZE.\"\"\""},{"line_number":410,"context_line":"        self.os_admin.servers_client.migrate_server(server_id)"},{"line_number":411,"context_line":"        tempest_waiters.wait_for_server_status("},{"line_number":412,"context_line":"            self.servers_client, server_id, \u0027VERIFY_RESIZE\u0027)"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"    def resize_server(self, server_id, flavor_ref):"},{"line_number":415,"context_line":"        \"\"\"Resize a server and wait for VERIFY_RESIZE.\"\"\""}],"source_content_type":"text/x-python","patch_set":25,"id":"d8ecab19_d85413bf","line":412,"range":{"start_line":412,"start_character":45,"end_line":412,"end_character":58},"in_reply_to":"26cdb430_4bffd10d","updated":"2026-08-27 00:24:07.000000000","message":"Done","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"65d2560d24fa63d8ddac891107f42fef33c3944e","unresolved":true,"context_lines":[{"line_number":442,"context_line":"        test_file \u003d \"/mnt/migration_test.txt\""},{"line_number":443,"context_line":"        share_tag \u003d \"share-migrate\""},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"        instance \u003d self.boot_instance(wait_until\u003d\"ACTIVE\")"},{"line_number":446,"context_line":"        self.addCleanup(self.servers_client.delete_server, instance[\u0027id\u0027])"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"        share \u003d self.create_share()"}],"source_content_type":"text/x-python","patch_set":25,"id":"b8c06aec_3e6bbea3","line":445,"range":{"start_line":445,"start_character":49,"end_line":445,"end_character":57},"updated":"2026-08-25 12:26:00.000000000","message":"constants.INSTANCE_STATUS_ACTIVE","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"bd3237d1ea9f24a4286cf9d8e153638cb0bbcff3","unresolved":false,"context_lines":[{"line_number":442,"context_line":"        test_file \u003d \"/mnt/migration_test.txt\""},{"line_number":443,"context_line":"        share_tag \u003d \"share-migrate\""},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"        instance \u003d self.boot_instance(wait_until\u003d\"ACTIVE\")"},{"line_number":446,"context_line":"        self.addCleanup(self.servers_client.delete_server, instance[\u0027id\u0027])"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"        share \u003d self.create_share()"}],"source_content_type":"text/x-python","patch_set":25,"id":"bb1d2ca9_47cc5821","line":445,"range":{"start_line":445,"start_character":49,"end_line":445,"end_character":57},"in_reply_to":"b8c06aec_3e6bbea3","updated":"2026-08-27 00:24:07.000000000","message":"Done","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"3077e5843b49b7cdddc0342098244bb1dd71c35a","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"1ea06583_bf65a4ec","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"updated":"2026-08-14 18:12:07.000000000","message":"May consider to add self.assertNotEqual(source_host, dest_host) to verify actual migration. Data assertion alone doesn\u0027t confirm that cross-host migration occurred.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"dfe22e866fdc92b24ec38a8205a91bba1ca799cf","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"96f0949c_3839f93e","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"1ea06583_bf65a4ec","updated":"2026-08-14 19:12:51.000000000","message":"@vhariria@redhat.com:\n\nHey, good suggestion, but we have a wrinkle.. whether the migration goes to a new host or can be same host is now left up to Nova to decide (and intentionally, so that we can test whatever the infrastructure allows). So we can\u0027t deterministically assert that the source and destination are different.\n\n\"resize\" can be done on the same host always. \n\nSo, asserting the state transition is a good/deterministic fate for this test; the host thingy can reasonably be tested by Nova API tests.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"bd3237d1ea9f24a4286cf9d8e153638cb0bbcff3","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"6009b0fb_8eb69178","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"48c9c08c_e9968a48","updated":"2026-08-27 00:24:07.000000000","message":"It simply can\u0027t, unfortunately :) \n\nI can\u0027t do single-host resize/cold-migration testing (i.e., devstack) without surgically hacking the libvirt driver in nova. I went down that path, and thought it was too flaky to do reliably. That\u0027s why this change adds a multinode LVM setup. I could do the same with CephFS... but was weighing the cost of doing so on our CI... maybe this is worth discussing together. (We need to continuously rationalize our CI jobs for master and for stable branches)","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"635f5a6167ffe0ec924a36d17fe85b2fc50be39c","unresolved":false,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"45bd5fa6_5680de22","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"6009b0fb_8eb69178","updated":"2026-09-03 06:06:39.000000000","message":"marking this resolved","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"65d2560d24fa63d8ddac891107f42fef33c3944e","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"edb77474_f151657a","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"84ca9b86_d6c5e38e","updated":"2026-08-25 12:26:00.000000000","message":"\u003e same-host resize would bypass the virtiofs-specific migration path\n\nWell, I think not necessarily... As goutham mentioned, it can be up to the nova scheduler to determine where the instance will land and it can land in the same host if no other is available. This could actually be the case for this devstack testing - we\u0027re testing with a single node and the source and destination host will very likely be the same.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"6d7ea9e9f73b9e5700aacc4985ad9a853df60fe1","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"84ca9b86_d6c5e38e","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"96f0949c_3839f93e","updated":"2026-08-25 10:21:45.000000000","message":"Understood, but in that case, same-host resize would bypass the virtiofs-specific migration path.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"fe61415465b3003bb1fa7bc2e05d1602fc205963","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"ef5c6df9_518eb133","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"edb77474_f151657a","updated":"2026-08-25 15:35:52.000000000","message":"yeah, it doesn\u0027t fit the definition of a \"blackbox\" that tempest tests need to be.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"19e826be471c4316ba0bf986df35e0642a626871","unresolved":true,"context_lines":[{"line_number":469,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":470,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":471,"context_line":"            remote_client, test_file)"},{"line_number":472,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":473,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"        self.confirm_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"48c9c08c_e9968a48","line":472,"range":{"start_line":472,"start_character":0,"end_line":472,"end_character":46},"in_reply_to":"ef5c6df9_518eb133","updated":"2026-08-26 12:03:24.000000000","message":"Understood. Good point, that path can be covered in extended tests if needed.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"65d2560d24fa63d8ddac891107f42fef33c3944e","unresolved":true,"context_lines":[{"line_number":504,"context_line":"        test_file \u003d \"/mnt/migration_revert_test.txt\""},{"line_number":505,"context_line":"        share_tag \u003d \"share-migrate-revert\""},{"line_number":506,"context_line":""},{"line_number":507,"context_line":"        instance \u003d self.boot_instance(wait_until\u003d\"ACTIVE\")"},{"line_number":508,"context_line":"        self.addCleanup(self.servers_client.delete_server, instance[\u0027id\u0027])"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"        share \u003d self.create_share()"}],"source_content_type":"text/x-python","patch_set":25,"id":"1e53d585_212f9511","line":507,"range":{"start_line":507,"start_character":50,"end_line":507,"end_character":56},"updated":"2026-08-25 12:26:00.000000000","message":"please use constants","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"bd3237d1ea9f24a4286cf9d8e153638cb0bbcff3","unresolved":false,"context_lines":[{"line_number":504,"context_line":"        test_file \u003d \"/mnt/migration_revert_test.txt\""},{"line_number":505,"context_line":"        share_tag \u003d \"share-migrate-revert\""},{"line_number":506,"context_line":""},{"line_number":507,"context_line":"        instance \u003d self.boot_instance(wait_until\u003d\"ACTIVE\")"},{"line_number":508,"context_line":"        self.addCleanup(self.servers_client.delete_server, instance[\u0027id\u0027])"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"        share \u003d self.create_share()"}],"source_content_type":"text/x-python","patch_set":25,"id":"01ff3d07_176bac65","line":507,"range":{"start_line":507,"start_character":50,"end_line":507,"end_character":56},"in_reply_to":"1e53d585_212f9511","updated":"2026-08-27 00:24:07.000000000","message":"Done","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"3077e5843b49b7cdddc0342098244bb1dd71c35a","unresolved":true,"context_lines":[{"line_number":531,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":532,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":533,"context_line":"            remote_client, test_file)"},{"line_number":534,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":535,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":536,"context_line":""},{"line_number":537,"context_line":"        self.revert_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"64c8073b_59f056ac","line":534,"range":{"start_line":534,"start_character":9,"end_line":534,"end_character":46},"updated":"2026-08-14 18:12:07.000000000","message":"Same suggestion as above.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"bd3237d1ea9f24a4286cf9d8e153638cb0bbcff3","unresolved":false,"context_lines":[{"line_number":531,"context_line":"        self.mount_share_via_virtiofs(remote_client, share_tag)"},{"line_number":532,"context_line":"        read_data \u003d self.read_data_from_mounted_share("},{"line_number":533,"context_line":"            remote_client, test_file)"},{"line_number":534,"context_line":"        self.assertEqual(test_data, read_data)"},{"line_number":535,"context_line":"        self.unmount_share_via_virtiofs(remote_client)"},{"line_number":536,"context_line":""},{"line_number":537,"context_line":"        self.revert_resize_server(instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":25,"id":"42f2f28b_d2ffc92f","line":534,"range":{"start_line":534,"start_character":9,"end_line":534,"end_character":46},"in_reply_to":"64c8073b_59f056ac","updated":"2026-08-27 00:24:07.000000000","message":"Acknowledged","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"}],"playbooks/install-virtiofsd.yaml":[{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"3077e5843b49b7cdddc0342098244bb1dd71c35a","unresolved":true,"context_lines":[{"line_number":2,"context_line":"- hosts: all"},{"line_number":3,"context_line":"  become: true"},{"line_number":4,"context_line":"  tasks:"},{"line_number":5,"context_line":"    - name: Install virtiofsd and NFS client for virtiofs share support"},{"line_number":6,"context_line":"      ansible.builtin.package:"},{"line_number":7,"context_line":"        name:"},{"line_number":8,"context_line":"          - virtiofsd"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"6e62d788_59a2a0a0","line":5,"range":{"start_line":5,"start_character":12,"end_line":5,"end_character":71},"updated":"2026-08-14 18:12:07.000000000","message":"Conside playbooks/install-virtiofsd-and-nfs.yaml or alike to clarify","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"bd3237d1ea9f24a4286cf9d8e153638cb0bbcff3","unresolved":false,"context_lines":[{"line_number":2,"context_line":"- hosts: all"},{"line_number":3,"context_line":"  become: true"},{"line_number":4,"context_line":"  tasks:"},{"line_number":5,"context_line":"    - name: Install virtiofsd and NFS client for virtiofs share support"},{"line_number":6,"context_line":"      ansible.builtin.package:"},{"line_number":7,"context_line":"        name:"},{"line_number":8,"context_line":"          - virtiofsd"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"100e067f_b9e73923","line":5,"range":{"start_line":5,"start_character":12,"end_line":5,"end_character":71},"in_reply_to":"3b6a12f5_82749ed4","updated":"2026-08-27 00:24:07.000000000","message":"Done","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"dfe22e866fdc92b24ec38a8205a91bba1ca799cf","unresolved":true,"context_lines":[{"line_number":2,"context_line":"- hosts: all"},{"line_number":3,"context_line":"  become: true"},{"line_number":4,"context_line":"  tasks:"},{"line_number":5,"context_line":"    - name: Install virtiofsd and NFS client for virtiofs share support"},{"line_number":6,"context_line":"      ansible.builtin.package:"},{"line_number":7,"context_line":"        name:"},{"line_number":8,"context_line":"          - virtiofsd"}],"source_content_type":"text/x-yaml","patch_set":25,"id":"3b6a12f5_82749ed4","line":5,"range":{"start_line":5,"start_character":12,"end_line":5,"end_character":71},"in_reply_to":"6e62d788_59a2a0a0","updated":"2026-08-14 19:12:51.000000000","message":"ah! yeah; true.. will update the file name if i make another patch.","commit_id":"2e5e86ff8d41d0835c90eec58bfe024665678733"}]}