)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"50e8cd2b16f6daf866d2b522d97b5da47d87f646","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2966ff15_fe1a7343","updated":"2022-01-03 17:57:31.000000000","message":"Hold on: we should test that [1] is present. In master we didn\u0027t check (that was an error) although we can assume a newer OVN version. But not in stable releases.\n\n[1]https://github.com/ovn-org/ovn/commit/4d3cb42b076bb58fd8f01ab8ad146ffd539f2152","commit_id":"90502f5f50903581d16c293ba4bdb309ab7dca29"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"c5dea7065396da8dc39b857b23e147f4bb451b45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4bc6d4b7_f3ea4f7b","in_reply_to":"2966ff15_fe1a7343","updated":"2022-01-10 15:54:21.000000000","message":"Is this not checked with the match_fn functions? (with \"if hasattr(row, \u0027up\u0027)\") I thought it would work with both OVN versions - with and without that new column","commit_id":"90502f5f50903581d16c293ba4bdb309ab7dca29"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e159e9e0a26cdd87e5316e7d926ea09b13ab7183","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fcb9c82f_b8c5391d","updated":"2022-02-01 15:10:09.000000000","message":"Thanks for working on these backport.\n\nThis patch should contain, if I\u0027m not wrong, in this order:\n- https://review.opendev.org/c/openstack/neutron/+/821544: this patch\n- https://review.opendev.org/c/openstack/neutron/+/823818: missing one\n- https://review.opendev.org/c/openstack/neutron/+/823412: included\n- https://review.opendev.org/c/openstack/neutron/+/824378: missing one\n\nDo you agree with this? The goal is to push the feature with all the needed fixes.\n","commit_id":"8ab61e5b736b2b0acc06c1b0a11e580b7f15d499"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"8cf759943af922d2f934087414d65500077f04ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7f1c7256_d2dee507","updated":"2022-01-31 14:37:23.000000000","message":"recheck","commit_id":"8ab61e5b736b2b0acc06c1b0a11e580b7f15d499"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"36411ed30f0887b1ea7315c3b7ffaf31320a8cc0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c20f8e85_63f362b8","in_reply_to":"47937b3a_673058e5","updated":"2022-02-02 08:24:41.000000000","message":"Perfect!","commit_id":"8ab61e5b736b2b0acc06c1b0a11e580b7f15d499"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"74fb62528de3266eecd6b1737e27283464849455","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e5013d9b_b9948dfd","in_reply_to":"c20f8e85_63f362b8","updated":"2022-02-02 10:19:10.000000000","message":"Done","commit_id":"8ab61e5b736b2b0acc06c1b0a11e580b7f15d499"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"e6f362c1a1071e6bacdfc550df7201395c9b6191","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"47937b3a_673058e5","in_reply_to":"fcb9c82f_b8c5391d","updated":"2022-02-01 16:18:40.000000000","message":"I agree with you that https://review.opendev.org/c/openstack/neutron/+/823818 must be included, I forgot that one.\n\nI don\u0027t agree though about adding https://review.opendev.org/c/openstack/neutron/+/824378. This is independent on this patch, and was there before this initial patch. So, I\u0027m doing the backports of that one in a different patch: https://review.opendev.org/c/openstack/neutron/+/826843","commit_id":"8ab61e5b736b2b0acc06c1b0a11e580b7f15d499"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"14ae4dbd5abb29fe534213f488e4aa1c0643ab9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d9a80baf_bd4280d3","updated":"2022-02-14 16:59:24.000000000","message":"The change itself is localized, nice to have and compatible with these versions. The squashed changes look good and as commented previously we need them in with the main backport to avoid breakage, and they are detailed in commit message. So looks good to go","commit_id":"c25b92a6b7f9af82c6dff5eb80366edd71f4d06e"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"78ec0a07811d3beffc06959da22cda406199a25d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d1871e13_0f8cff42","updated":"2022-02-16 07:11:58.000000000","message":"recheck","commit_id":"6695f64f5f8e6e35758edac54bf675254d552e98"}]}