)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"b2d08cbded16d493f2cfba02c0cf88f3d89d3fcd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2e16baa4_39342859","updated":"2023-03-07 08:34:11.000000000","message":"Don\u0027t it means that now effectively min supported ovn version is 20.03? If so, should we maybe add release note about that?","commit_id":"999116126e2a7d5c53f275228a3eeda5e98c6445"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"e97840638663ef047d07d9e3eddfe3967ee3b888","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"aa69caab_01c76ed2","updated":"2023-03-03 06:49:26.000000000","message":"LGTM","commit_id":"999116126e2a7d5c53f275228a3eeda5e98c6445"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1e31c4cdebd9b0784171c82926905c8e69909ad5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"89704e5b_17f4c599","in_reply_to":"2e16baa4_39342859","updated":"2023-03-07 08:45:26.000000000","message":"Yes but","commit_id":"999116126e2a7d5c53f275228a3eeda5e98c6445"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1e31c4cdebd9b0784171c82926905c8e69909ad5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"68067ee7_fba6708a","in_reply_to":"2e16baa4_39342859","updated":"2023-03-07 08:45:26.000000000","message":"Yes but that was done in previous patches [1][2]. We are already enforcing newer versions since [1] and [2], and we have released notes about this. This is just an extra check and to print, if necessary, what will fail in the environment if the NB version is lower than required.\n\n[1]https://opendev.org/openstack/neutron/commit/ce1a87057a12c4f27bb4eb2d2a6f2b29d2709d65\n[2]https://opendev.org/openstack/neutron/commit/5d94a10a1507625a861585bc83a295fb9442704e","commit_id":"999116126e2a7d5c53f275228a3eeda5e98c6445"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ce7fbc9fef137797553a6da2404d3415e8058e43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0a0beefb_c603b01e","in_reply_to":"89704e5b_17f4c599","updated":"2023-03-07 09:19:01.000000000","message":"ok, so LGTM :)","commit_id":"999116126e2a7d5c53f275228a3eeda5e98c6445"}],"neutron/cmd/sanity_check.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"459b9eb91f113c3539ff708c96e12523cf5a63cf","unresolved":true,"context_lines":[{"line_number":342,"context_line":"def check_ovn_nb_db_schema_gateway_chassis():"},{"line_number":343,"context_line":"    result \u003d checks.ovn_nb_db_schema_gateway_chassis_supported()"},{"line_number":344,"context_line":"    if not result:"},{"line_number":345,"context_line":"        LOG.warning(\u0027OVN SB DB schema does not support \"Chassis_Gateway\" \u0027"},{"line_number":346,"context_line":"                    \u0027table. This support was added in DB schema version 5.7.\u0027)"},{"line_number":347,"context_line":"    return result"},{"line_number":348,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bd31af88_cbd170d3","line":345,"range":{"start_line":345,"start_character":25,"end_line":345,"end_character":27},"updated":"2023-02-22 17:31:57.000000000","message":"s/NB","commit_id":"f4e0b3377508dc5a1f044d1e554c8633f1c1521a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f16bc3be8e27384d68222ecff6879ce6ade7a045","unresolved":false,"context_lines":[{"line_number":342,"context_line":"def check_ovn_nb_db_schema_gateway_chassis():"},{"line_number":343,"context_line":"    result \u003d checks.ovn_nb_db_schema_gateway_chassis_supported()"},{"line_number":344,"context_line":"    if not result:"},{"line_number":345,"context_line":"        LOG.warning(\u0027OVN SB DB schema does not support \"Chassis_Gateway\" \u0027"},{"line_number":346,"context_line":"                    \u0027table. This support was added in DB schema version 5.7.\u0027)"},{"line_number":347,"context_line":"    return result"},{"line_number":348,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e54327b5_c14f2cd9","line":345,"range":{"start_line":345,"start_character":25,"end_line":345,"end_character":27},"in_reply_to":"bd31af88_cbd170d3","updated":"2023-02-23 09:11:45.000000000","message":"Done","commit_id":"f4e0b3377508dc5a1f044d1e554c8633f1c1521a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"459b9eb91f113c3539ff708c96e12523cf5a63cf","unresolved":true,"context_lines":[{"line_number":429,"context_line":"                    default\u003dFalse),"},{"line_number":430,"context_line":"    BoolOptCallback(\u0027ovn_nb_db_schema_gateway_chassis_support\u0027,"},{"line_number":431,"context_line":"                    check_ovn_nb_db_schema_gateway_chassis,"},{"line_number":432,"context_line":"                    help\u003d_(\u0027Check OVN SB DB schema support \"Gateway_Chassis\"\u0027),"},{"line_number":433,"context_line":"                    default\u003dFalse),"},{"line_number":434,"context_line":"]"},{"line_number":435,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1e3c98c7_cfc635c9","line":432,"range":{"start_line":432,"start_character":38,"end_line":432,"end_character":40},"updated":"2023-02-22 17:31:57.000000000","message":"s/NB\n\nAnd super-nit is you don\u0027t really need the \"\" here or above as the other ones don\u0027t use it.","commit_id":"f4e0b3377508dc5a1f044d1e554c8633f1c1521a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f16bc3be8e27384d68222ecff6879ce6ade7a045","unresolved":false,"context_lines":[{"line_number":429,"context_line":"                    default\u003dFalse),"},{"line_number":430,"context_line":"    BoolOptCallback(\u0027ovn_nb_db_schema_gateway_chassis_support\u0027,"},{"line_number":431,"context_line":"                    check_ovn_nb_db_schema_gateway_chassis,"},{"line_number":432,"context_line":"                    help\u003d_(\u0027Check OVN SB DB schema support \"Gateway_Chassis\"\u0027),"},{"line_number":433,"context_line":"                    default\u003dFalse),"},{"line_number":434,"context_line":"]"},{"line_number":435,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"b29a6518_09667990","line":432,"range":{"start_line":432,"start_character":38,"end_line":432,"end_character":40},"in_reply_to":"1e3c98c7_cfc635c9","updated":"2023-02-23 09:11:45.000000000","message":"Done","commit_id":"f4e0b3377508dc5a1f044d1e554c8633f1c1521a"}]}
