)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"75111f384bccd4b40d6b5b97e676ae7cff36ca55","unresolved":true,"context_lines":[{"line_number":11,"context_line":"status."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a81a741c_c5cd00bb","line":14,"range":{"start_line":14,"start_character":2,"end_line":14,"end_character":32},"updated":"2022-08-22 06:54:50.000000000","message":"can you please point where a passed test could be seen?","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"c57208a7dd46940df5bb40f8b103a1fd0995ab05","unresolved":true,"context_lines":[{"line_number":11,"context_line":"status."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9cea86de_d302d69b","line":14,"range":{"start_line":14,"start_character":2,"end_line":14,"end_character":32},"in_reply_to":"47b51243_66d964d4","updated":"2022-08-22 10:44:57.000000000","message":"So I guess a Depends-On patch on tempest (unskip) should be good to have","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"82e84ce486e561fdb4969169a750973e9cf92d35","unresolved":false,"context_lines":[{"line_number":11,"context_line":"status."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"246c81df_a5c6e834","line":14,"range":{"start_line":14,"start_character":2,"end_line":14,"end_character":32},"in_reply_to":"9cea86de_d302d69b","updated":"2022-09-02 10:51:06.000000000","message":"I wasn\u0027t able to run a tempest test on zuul. So I removed this line since I wasn\u0027t fixing the test, I was fixing an error spotted by a test","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"a0287a2433e4b31e3b77b571086611599c242da4","unresolved":true,"context_lines":[{"line_number":11,"context_line":"status."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"47b51243_66d964d4","line":14,"range":{"start_line":14,"start_character":2,"end_line":14,"end_character":32},"in_reply_to":"a81a741c_c5cd00bb","updated":"2022-08-22 09:53:55.000000000","message":"I think that you won\u0027t be able to see it, this test on u/s is skipped:\nhttps://6779a19d629122a2ad74-09e4be48fe62aca6e4b03d954e19defe.ssl.cf1.rackcdn.com/853203/3/check/neutron-tempest-plugin-scenario-ovn-wallaby/99aaf5b/testr_results.html (search for the string: test_trunk_vm_migration)","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"75111f384bccd4b40d6b5b97e676ae7cff36ca55","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5e99a28f_c2b9654c","line":15,"updated":"2022-08-22 06:54:50.000000000","message":"Is there a bug to be referenced here?","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"82e84ce486e561fdb4969169a750973e9cf92d35","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d03ca72c_37f2e0ec","line":15,"in_reply_to":"2c937ed2_6570f4fd","updated":"2022-09-02 10:51:06.000000000","message":"Done, I recreated by hand and filled a LP.\nhttps://bugs.launchpad.net/neutron/+bug/1988549","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"a0287a2433e4b31e3b77b571086611599c242da4","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fcabac5d_d75a4ff4","line":15,"in_reply_to":"5e99a28f_c2b9654c","updated":"2022-08-22 09:53:55.000000000","message":"Since it is not executed I didn\u0027t open a LP","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"c57208a7dd46940df5bb40f8b103a1fd0995ab05","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This fixes tempest test:"},{"line_number":14,"context_line":"- test_live_migration_with_trunk while using ML2/OVN"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia0f7a6e8510af2c3545993e0d0d4bb06a9b70b79"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"2c937ed2_6570f4fd","line":15,"in_reply_to":"fcabac5d_d75a4ff4","updated":"2022-08-22 10:44:57.000000000","message":"The bug should be filed for the issue fixed by this patch (not for the test) - this is needed in order to track the fix across stable branches","commit_id":"fb0d5729e82eb58a17668e6ad9195e217db16a17"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"69d90f44729c402aeb1ed75e54b1e4a5e8dd1d78","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[Trunk] Update the trunk status with the parent status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After a trunk VM has been migrated the trunk status remains"},{"line_number":10,"context_line":"DOWN, After the parent port is back to active modify the trun"},{"line_number":11,"context_line":"status."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1988549"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f8d78119_9bb7a049","line":10,"range":{"start_line":10,"start_character":60,"end_line":10,"end_character":61},"updated":"2022-09-09 15:18:08.000000000","message":"nit: trunk","commit_id":"fdea7f223e390e872f0c2dc4736240ab03494a76"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"ec457730cd3aaeba4b48aef6e612c601f9c3f868","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Trunk] Update the trunk status with the parent status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After a trunk VM has been migrated the trunk status remains"},{"line_number":10,"context_line":"DOWN, After the parent port is back to active modify the trun"},{"line_number":11,"context_line":"status."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1988549"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"66226012_a534fc71","line":10,"range":{"start_line":10,"start_character":60,"end_line":10,"end_character":61},"in_reply_to":"f8d78119_9bb7a049","updated":"2022-09-20 16:03:13.000000000","message":"Done","commit_id":"fdea7f223e390e872f0c2dc4736240ab03494a76"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"71fd4cddbfb3f623aa3ba8008a2d2fd1956be701","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e4208cc7_9a080463","updated":"2022-08-20 01:43:01.000000000","message":"LGTM","commit_id":"9d82156d8cf9e697db3c2b22c3711f0be9f59ff9"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"4ca260315746681128c2e9923dc70fe827934140","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bb4108af_eafe2db2","updated":"2022-09-12 06:34:36.000000000","message":"There has been several issues with trunk/parent-port/sub-ports status updates, perhaps worth to add a functional test covering this new case to ensure it works as expected","commit_id":"fdea7f223e390e872f0c2dc4736240ab03494a76"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"ec457730cd3aaeba4b48aef6e612c601f9c3f868","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7647222a_5d322156","updated":"2022-09-20 16:03:13.000000000","message":"Thanks all 😊","commit_id":"ab527bfd474438838cedc422b5ee9fb6bf0c62ed"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"4bf4ab28598cdfbf8259f3fe9b3c3fe7bc1204a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a183a367_8dae7e41","updated":"2022-10-05 10:04:36.000000000","message":"Recheck - neutron.tests.functional.agent.l3.extensions.test_ndp_proxy_extension.TestL3AgentNDPProxyExtensionDVR.test_direct_route_for_address_scope failed, not related to this change","commit_id":"178ee6fd3d76802cd7f577ad3d0d190117e78962"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"3a3b8cf606ffcccbbe473a9a0fee007f86ac54e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"2f18e935_95d5b10b","updated":"2022-10-27 08:49:13.000000000","message":"recheck neutron-tempest-plugin-ovn latest builds is green","commit_id":"178ee6fd3d76802cd7f577ad3d0d190117e78962"}],"neutron/services/trunk/plugin.py":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"71fd4cddbfb3f623aa3ba8008a2d2fd1956be701","unresolved":true,"context_lines":[{"line_number":462,"context_line":""},{"line_number":463,"context_line":"        context \u003d payload.context"},{"line_number":464,"context_line":"        original_port \u003d payload.states[0]"},{"line_number":465,"context_line":"        orig_vif_type \u003d original_port.get(portbindings.VIF_TYPE)"},{"line_number":466,"context_line":"        new_vif_type \u003d updated_port.get(portbindings.VIF_TYPE)"},{"line_number":467,"context_line":"        original_status \u003d original_port.get(\u0027status\u0027)"},{"line_number":468,"context_line":"        updated_status \u003d updated_port.get(\u0027status\u0027)"},{"line_number":469,"context_line":"        vif_type_changed \u003d orig_vif_type !\u003d new_vif_type"},{"line_number":470,"context_line":"        trunk_id \u003d trunk_details[\u0027trunk_id\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1469b9a3_982aee86","line":467,"range":{"start_line":465,"start_character":0,"end_line":467,"end_character":53},"updated":"2022-08-20 01:43:01.000000000","message":"tiny nit: keep `original_*` variables grouped together. Dont need to push new change just for this, only if you are amending for some other reason(s) :)","commit_id":"9d82156d8cf9e697db3c2b22c3711f0be9f59ff9"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"ec457730cd3aaeba4b48aef6e612c601f9c3f868","unresolved":false,"context_lines":[{"line_number":462,"context_line":""},{"line_number":463,"context_line":"        context \u003d payload.context"},{"line_number":464,"context_line":"        original_port \u003d payload.states[0]"},{"line_number":465,"context_line":"        orig_vif_type \u003d original_port.get(portbindings.VIF_TYPE)"},{"line_number":466,"context_line":"        new_vif_type \u003d updated_port.get(portbindings.VIF_TYPE)"},{"line_number":467,"context_line":"        original_status \u003d original_port.get(\u0027status\u0027)"},{"line_number":468,"context_line":"        updated_status \u003d updated_port.get(\u0027status\u0027)"},{"line_number":469,"context_line":"        vif_type_changed \u003d orig_vif_type !\u003d new_vif_type"},{"line_number":470,"context_line":"        trunk_id \u003d trunk_details[\u0027trunk_id\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"b88fb83b_f22887e2","line":467,"range":{"start_line":465,"start_character":0,"end_line":467,"end_character":53},"in_reply_to":"1469b9a3_982aee86","updated":"2022-09-20 16:03:13.000000000","message":"Done","commit_id":"9d82156d8cf9e697db3c2b22c3711f0be9f59ff9"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"55bcef9fb6cc6351bcd2a9ac2961682d08c19582","unresolved":true,"context_lines":[{"line_number":479,"context_line":"                new_status !\u003d orig_status:"},{"line_number":480,"context_line":"            self.update_trunk("},{"line_number":481,"context_line":"                context, trunk_id,"},{"line_number":482,"context_line":"                {\u0027trunk\u0027: {\u0027status\u0027: constants.TRUNK_ACTIVE_STATUS}})"}],"source_content_type":"text/x-python","patch_set":4,"id":"f928932d_dd977637","line":482,"updated":"2022-09-12 14:58:31.000000000","message":"Maybe You can add some unit tests to cover that?","commit_id":"fdea7f223e390e872f0c2dc4736240ab03494a76"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"ec457730cd3aaeba4b48aef6e612c601f9c3f868","unresolved":false,"context_lines":[{"line_number":479,"context_line":"                new_status !\u003d orig_status:"},{"line_number":480,"context_line":"            self.update_trunk("},{"line_number":481,"context_line":"                context, trunk_id,"},{"line_number":482,"context_line":"                {\u0027trunk\u0027: {\u0027status\u0027: constants.TRUNK_ACTIVE_STATUS}})"}],"source_content_type":"text/x-python","patch_set":4,"id":"830c8abb_0ff105e1","line":482,"in_reply_to":"f928932d_dd977637","updated":"2022-09-20 16:03:13.000000000","message":"Done","commit_id":"fdea7f223e390e872f0c2dc4736240ab03494a76"}]}