)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"2da001f8b5440fbdb40b7ae7fbbc84d555cdd4db","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     LIU Yulong \u003ci@liuyulong.me\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-04-06 09:32:27 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Pass physical bridge informations to OVS agent extension API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The metadata agent extension needs the patch ports informations"},{"line_number":10,"context_line":"between br-int and br-meta to add direct flows."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ad3a5de0_1e92070d","line":7,"updated":"2023-05-03 09:43:07.000000000","message":"bridge mapping and ofports to ovs agent extention API\n\nYou should also mention the function introduced","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"1e3c6adcbcd4e12a6c1c1fc9adb07ddf105ad7c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"180f32b8_ec44a889","updated":"2022-12-07 08:16:25.000000000","message":"Hey. I think the commit message need a bit more details Also I would recommend to add asserting new attributes to avoid any typo that may not have been noticed.","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"2efdea8a6233069d0e5117b606b70d23ff370615","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dd1832bb_7e027c61","updated":"2022-12-07 07:08:26.000000000","message":"recheck","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c99777cbe79be87fd38fd438736416c60d2170a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4e1aeab4_d4fc003a","updated":"2022-12-14 06:32:47.000000000","message":"recheck","commit_id":"009577a7f27014d4e5dedd43892007ef9968172a"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"2143749c0c1402e2119c0696401a8dd1409b3127","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"40c305f2_b3e1275f","updated":"2022-12-20 07:20:28.000000000","message":"recheck","commit_id":"a7fdf7665b8951283a8e5758729fc9382bcd709f"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"432297cea7be4a5b05f4093673fc8eba822f7422","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6e9d52b4_e200b50f","updated":"2022-12-16 07:42:46.000000000","message":"recheck","commit_id":"a7fdf7665b8951283a8e5758729fc9382bcd709f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ebfd2fa3b671ace59bcebdfca1497c05fd4b82ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bc4dda8d_fa83c73c","updated":"2023-03-01 09:39:56.000000000","message":"-1 for visibility","commit_id":"11363a40cb81d87cd111a5eea36b6eae45761477"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"782dfcc6fe9e9cb4f4d0b9d21f68bac327d65200","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cfd09eed_95493f7e","updated":"2023-02-14 09:56:07.000000000","message":"recheck","commit_id":"11363a40cb81d87cd111a5eea36b6eae45761477"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"2da001f8b5440fbdb40b7ae7fbbc84d555cdd4db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c4b2fcf4_57dcf9e1","updated":"2023-05-03 09:43:07.000000000","message":"Thanks","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"04b9fd0ea41e07adcead5f21684d7a36fd504c2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"86b19c96_a1529a37","updated":"2023-05-01 22:02:31.000000000","message":"The attributes that this patch adds to the ovs agent extension API object are used in https://review.opendev.org/c/openstack/neutron/+/881535. So I am good with this change","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"bd8b53ffcb8a008bcdf2839943d701a4c0866fe9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"69e87d05_d88eba36","updated":"2023-04-25 07:38:34.000000000","message":"There are some interrogations pending","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"7a7d4d8de969b1268450fb10678978f6b10ec603","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5fcc2bf5_9a79bee8","updated":"2023-05-08 03:27:48.000000000","message":"recheck","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"8a7176e941be4c2940fb7edd674192a1911c3b05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"911c659c_5c6d7eba","updated":"2023-05-08 00:32:50.000000000","message":"recheck","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"f8d83870b9281065246a5417c8c41696c081ad69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"34fe87b7_4a36067f","in_reply_to":"69e87d05_d88eba36","updated":"2023-04-26 04:05:48.000000000","message":"Here we go:\nhttps://review.opendev.org/c/openstack/neutron/+/881535","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_agent_extension_api.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"1e3c6adcbcd4e12a6c1c1fc9adb07ddf105ad7c3","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"30a5254b_dfff89ff","line":45,"updated":"2022-12-07 08:16:25.000000000","message":"You have added phys_ofports and bridge_mapping as new public attribute I don\u0027t see where this is used At least that should have been mentioned. Also tests to assert their usage.","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"f8d83870b9281065246a5417c8c41696c081ad69","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"407fa6ef_416a217e","line":45,"in_reply_to":"001ed9e8_53b83a31","updated":"2023-04-26 04:05:48.000000000","message":"Here we go:\nhttps://review.opendev.org/c/openstack/neutron/+/881535","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"af8ee7b15d42c0bdd28cd5697ba59f6c84713501","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"6c849452_3509e4a9","line":45,"in_reply_to":"0b0e0f87_40727453","updated":"2023-03-09 00:51:53.000000000","message":"Sorry for the late reply, this will be used in the new ovs agent extension.","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ebfd2fa3b671ace59bcebdfca1497c05fd4b82ce","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"0b0e0f87_40727453","line":45,"in_reply_to":"30a5254b_dfff89ff","updated":"2023-03-01 09:39:56.000000000","message":"+1 to this question","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"38bc5e0b5d98687fbfd818e269205c6c6f840717","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"450fad3c_d600174e","line":45,"in_reply_to":"407fa6ef_416a217e","updated":"2023-04-28 07:18:33.000000000","message":"thanks this one was missing from the chain :-)","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"cdde07a1ee63e1c9ed99ce0dd4e8fb169eaa35ec","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"a5b738c5_db42565b","line":45,"in_reply_to":"450fad3c_d600174e","updated":"2023-05-03 09:27:20.000000000","message":"Thank you LIU!","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"ca575fe484446ede7d8974015944481fb508c485","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"b4dc007e_5ee48858","line":45,"in_reply_to":"6c849452_3509e4a9","updated":"2023-04-05 10:09:58.000000000","message":"Hum I don\u0027t understand yet the whole implementation. Do you have all the patches related available on topic distributed_metadata_data_path?","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"d1a2e227db613e8daaa53aaa4811912261142ebd","unresolved":true,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def __init__(self, int_br, tun_br, phys_brs\u003dNone,"},{"line_number":44,"context_line":"                 plugin_rpc\u003dNone,"},{"line_number":45,"context_line":"                 phys_ofports\u003dNone,"},{"line_number":46,"context_line":"                 bridge_mappings\u003dNone):"},{"line_number":47,"context_line":"        super(OVSAgentExtensionAPI, self).__init__()"},{"line_number":48,"context_line":"        self.br_int \u003d int_br"}],"source_content_type":"text/x-python","patch_set":1,"id":"001ed9e8_53b83a31","line":45,"in_reply_to":"b4dc007e_5ee48858","updated":"2023-04-14 14:41:29.000000000","message":"after checking the latest ps agree where it is used? I can\u0027t find it in https://review.opendev.org/c/openstack/neutron/+/864649","commit_id":"03c31992a5630d07f217ee5b32b4d5cd69069388"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"2da001f8b5440fbdb40b7ae7fbbc84d555cdd4db","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        for phy_br in self.br_phys.values():"},{"line_number":82,"context_line":"            yield OVSCookieBridge(phy_br)"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    def request_physical_br(self, name):"},{"line_number":85,"context_line":"        \"\"\"Allows extensions to request one physical bridge to use for"},{"line_number":86,"context_line":"        extension specific flows."},{"line_number":87,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":6,"id":"6914c5e7_37f758bf","line":84,"updated":"2023-05-03 09:43:07.000000000","message":"`s/physical/phy` if we want to keep the convention used here.\n\nFunction used here:\nhttps://review.opendev.org/c/openstack/neutron/+/881535/4/neutron/agent/l2/extensions/metadata/metadata_path.py","commit_id":"5a17f2b24a51f62c406c1ad0ffa797480ef943f4"}]}