)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7b226a14df3f7ea56254d7e888457e9c6ff50512","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2023-01-02 14:50:46 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[OVN] Fix availability zones changes check"},{"line_number":8,"context_line":"Functional test for network with external ports update added"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Closes-Bug: #2000634"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"a3fbd336_ba7a27ad","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":17},"updated":"2023-01-02 14:09:45.000000000","message":"1) This is not needed. Testing is mandatory (in most of the cases) and we can see that in the files changed.\n2) Any comment apart from the title, should have an empty line in the middle. If not, this is considered as title too.","commit_id":"b7e729de01fa1f4afa5ef0c299c91397a12ca746"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6baed0d413460a3d1af089285a412f389431dc03","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"13baf8f0_bfc2e348","updated":"2023-01-02 08:53:33.000000000","message":"This change (and the original one that introduced this line) deserves a test. You can add it in the FT class \"TestExternalPorts\"","commit_id":"516461cae61c43475dc3811b4a09f901c9827fba"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"a9a07c8a8237342fa7284f530bdf2a11c721ceb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"447d30eb_131e0887","updated":"2023-01-02 14:03:08.000000000","message":"I\u0027ve added a functional test for mtu update in network with external ports. This choice is a bit arbitrary I must say, however it is doing its job, as we didn\u0027t have test coverage here before.","commit_id":"b7e729de01fa1f4afa5ef0c299c91397a12ca746"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7b226a14df3f7ea56254d7e888457e9c6ff50512","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6f8e60eb_52cf1d46","updated":"2023-01-02 14:09:45.000000000","message":"The patch is ok, please remove the second line in the commit message.","commit_id":"b7e729de01fa1f4afa5ef0c299c91397a12ca746"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"5e1da9284ba80700319a2dbe752b077007844bed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8f1a9c1d_7a7aaa15","updated":"2023-01-03 09:39:04.000000000","message":"We should backport it to xena/yoga/zed (introduced with https://review.opendev.org/c/openstack/neutron/+/762550) As one cannot update network with ovn external ports (vnic_type: direct, direct-physical, macvtap, baremetal)","commit_id":"5e19eaadbb298aaf58b5389fea74c9b8064d0f41"}]}