)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8e050d59e01a88dcd3add38cd007d6743308d372","unresolved":true,"context_lines":[{"line_number":16,"context_line":"of ovn-lb-hm:distributed to be different but will still receive the"},{"line_number":17,"context_line":"treatment of a localport in the OVN NB DB."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: 2038091"},{"line_number":20,"context_line":"Change-Id: I70de163fad34371de10fec28c51959384900ebc8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"5456afe1_3521e79f","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":6},"updated":"2023-10-16 11:33:17.000000000","message":"nit: related-bug (this patch doesn\u0027t solve it)","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"2f2b2e0369245ee57e8bae96bd8e70645206ae8a","unresolved":false,"context_lines":[{"line_number":16,"context_line":"of ovn-lb-hm:distributed to be different but will still receive the"},{"line_number":17,"context_line":"treatment of a localport in the OVN NB DB."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: 2038091"},{"line_number":20,"context_line":"Change-Id: I70de163fad34371de10fec28c51959384900ebc8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"65bde2b2_fec02ac1","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":6},"in_reply_to":"5456afe1_3521e79f","updated":"2023-10-16 11:35:15.000000000","message":"Done","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cacfd950586763043e409cd74d0d6e23a7551045","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8c2c3f31_639bc34b","updated":"2023-10-05 08:23:33.000000000","message":"Thx Brian for your quick review!","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"56947acf498886d7ff76310b9bea9b5c8549d219","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5eb21484_54ec582b","updated":"2023-10-10 05:29:46.000000000","message":"This may need a release note to explain ovn-octavia bits will take care of migrating the ports","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c8843dfa37bee942e57e4a8e2e6be1d0b8c58b72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d414b532_2697ff8e","updated":"2023-10-09 20:56:23.000000000","message":"Will existing ports stay with the old `distributed` `device_owner`? Should they be migrated somehow?","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"41712a884107b7151149849401ee43c51415433e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"59f6cd72_6ee9263d","updated":"2023-10-05 10:41:21.000000000","message":"good catch!","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"f91a4511224d0cdb789cd176ff77d002986c7dde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"89482348_4e78199c","updated":"2023-10-05 13:10:25.000000000","message":"recheck neutron-functional-with-uwsgi timeout","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"bd1d128b1a6169b8f4eef0228af504a6ead55ab4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"11835fd5_84c7d798","updated":"2023-10-09 09:30:29.000000000","message":"recheck neutron-functional-with-uwsgi unrelated","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"56947acf498886d7ff76310b9bea9b5c8549d219","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7b85eafc_5256c723","in_reply_to":"d414b532_2697ff8e","updated":"2023-10-10 05:29:46.000000000","message":"Fernando is working on a maintenance task on the ovn-octavia-provider task to migrate them, but I agree this is probably worth a release note to explain it","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"98bd496f7fa7a5736adc1c81fa05340b98c6151e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"28b7b566_05182fd0","updated":"2023-10-12 17:25:01.000000000","message":"Oh, one more consideration: does this mean that a particular Octavia version must be used with the newer Neutron? (And the old one would be broken?) Is it reflected in some supported version matrix for Octavia?","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3d8e68872abcaef056be48e451baa1722b233b3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8c4dd6cb_f1264f4f","updated":"2023-10-12 17:23:18.000000000","message":"releasenote need some love, otherwise I\u0027m ok","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"27857a6d48bd8998b2e2705583cd1c6fe18dc56b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b2e6eb78_a2561bae","in_reply_to":"28b7b566_05182fd0","updated":"2023-10-13 08:35:44.000000000","message":"Yeah, combinations are as follows:\n\nOVN-Octavia-provider NEW + neutron NEW (perfect scenario)\nOVN-Octavia-provider OLD + neutron OLD (it works, with the issue triggering neutron-db-sync)\nOVN-Octavia-provider OLD + neutron NEW (it works, with the issue triggering neutron-db-sync)\nOVN-Octavia-provider NEW + neutron OLD (it breaks! Neutron will not understand a port with ovn-lb-hm:distributed not giving a LOCALPORT behavior)\n\nI think the release note on patch [1] is the correct point, for those users deploying the new version of ovn-provider to be aware that they require a new version of Neutron.\n\n[1] https://review.opendev.org/c/openstack/ovn-octavia-provider/+/897348","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"34b4ed1434c10ea52e932eb940e4f530b3b61bd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"73512f64_bbd3714a","updated":"2023-10-16 11:21:56.000000000","message":"Do you have plans to backport this patch? This issue should be affecting previous version too.","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"29f266757b26500bec81668687164539da0a99a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f845849c_f7aa1c1f","updated":"2023-10-13 15:51:44.000000000","message":"It\u0027s fine. I think the fact that you must upgrade neutron before octavia may complicate matters for some deployment tools, but as long as 1) octavia team is ok with it and 2) it\u0027s captured in the upgrades section of the octavia provider release notes, it should be ok from neutron team perspective.","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"2f2b2e0369245ee57e8bae96bd8e70645206ae8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"402a425e_9f723e65","in_reply_to":"73512f64_bbd3714a","updated":"2023-10-16 11:35:15.000000000","message":"yeah, all stables branches to wallaby.","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"2f2b2e0369245ee57e8bae96bd8e70645206ae8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c31277f2_1d3c036a","in_reply_to":"770ec31b_e9d0a68b","updated":"2023-10-16 11:35:15.000000000","message":"Done","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"34b4ed1434c10ea52e932eb940e4f530b3b61bd0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"770ec31b_e9d0a68b","in_reply_to":"f845849c_f7aa1c1f","updated":"2023-10-16 11:21:56.000000000","message":"+1, that should be documented in Octavia.","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"}],"neutron/common/ovn/constants.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9241724540abd6746a2af52bdbb44f5336df7322","unresolved":true,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"# TODO(froyo): Move this to neutron-lib as soon as possible, and when a new"},{"line_number":74,"context_line":"# release is created and pointed to in the requirements remove this code"},{"line_number":75,"context_line":"OVN_LB_HM_PORT_DISTRIBUTED \u003d \u0027ovn-lb-hm:distributed\u0027"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"# Agent extension constants"},{"line_number":78,"context_line":"OVN_AGENT_DESC_KEY \u003d \u0027neutron:description\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"6e6c6bb2_a4c253d1","line":75,"updated":"2023-10-04 21:07:16.000000000","message":"Ok, this closely follows the LB_HM_PORT_PREFIX used in the ovn provider.","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cacfd950586763043e409cd74d0d6e23a7551045","unresolved":false,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"# TODO(froyo): Move this to neutron-lib as soon as possible, and when a new"},{"line_number":74,"context_line":"# release is created and pointed to in the requirements remove this code"},{"line_number":75,"context_line":"OVN_LB_HM_PORT_DISTRIBUTED \u003d \u0027ovn-lb-hm:distributed\u0027"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"# Agent extension constants"},{"line_number":78,"context_line":"OVN_AGENT_DESC_KEY \u003d \u0027neutron:description\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"eda7ad54_a8e54b38","line":75,"in_reply_to":"6e6c6bb2_a4c253d1","updated":"2023-10-05 08:23:33.000000000","message":"yeah, indeed related closely to this patch https://review.opendev.org/c/openstack/ovn-octavia-provider/+/897348","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e3e2212dfea2b8e8f740e55a58a36b5d1b0e8d0f","unresolved":true,"context_lines":[{"line_number":71,"context_line":"OVN_NAME_PREFIX \u003d \u0027neutron-\u0027"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"# TODO(froyo): Move this to neutron-lib as soon as possible, and when a new"},{"line_number":74,"context_line":"# release is created and pointed to in the requirements remove this code"},{"line_number":75,"context_line":"OVN_LB_HM_PORT_DISTRIBUTED \u003d \u0027ovn-lb-hm:distributed\u0027"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"# Agent extension constants"}],"source_content_type":"text/x-python","patch_set":3,"id":"fd2657f1_88bf6ba0","line":74,"range":{"start_line":74,"start_character":0,"end_line":74,"end_character":72},"updated":"2023-10-06 07:53:04.000000000","message":"+1","commit_id":"72798b19bfab92601c2a72fd94245364cf857dae"}],"neutron/common/ovn/utils.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"abb9be3f7a8d92d7b915ef43b3dda05944825c40","unresolved":true,"context_lines":[{"line_number":609,"context_line":""},{"line_number":610,"context_line":""},{"line_number":611,"context_line":"def is_ovn_lb_hm_port(port):"},{"line_number":612,"context_line":"    return (port[\u0027device_owner\u0027] \u003d\u003d constants.OVN_LB_HM_PORT_DISTRIBUTED)"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"def is_gateway_chassis_invalid(chassis_name, gw_chassis,"}],"source_content_type":"text/x-python","patch_set":2,"id":"a1278a50_c5e731de","line":612,"range":{"start_line":612,"start_character":11,"end_line":612,"end_character":73},"updated":"2023-10-05 08:27:44.000000000","message":"should we also add device_id.startwith(ovn-lb-hm) or similar?","commit_id":"3900e8c0b79ff364b5af423897d159486d887433"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"41712a884107b7151149849401ee43c51415433e","unresolved":false,"context_lines":[{"line_number":609,"context_line":""},{"line_number":610,"context_line":""},{"line_number":611,"context_line":"def is_ovn_lb_hm_port(port):"},{"line_number":612,"context_line":"    return (port[\u0027device_owner\u0027] \u003d\u003d constants.OVN_LB_HM_PORT_DISTRIBUTED)"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"def is_gateway_chassis_invalid(chassis_name, gw_chassis,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7bb402e6_34342db8","line":612,"range":{"start_line":612,"start_character":11,"end_line":612,"end_character":73},"in_reply_to":"a1278a50_c5e731de","updated":"2023-10-05 10:41:21.000000000","message":"Done","commit_id":"3900e8c0b79ff364b5af423897d159486d887433"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"abb9be3f7a8d92d7b915ef43b3dda05944825c40","unresolved":true,"context_lines":[{"line_number":381,"context_line":""},{"line_number":382,"context_line":"            # Metadata or OVN LB HM port."},{"line_number":383,"context_line":"            if (utils.is_ovn_metadata_port(port) or"},{"line_number":384,"context_line":"                    utils.is_ovn_metadata_port(port)):"},{"line_number":385,"context_line":"                port_type \u003d ovn_const.LSP_TYPE_LOCALPORT"},{"line_number":386,"context_line":""},{"line_number":387,"context_line":"            if utils.is_port_external(port):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a988961e_7b4ed206","line":384,"updated":"2023-10-05 08:27:44.000000000","message":"typo here, this should be is_ovn_lb_hm_port, right?","commit_id":"3900e8c0b79ff364b5af423897d159486d887433"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"41712a884107b7151149849401ee43c51415433e","unresolved":false,"context_lines":[{"line_number":381,"context_line":""},{"line_number":382,"context_line":"            # Metadata or OVN LB HM port."},{"line_number":383,"context_line":"            if (utils.is_ovn_metadata_port(port) or"},{"line_number":384,"context_line":"                    utils.is_ovn_metadata_port(port)):"},{"line_number":385,"context_line":"                port_type \u003d ovn_const.LSP_TYPE_LOCALPORT"},{"line_number":386,"context_line":""},{"line_number":387,"context_line":"            if utils.is_port_external(port):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1af58ae7_ce983e67","line":384,"in_reply_to":"a988961e_7b4ed206","updated":"2023-10-05 10:41:21.000000000","message":"Done","commit_id":"3900e8c0b79ff364b5af423897d159486d887433"}],"neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_impl_idl.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9241724540abd6746a2af52bdbb44f5336df7322","unresolved":true,"context_lines":[{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import uuid"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from neutron_lib import constants"},{"line_number":19,"context_line":"from oslo_utils import uuidutils"},{"line_number":20,"context_line":"from ovsdbapp.backend.ovs_idl import connection"},{"line_number":21,"context_line":"from ovsdbapp import constants as const"}],"source_content_type":"text/x-python","patch_set":1,"id":"bcc97dbd_0bf454f4","line":18,"updated":"2023-10-04 21:07:16.000000000","message":"not necessary, see below","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cacfd950586763043e409cd74d0d6e23a7551045","unresolved":true,"context_lines":[{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import uuid"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from neutron_lib import constants"},{"line_number":19,"context_line":"from oslo_utils import uuidutils"},{"line_number":20,"context_line":"from ovsdbapp.backend.ovs_idl import connection"},{"line_number":21,"context_line":"from ovsdbapp import constants as const"}],"source_content_type":"text/x-python","patch_set":1,"id":"dc69ec4d_d172e1a0","line":18,"in_reply_to":"bcc97dbd_0bf454f4","updated":"2023-10-05 08:23:33.000000000","message":"IINW, still needed by L117.","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9241724540abd6746a2af52bdbb44f5336df7322","unresolved":true,"context_lines":[{"line_number":128,"context_line":"        result \u003d self.api.get_metadata_port_network(str(binding.datapath.uuid))"},{"line_number":129,"context_line":"        self.assertEqual(binding, result)"},{"line_number":130,"context_line":"        self.assertEqual(binding.datapath.external_ids[\u0027logical-switch\u0027],"},{"line_number":131,"context_line":"                         str(switch.uuid))"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def test_get_metadata_port_network_other_non_metadata_port(self):"},{"line_number":134,"context_line":"        chassis, switch, port, binding \u003d self._add_switch_port("}],"source_content_type":"text/x-python","patch_set":1,"id":"a31e065e_6216d961","line":131,"updated":"2023-10-04 21:07:16.000000000","message":"nit: could do a similar check as L143 below here.","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cacfd950586763043e409cd74d0d6e23a7551045","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        result \u003d self.api.get_metadata_port_network(str(binding.datapath.uuid))"},{"line_number":129,"context_line":"        self.assertEqual(binding, result)"},{"line_number":130,"context_line":"        self.assertEqual(binding.datapath.external_ids[\u0027logical-switch\u0027],"},{"line_number":131,"context_line":"                         str(switch.uuid))"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def test_get_metadata_port_network_other_non_metadata_port(self):"},{"line_number":134,"context_line":"        chassis, switch, port, binding \u003d self._add_switch_port("}],"source_content_type":"text/x-python","patch_set":1,"id":"870a90ea_109e52dc","line":131,"in_reply_to":"a31e065e_6216d961","updated":"2023-10-05 08:23:33.000000000","message":"Done","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9241724540abd6746a2af52bdbb44f5336df7322","unresolved":true,"context_lines":[{"line_number":135,"context_line":"            self.data[\u0027chassis\u0027][0][\u0027name\u0027])"},{"line_number":136,"context_line":"        chassis, switch, port_lb_hm, binding \u003d self._add_switch_port("},{"line_number":137,"context_line":"            self.data[\u0027chassis\u0027][0][\u0027name\u0027],"},{"line_number":138,"context_line":"            device_owner\u003dconstants.OVN_LB_HM_PORT_DISTRIBUTED)"},{"line_number":139,"context_line":"        result \u003d self.api.get_metadata_port_network(str(binding.datapath.uuid))"},{"line_number":140,"context_line":"        self.assertEqual(binding, result)"},{"line_number":141,"context_line":"        self.assertEqual(binding.datapath.external_ids[\u0027logical-switch\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"022f2c7c_dc1b45d3","line":138,"range":{"start_line":138,"start_character":25,"end_line":138,"end_character":34},"updated":"2023-10-04 21:07:16.000000000","message":"s/ovn_const","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cacfd950586763043e409cd74d0d6e23a7551045","unresolved":false,"context_lines":[{"line_number":135,"context_line":"            self.data[\u0027chassis\u0027][0][\u0027name\u0027])"},{"line_number":136,"context_line":"        chassis, switch, port_lb_hm, binding \u003d self._add_switch_port("},{"line_number":137,"context_line":"            self.data[\u0027chassis\u0027][0][\u0027name\u0027],"},{"line_number":138,"context_line":"            device_owner\u003dconstants.OVN_LB_HM_PORT_DISTRIBUTED)"},{"line_number":139,"context_line":"        result \u003d self.api.get_metadata_port_network(str(binding.datapath.uuid))"},{"line_number":140,"context_line":"        self.assertEqual(binding, result)"},{"line_number":141,"context_line":"        self.assertEqual(binding.datapath.external_ids[\u0027logical-switch\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"d2dafc28_80ca3fa5","line":138,"range":{"start_line":138,"start_character":25,"end_line":138,"end_character":34},"in_reply_to":"022f2c7c_dc1b45d3","updated":"2023-10-05 08:23:33.000000000","message":"Done","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9241724540abd6746a2af52bdbb44f5336df7322","unresolved":true,"context_lines":[{"line_number":142,"context_line":"                         str(switch.uuid))"},{"line_number":143,"context_line":"        self.assertEqual("},{"line_number":144,"context_line":"            port_lb_hm.external_ids[ovn_const.OVN_DEVICE_OWNER_EXT_ID_KEY],"},{"line_number":145,"context_line":"            constants.OVN_LB_HM_PORT_DISTRIBUTED)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    def test_get_metadata_port_network_missing(self):"},{"line_number":148,"context_line":"        val \u003d str(uuid.uuid4())"}],"source_content_type":"text/x-python","patch_set":1,"id":"4b8fdee1_ae3763d6","line":145,"range":{"start_line":145,"start_character":12,"end_line":145,"end_character":21},"updated":"2023-10-04 21:07:16.000000000","message":"s/ovn_const","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"cacfd950586763043e409cd74d0d6e23a7551045","unresolved":false,"context_lines":[{"line_number":142,"context_line":"                         str(switch.uuid))"},{"line_number":143,"context_line":"        self.assertEqual("},{"line_number":144,"context_line":"            port_lb_hm.external_ids[ovn_const.OVN_DEVICE_OWNER_EXT_ID_KEY],"},{"line_number":145,"context_line":"            constants.OVN_LB_HM_PORT_DISTRIBUTED)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    def test_get_metadata_port_network_missing(self):"},{"line_number":148,"context_line":"        val \u003d str(uuid.uuid4())"}],"source_content_type":"text/x-python","patch_set":1,"id":"82514642_3262b512","line":145,"range":{"start_line":145,"start_character":12,"end_line":145,"end_character":21},"in_reply_to":"4b8fdee1_ae3763d6","updated":"2023-10-05 08:23:33.000000000","message":"Done","commit_id":"d0544bed7e1aa55a759a7ddb596b0876ede75964"}],"releasenotes/notes/new-device-owner-for-ovn-lb-hm-ports-f5a648c4d948c5c8.yaml":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3d8e68872abcaef056be48e451baa1722b233b3b","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"86b3c8b3_fca052ab","line":4,"range":{"start_line":4,"start_character":41,"end_line":4,"end_character":44},"updated":"2023-10-12 17:23:18.000000000","message":"OVN is all caps","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"27857a6d48bd8998b2e2705583cd1c6fe18dc56b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"94bde18b_639b0ee5","line":4,"range":{"start_line":4,"start_character":41,"end_line":4,"end_character":44},"in_reply_to":"86b3c8b3_fca052ab","updated":"2023-10-13 08:35:44.000000000","message":"Done","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3d8e68872abcaef056be48e451baa1722b233b3b","unresolved":true,"context_lines":[{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"b9c27681_ebaf2f13","line":5,"range":{"start_line":5,"start_character":15,"end_line":5,"end_character":26},"updated":"2023-10-12 17:23:18.000000000","message":"typo","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"27857a6d48bd8998b2e2705583cd1c6fe18dc56b","unresolved":false,"context_lines":[{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3e08e9d8_1196d4ae","line":5,"range":{"start_line":5,"start_character":15,"end_line":5,"end_character":26},"in_reply_to":"b9c27681_ebaf2f13","updated":"2023-10-13 08:35:44.000000000","message":"Done","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3d8e68872abcaef056be48e451baa1722b233b3b","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"},{"line_number":9,"context_line":"    when this patch will be deployed."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"b4f51b49_bd92f8e0","line":6,"range":{"start_line":6,"start_character":34,"end_line":6,"end_character":48},"updated":"2023-10-12 17:23:18.000000000","message":"responsIbility","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3d8e68872abcaef056be48e451baa1722b233b3b","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"},{"line_number":9,"context_line":"    when this patch will be deployed."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e7712c9d_d04e425a","line":6,"range":{"start_line":6,"start_character":59,"end_line":6,"end_character":64},"updated":"2023-10-12 17:23:18.000000000","message":"the / these","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"27857a6d48bd8998b2e2705583cd1c6fe18dc56b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"},{"line_number":9,"context_line":"    when this patch will be deployed."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df92e99a_c3c044cc","line":6,"range":{"start_line":6,"start_character":34,"end_line":6,"end_character":48},"in_reply_to":"b4f51b49_bd92f8e0","updated":"2023-10-13 08:35:44.000000000","message":"Done","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"27857a6d48bd8998b2e2705583cd1c6fe18dc56b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The new value for \u0027device_owner\u0027 for ovn loadbalancer health monitor ports"},{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"},{"line_number":9,"context_line":"    when this patch will be deployed."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"4cb004b7_4b776633","line":6,"range":{"start_line":6,"start_character":59,"end_line":6,"end_character":64},"in_reply_to":"e7712c9d_d04e425a","updated":"2023-10-13 08:35:44.000000000","message":"Done","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3d8e68872abcaef056be48e451baa1722b233b3b","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"},{"line_number":9,"context_line":"    when this patch will be deployed."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5ddc1ca1_56bae0ff","line":9,"range":{"start_line":8,"start_character":36,"end_line":9,"end_character":37},"updated":"2023-10-12 17:23:18.000000000","message":"Suggest: Octavia will take care of database conversion for the ports.\n\n(Why? Because the notion of a \"patch\" doesn\u0027t fit release notes audience. Plus grammar is not ideal.)","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"27857a6d48bd8998b2e2705583cd1c6fe18dc56b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    (ovn-lb-hm:dsitributed) is now supported by Neutron, giving a behaviour to"},{"line_number":6,"context_line":"    those ports as LOCALPORT. The responsability to define those ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    ovn-octavia-provider component. Also the update of the any existing port"},{"line_number":9,"context_line":"    when this patch will be deployed."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5c2c779b_d025dfe4","line":9,"range":{"start_line":8,"start_character":36,"end_line":9,"end_character":37},"in_reply_to":"5ddc1ca1_56bae0ff","updated":"2023-10-13 08:35:44.000000000","message":"ack, a release note reader may have no idea about the patch... make sense!thx","commit_id":"070db990bde578ca5298590d858c56601c4727d2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"34b4ed1434c10ea52e932eb940e4f530b3b61bd0","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    (ovn-lb-hm:distributed) is now supported by Neutron, providing a LOCALPORT"},{"line_number":6,"context_line":"    behavior to these ports. The responsibility to define these ports with the"},{"line_number":7,"context_line":"    new value instead of the old one (network:distributed) is under the"},{"line_number":8,"context_line":"    OVN-Octavia Provider driver, which will take care of database conversion"},{"line_number":9,"context_line":"    for these ports."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ba14464a_4a4bd4a8","line":9,"range":{"start_line":8,"start_character":33,"end_line":9,"end_character":19},"updated":"2023-10-16 11:21:56.000000000","message":"So Octavia will handle the port device owner migration, nice to know that.","commit_id":"dc7972bc7267cf15c276040f3dc3c4b0721fd9d0"}]}