)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c4d93c36495915e624c5a00b3d6893d3994071af","unresolved":true,"context_lines":[{"line_number":1,"context_line":"Parent:     79d9b0cc (Merge \"If OVS Manager creation failes retry to set values\")"},{"line_number":2,"context_line":"Author:     Maksim Kostrow \u003cmaks.kostrow@yandex.ru\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-04-02 11:47:59 +0300"},{"line_number":4,"context_line":"Commit:     Maksim Kostrow \u003cmaks.kostrow@yandex.ru\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"f0fd70ec_ea9e20d4","line":1,"updated":"2025-04-07 08:45:54.000000000","message":"Nit: please add line-breaks to the commit msg to make it readable in gerrit also","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"df66d47f833044043eb46a5c2407a93797d59eac","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     79d9b0cc (Merge \"If OVS Manager creation failes retry to set values\")"},{"line_number":2,"context_line":"Author:     Maksim Kostrow \u003cmaks.kostrow@yandex.ru\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-04-02 11:47:59 +0300"},{"line_number":4,"context_line":"Commit:     Maksim Kostrow \u003cmaks.kostrow@yandex.ru\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d7e59108_568ac264","line":1,"in_reply_to":"347b865f_1387bd62","updated":"2025-06-26 09:08:04.000000000","message":"Done","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"},{"author":{"_account_id":37891,"name":"Maksim Kostrow","display_name":"Maksim Kostrow","email":"maks.kostrow@yandex.ru","username":"maks-bezbond"},"change_message_id":"1b9c2f3b3d5bb18987c0e9648acdff7989a764f7","unresolved":true,"context_lines":[{"line_number":1,"context_line":"Parent:     79d9b0cc (Merge \"If OVS Manager creation failes retry to set values\")"},{"line_number":2,"context_line":"Author:     Maksim Kostrow \u003cmaks.kostrow@yandex.ru\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-04-02 11:47:59 +0300"},{"line_number":4,"context_line":"Commit:     Maksim Kostrow \u003cmaks.kostrow@yandex.ru\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"347b865f_1387bd62","line":1,"in_reply_to":"f0fd70ec_ea9e20d4","updated":"2025-05-15 16:28:01.000000000","message":"fix","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c4d93c36495915e624c5a00b3d6893d3994071af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"34100c01_562c03f1","updated":"2025-04-07 08:45:54.000000000","message":"Thanks for checking, please consider the comments","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a6c98c91b96d7118213ed2f2a44c558f543903bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3d99cd42_1d44002a","updated":"2025-04-02 09:31:09.000000000","message":"Thanks for the unit test.\n\nThis is why we need to use constants. Please replace \u0027sub_ports\u0027 (and \u0027subports\u0027) with the corresponding constant [1]. Btw, I\u0027ve seen this \u0027subports\u0027 constant also used in the OVN code, so this patch needs a wider coverage or a follow-up patch.\n\n[1]https://github.com/openstack/neutron-lib/blob/950f0f3a8a722da115c03f2abb756a732a3ec4c1/neutron_lib/api/definitions/trunk.py#L46","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"82ee0e600d161f4cafef08ca0c63bf42cdd57a2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"52119ef3_a9edce0c","updated":"2025-06-26 08:56:32.000000000","message":"good catch, thanks","commit_id":"1c93c84d1c16d9ede1950f9756b8bd04302fdc68"},{"author":{"_account_id":37891,"name":"Maksim Kostrow","display_name":"Maksim Kostrow","email":"maks.kostrow@yandex.ru","username":"maks-bezbond"},"change_message_id":"a8f64617e4e82e598c14a5ed91f642791a0223bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"63825486_543dc352","updated":"2025-06-27 08:20:21.000000000","message":"recheck due to failed timeout in neutron functional tests","commit_id":"1c93c84d1c16d9ede1950f9756b8bd04302fdc68"}],"neutron/services/trunk/plugin.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c4d93c36495915e624c5a00b3d6893d3994071af","unresolved":true,"context_lines":[{"line_number":112,"context_line":"        subport_ids \u003d []"},{"line_number":113,"context_line":"        trunk_ports \u003d []"},{"line_number":114,"context_line":"        for p in ports_res:"},{"line_number":115,"context_line":"            if \u0027trunk_details\u0027 in p and \u0027sub_ports\u0027 in p[\u0027trunk_details\u0027]:"},{"line_number":116,"context_line":"                trunk_ports.append(p)"},{"line_number":117,"context_line":"                for subp in p[\u0027trunk_details\u0027][\u0027sub_ports\u0027]:"},{"line_number":118,"context_line":"                    subport_ids.append(subp[\u0027port_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"2c8b88e4_ba945c38","line":115,"range":{"start_line":115,"start_character":41,"end_line":115,"end_character":50},"updated":"2025-04-07 08:45:54.000000000","message":"as Rodolfo wrote please use the constant from neutron-lib:\nhttps://opendev.org/openstack/neutron-lib/src/branch/master/neutron_lib/api/definitions/trunk.py#L46","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"},{"author":{"_account_id":37891,"name":"Maksim Kostrow","display_name":"Maksim Kostrow","email":"maks.kostrow@yandex.ru","username":"maks-bezbond"},"change_message_id":"1b9c2f3b3d5bb18987c0e9648acdff7989a764f7","unresolved":true,"context_lines":[{"line_number":112,"context_line":"        subport_ids \u003d []"},{"line_number":113,"context_line":"        trunk_ports \u003d []"},{"line_number":114,"context_line":"        for p in ports_res:"},{"line_number":115,"context_line":"            if \u0027trunk_details\u0027 in p and \u0027sub_ports\u0027 in p[\u0027trunk_details\u0027]:"},{"line_number":116,"context_line":"                trunk_ports.append(p)"},{"line_number":117,"context_line":"                for subp in p[\u0027trunk_details\u0027][\u0027sub_ports\u0027]:"},{"line_number":118,"context_line":"                    subport_ids.append(subp[\u0027port_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"a4c08c15_a53d5de1","line":115,"range":{"start_line":115,"start_character":41,"end_line":115,"end_character":50},"in_reply_to":"2c8b88e4_ba945c38","updated":"2025-05-15 16:28:01.000000000","message":"fix","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"df66d47f833044043eb46a5c2407a93797d59eac","unresolved":false,"context_lines":[{"line_number":112,"context_line":"        subport_ids \u003d []"},{"line_number":113,"context_line":"        trunk_ports \u003d []"},{"line_number":114,"context_line":"        for p in ports_res:"},{"line_number":115,"context_line":"            if \u0027trunk_details\u0027 in p and \u0027sub_ports\u0027 in p[\u0027trunk_details\u0027]:"},{"line_number":116,"context_line":"                trunk_ports.append(p)"},{"line_number":117,"context_line":"                for subp in p[\u0027trunk_details\u0027][\u0027sub_ports\u0027]:"},{"line_number":118,"context_line":"                    subport_ids.append(subp[\u0027port_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"b30b0759_9924e1d2","line":115,"range":{"start_line":115,"start_character":41,"end_line":115,"end_character":50},"in_reply_to":"a4c08c15_a53d5de1","updated":"2025-06-26 09:08:04.000000000","message":"Done","commit_id":"40a622d811e263c3bf5e7785dfc35fd43b718810"}]}
