)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"68b3b01aab323b4fdc388eba9c30a9ed7b81fa32","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Currently neutron can report ports to have MAC addresses"},{"line_number":10,"context_line":"in upper case when they\u0027re created like that. In the meanwhile"},{"line_number":11,"context_line":"libvirt configuration file always stores MAC in lower case"},{"line_number":12,"context_line":"which leads to KeyError while trying to retrieve migrate_vif."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1945646"},{"line_number":15,"context_line":"Change-Id: Ie3129ee395427337e9abcef2f938012608f643e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d85669df_e623156e","line":12,"updated":"2021-12-16 11:03:08.000000000","message":"actually there is a conflict when cherry-picking this patch from wallaby to victoria. In nova we follow the stable policy suggestion to include this information in the commit message ( \nhttps://docs.openstack.org/project-team-guide/stable-branches.html#conflicts )\nSo here we are missing here something like this:\n\n\nConflicts:\n  nova/tests/unit/virt/libvirt/test_migration.py\n\nNote: conflict is caused by not having six.text_type removal patch\nI779bd1446dc1f070fa5100ccccda7881fa508d79 in stable/victoria.","commit_id":"496b2ee73b833b8e19322fd7d8cca66f42fa9124"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"68b3b01aab323b4fdc388eba9c30a9ed7b81fa32","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1945646"},{"line_number":15,"context_line":"Change-Id: Ie3129ee395427337e9abcef2f938012608f643e1"},{"line_number":16,"context_line":"(cherry picked from commit 6a15169ed9f16672c2cde1d7f27178bb7868c41f)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"e86e0c25_fb73ccd1","line":16,"updated":"2021-12-16 11:03:08.000000000","message":"with \u0027git cherry-pick -x 6c3d5de659e558e8f6ee353475b54ff3ca7240ee\u0027 from stable/wallaby here we should see the correct cherry pick order:\n\n(cherry picked from commit 6a15169ed9f16672c2cde1d7f27178bb7868c41f)\n(cherry picked from commit 63a6388f6a0265f84232731aba8aec1bff3c6d18)\n(cherry picked from commit 6c3d5de659e558e8f6ee353475b54ff3ca7240ee)","commit_id":"496b2ee73b833b8e19322fd7d8cca66f42fa9124"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"948fd5276f83c9f0000c5a4b345b8843f0f6f31f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0f237df9_a93a4360","updated":"2021-11-12 20:06:40.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"ac86f4caaa4e1b477342b185bc427fe5403bc041","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8219b535_6b4391ab","updated":"2021-11-12 20:06:38.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"db7107915eceac96a13ef94858bbb8a9159f23e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"88a13039_83a7f916","updated":"2021-11-09 13:32:16.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"851cefc5375a47f04d065ae62f38f718c1bc3f7b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9701c2bb_3d817823","updated":"2021-11-09 13:29:29.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"35a5938fbf58b7545a72e6ff5fea3ae8e938bb75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"975d9510_a904ab25","updated":"2021-11-13 16:12:56.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"038a77062ee3b33ec4ff8c02b8198e680b5089c0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9f02260_df47018d","updated":"2021-11-09 01:13:30.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"c07be9d181bc38798b7d8f80fa41a843ab916dc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e451d7f1_4b94db9d","updated":"2021-11-10 21:58:02.000000000","message":"recheck","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"09daefbd01deba0526b19ef0250acadcb2de9db8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fe353946_45f5984c","updated":"2021-11-12 14:00:17.000000000","message":"recheck\n","commit_id":"b6058d54c2e10773e10c494ece85d3accccbba4e"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"68b3b01aab323b4fdc388eba9c30a9ed7b81fa32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f2f0fc06_64ccc682","updated":"2021-12-16 11:03:08.000000000","message":"Hi David, thanks for backporting this fix. Could you please re-apply the cherry pick from stable/wallaby and correcting the things I\u0027ve commented?","commit_id":"496b2ee73b833b8e19322fd7d8cca66f42fa9124"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"b591b8490d78c3c631c90c45a9ba3b4790773946","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a89652d1_46d7c6bc","updated":"2021-12-16 06:40:34.000000000","message":"recheck","commit_id":"496b2ee73b833b8e19322fd7d8cca66f42fa9124"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"2681fc58db222a0352955c8c6c55dbde1f7b21cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a2ef9688_a383f709","updated":"2021-12-16 16:02:58.000000000","message":"recheck","commit_id":"8fd7d0870de8db6cd4574a234e234c81af19d87c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"bd2060668d0f04e8093fc1eb4b36f79bcc751095","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"241a4a2a_9353f590","in_reply_to":"a2ef9688_a383f709","updated":"2021-12-16 17:28:23.000000000","message":"it won\u0027t help as there is a problem now with the backport o:)","commit_id":"8fd7d0870de8db6cd4574a234e234c81af19d87c"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"75ab39b14004c3455a86b033f6e43f382e2569a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a26f7dad_dbb95eb3","updated":"2021-12-16 19:53:40.000000000","message":"Thanks for the update Alexey! :)\nThe backport looks good to me: merged in stable/wallaby, conflict is noted and resolved correctly.","commit_id":"5b93e17d0527759a5fc2132de45d241849412615"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"448ac498ded2ce11952cee3771e8188596b4f5f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"bcad3571_70b05fcc","updated":"2021-12-23 14:04:35.000000000","message":"I have made a mistake and updated commit message. Change is reverted","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"0551989acad57bb79afce840f2ab8e4b59f9e26f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9db03c2a_241f792c","updated":"2021-12-23 18:25:12.000000000","message":"Same content that I\u0027ve +2\u0027d above, so adding it again","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"5cf39a257f1a0507678502a89d4cc9bc5a5d4ad8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ba35a1c1_3b40d2b4","updated":"2022-01-06 18:59:03.000000000","message":"Useful bug fix from stable/wallaby, conflict is resolved correctly and documented in the commit message, LGTM","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"8353b94821dc24fc015b592def7b336e020a6c64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"247c5c0e_8c1fdbc2","updated":"2022-01-10 14:45:46.000000000","message":"recheck","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"ad4355ef70aa3f6888aef563404fa2f14d532628","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4df0a8ef_ac126b70","updated":"2022-01-07 10:26:57.000000000","message":"recheck","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"edacdda1d3f0a8c90467efa23854bf8e312aa6a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"76082579_e3eb4035","updated":"2022-01-10 18:07:29.000000000","message":"recheck","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"0320399d6061c83df7e5024c327f60349fa7b130","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"dee04797_7c281aa7","updated":"2022-01-07 12:09:33.000000000","message":"recheck","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"7ed436c5507891925a097949f5a985daadeb6c7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fcc3e5e4_e87eff66","updated":"2022-01-07 01:33:31.000000000","message":"recheck","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"59d4fc8ae88cfbdd91a1911bccb712727096c302","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0bf2c6f2_1c9e231e","updated":"2022-01-10 14:45:33.000000000","message":"recheck\n\nDetails: {\u0027code\u0027: 400, \u0027message\u0027: \u0027Invalid volume: Volume status must be available or error or error_restoring or error_extending or error_managing and must not be migrating, attached, belong to a group, have snapshots or be disassociated from snapshots after volume transfer.\u0027}, \u003ctraceback object at 0x7f9e71331840\u003e), (\u003cclass \u0027tempest.lib.exceptions.TimeoutException\u0027\u003e, Request timed out\nDetails: (LiveAutoBlockMigrationV225Test:tearDownClass) Failed to delete volume 6061cf46-2e7e-44a7-beb7-aaf7d48405b8 within the required time (196 s). Timer started at 1641560347. Timer ended at 1641560543waited for 196, \u003ctraceback object at 0x7f9e7139dac0\u003e))","commit_id":"28d0059c1f52e51add31bff50f1f6e443c938792"}],"nova/tests/unit/virt/libvirt/test_migration.py":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"68b3b01aab323b4fdc388eba9c30a9ed7b81fa32","unresolved":true,"context_lines":[{"line_number":991,"context_line":"        doc \u003d etree.fromstring(original_xml)"},{"line_number":992,"context_line":"        ex \u003d self.assertRaises(KeyError, migration._update_vif_xml,"},{"line_number":993,"context_line":"                               doc, data, get_vif_config)"},{"line_number":994,"context_line":"        self.assertIn(\"ca:fe:de:ad:be:ef\", str(ex))"},{"line_number":995,"context_line":""},{"line_number":996,"context_line":"    def test_update_vif_xml_lower_case_mac(self):"},{"line_number":997,"context_line":"        \"\"\"Tests that the vif in the migrate data is not found in the existing"}],"source_content_type":"text/x-python","patch_set":3,"id":"1939e55f_be0e27dd","line":994,"range":{"start_line":994,"start_character":43,"end_line":994,"end_character":46},"updated":"2021-12-16 11:03:08.000000000","message":"since this patch will be backported to at least stable/train, let\u0027s just keep six.text_type here.","commit_id":"496b2ee73b833b8e19322fd7d8cca66f42fa9124"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"bd2060668d0f04e8093fc1eb4b36f79bcc751095","unresolved":true,"context_lines":[{"line_number":991,"context_line":"        doc \u003d etree.fromstring(original_xml)"},{"line_number":992,"context_line":"        ex \u003d self.assertRaises(KeyError, migration._update_vif_xml,"},{"line_number":993,"context_line":"                               doc, data, get_vif_config)"},{"line_number":994,"context_line":"        self.assertIn(\"CA:FE:DE:AD:BE:EF\", six.text_type(ex))"},{"line_number":995,"context_line":""},{"line_number":996,"context_line":"    def test_update_vif_xml_lower_case_mac(self):"},{"line_number":997,"context_line":"        \"\"\"Tests that the vif in the migrate data is not found in the existing"}],"source_content_type":"text/x-python","patch_set":5,"id":"7119ecb1_b1a28a94","line":994,"range":{"start_line":994,"start_character":23,"end_line":994,"end_character":40},"updated":"2021-12-16 17:28:23.000000000","message":"this should be lowercase (as in the original patch) to make the tests pass.","commit_id":"8fd7d0870de8db6cd4574a234e234c81af19d87c"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"cc27381c9859fdb151b993d2ac75fadcfa44f5a2","unresolved":true,"context_lines":[{"line_number":991,"context_line":"        doc \u003d etree.fromstring(original_xml)"},{"line_number":992,"context_line":"        ex \u003d self.assertRaises(KeyError, migration._update_vif_xml,"},{"line_number":993,"context_line":"                               doc, data, get_vif_config)"},{"line_number":994,"context_line":"        self.assertIn(\"CA:FE:DE:AD:BE:EF\", six.text_type(ex))"},{"line_number":995,"context_line":""},{"line_number":996,"context_line":"    def test_update_vif_xml_lower_case_mac(self):"},{"line_number":997,"context_line":"        \"\"\"Tests that the vif in the migrate data is not found in the existing"}],"source_content_type":"text/x-python","patch_set":5,"id":"14509f29_b76ff30e","line":994,"range":{"start_line":994,"start_character":23,"end_line":994,"end_character":40},"in_reply_to":"7119ecb1_b1a28a94","updated":"2021-12-16 18:02:04.000000000","message":"Haha :) thank you very much Elod!","commit_id":"8fd7d0870de8db6cd4574a234e234c81af19d87c"}]}