)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"c66d8b7802d2e1af8e0ec10a146693c5632a4b3f","unresolved":true,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"NOTE(auniyal): conflicts are due to the following changes that are not"},{"line_number":18,"context_line":"in Ussuri"},{"line_number":19,"context_line":"\t* Ie0852a89fc9423a92baa7c29a8806c0628cae220"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Related-Bug: #1628606"},{"line_number":22,"context_line":"Change-Id: I48dbe0aae8a3943fdde69cda1bd663d70ea0eb19"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"0f81a170_6b6b31eb","line":19,"updated":"2022-11-24 08:50:07.000000000","message":"this is no longer true, we backported the helper modification https://review.opendev.org/c/openstack/nova/+/746527\n\nPlease modify the commit msg.","commit_id":"b9669d7882db54a7512a18d6b0ee1b5a1ba8c584"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"ffa846d05d9e45e611eb5428cf0cbbcb055842ff","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a regression test or repoducer for post live migration"},{"line_number":10,"context_line":"fail at destination, the possible casue can be fail to get"},{"line_number":11,"context_line":"instance network info or  block device info"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Related-Bug: #1628606"},{"line_number":14,"context_line":"Change-Id: I48dbe0aae8a3943fdde69cda1bd663d70ea0eb19"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d9ec6b97_117a250a","line":11,"updated":"2022-11-28 08:49:37.000000000","message":"I don\u0027t see the paragraph explaining the stable-only change that was introduced in stable/zed and that was modifying _live_migrate() to return a server.\n\nPlease cherry-pick -x again from stable/wallaby so we are getting this.\nhttps://review.opendev.org/c/openstack/nova/+/863900","commit_id":"e3827806fd69b74d12a5c9bc592550bcbe433e01"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"04ad89ec69bb0b2c82be58f4bd710e103d0d9d3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"805a27e2_278da75e","updated":"2023-01-11 10:14:43.000000000","message":"looks good, the difference with the victoria version is noted.","commit_id":"5e955b62fa63b72816369a21af283a2b64f4af27"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1a57f80f2d11ddd074bc3da48a1c0e02b350c368","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d292f0b1_cbbd92f4","updated":"2022-12-08 09:07:10.000000000","message":"thanks for having worked on the commit msg.","commit_id":"5e955b62fa63b72816369a21af283a2b64f4af27"}],"nova/tests/functional/integrated_helpers.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"021a52d38b048abc3d85a85567407456d1c3fa09","unresolved":true,"context_lines":[{"line_number":427,"context_line":"        self._migrate_or_resize(server, resize_req)"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    def _live_migrate("},{"line_number":430,"context_line":"        self, server, migration_expected_state\u003d\u0027completed\u0027,"},{"line_number":431,"context_line":"        server_expected_state\u003d\u0027ACTIVE\u0027,"},{"line_number":432,"context_line":"    ):"},{"line_number":433,"context_line":"        self.api.post_server_action("}],"source_content_type":"text/x-python","patch_set":3,"id":"a2d3f1c0_e0309eac","line":430,"updated":"2022-11-28 09:04:53.000000000","message":"I don\u0027t see which commit added this default value. Can you either explain in the commit msg why you don\u0027t want to backport this other stable/victoria commit (and mention which commit sha1 introduced this default value) or please cherry-pick that commit to stable/ussuri as a dependent change.\n\nAs a reminder, we always prefer to cherry-pick dependent commits over directly touching the files, as the latter introduces a technical debt that we need to carry over for a long period in time. \nIn general, the only case where we accept to tamper a file after cherry-picking is when the dependency is way larger than the need and introduces more problems that the one we\u0027re trying to solve by fixing the merge conflict.","commit_id":"e3827806fd69b74d12a5c9bc592550bcbe433e01"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1a57f80f2d11ddd074bc3da48a1c0e02b350c368","unresolved":false,"context_lines":[{"line_number":427,"context_line":"        self._migrate_or_resize(server, resize_req)"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    def _live_migrate("},{"line_number":430,"context_line":"        self, server, migration_expected_state\u003d\u0027completed\u0027,"},{"line_number":431,"context_line":"        server_expected_state\u003d\u0027ACTIVE\u0027,"},{"line_number":432,"context_line":"    ):"},{"line_number":433,"context_line":"        self.api.post_server_action("}],"source_content_type":"text/x-python","patch_set":3,"id":"ee2e1433_ddc0f187","line":430,"in_reply_to":"a2d3f1c0_e0309eac","updated":"2022-12-08 09:07:10.000000000","message":"Done","commit_id":"e3827806fd69b74d12a5c9bc592550bcbe433e01"}]}
