)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6c3b1ae7b405c50dd89b9d60755338ea22f43182","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"669449eb_1a483cf6","updated":"2023-09-28 08:12:36.000000000","message":"check experimental","commit_id":"180e961c246583737db2728d19d76a97e712627d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4c2a92bd1f9642d7ae8e6e191b28c73bfb579954","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"5152ff15_d380a76f","updated":"2023-10-11 15:19:44.000000000","message":"**NOTE**: we are no longer (we shouldn\u0027t) using Ubuntu Focal and OVN 20.03 in the CI.","commit_id":"b36f6118a4bd6f5e550f2e1f642c08b29cbb04fd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"605f77edd31ed8e5b7de06f19c277f7b8193bd3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7349e952_48919443","updated":"2023-10-13 07:15:16.000000000","message":"recheck tempest","commit_id":"b36f6118a4bd6f5e550f2e1f642c08b29cbb04fd"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"585cd8e6d934755d3385d3d513031b0768879e89","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6537d6de_421659a2","updated":"2023-10-17 19:08:29.000000000","message":"recheck unrelated failure","commit_id":"3244870653e3c0cb8c5d6165cd08f45388b2ceb7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"49e1f25030ab6ee3934ee44ca4f403eedc6ad2ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"29bf2fd7_bf086a5e","updated":"2023-10-19 14:57:59.000000000","message":"recheck tempest-integrated-networking","commit_id":"3d5b33de62d42ada480383952bde6d417f8a9a47"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"9bca8f015cca9bf688d3b4deffd8dadf6d9f64a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7604c63a_cb91a1eb","updated":"2023-12-18 17:04:42.000000000","message":"- 1\n\nPlease see the comment right above this block\n\n# PLEASE DO NOT ADD NEW VERSION CHECK TESTS. The entire point of the sanity\n￼47\n# check system is to avoid doing version number checking since different\n￼47\n# check system is to avoid doing version number checking since different\n￼48\n# distros may backport fixes/features to different versioned releases and they\n￼48\n# distros may backport fixes/features to different versioned releases and they\n￼49\n# are not a reliable way to test for features/bugs. Please write sanity checks\n￼49\n# are not a reliable way to test for features/bugs. Please write sanity checks\n￼50\n# that actually test for the usage of the feature in combination with it being\n￼50\n# that actually test for the usage of the feature in combination with it being\n￼51\n# configured to use on the running system (e.g. via a config option).\n￼51\n# configured to use on the running system (e.g. via a config option).","commit_id":"a579e504e982980427c3b3cc2958f6b673f2cbc0"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"0991749d24b9e9250a650729ac519340aa7c8fae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"9e42c9cf_2af085d3","in_reply_to":"7604c63a_cb91a1eb","updated":"2023-12-18 17:05:17.000000000","message":"doh, should have used a code block","commit_id":"a579e504e982980427c3b3cc2958f6b673f2cbc0"}],"neutron/cmd/sanity/checks.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"8af76a60d6c781e99696e89c4354c1bdd12ea115","unresolved":true,"context_lines":[{"line_number":64,"context_line":"OVN_NB_DB_SCHEMA_STATELESS_NAT \u003d \u00275.17.0\u0027"},{"line_number":65,"context_line":"OVN_SB_DB_SCHEMA_VIRTUAL_PORT \u003d \u00272.5.0\u0027"},{"line_number":66,"context_line":"OVN_LOCALNET_LEARN_FDB \u003d \u002722.09.0\u0027"},{"line_number":67,"context_line":"OVN_SB_DB_SCHEMA_CHASSIS_PRIVATE \u003d \u00272.9.0\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"class OVNCheckType(enum.Enum):"}],"source_content_type":"text/x-python","patch_set":10,"id":"b210254f_1c89ec5b","line":67,"updated":"2023-12-18 17:10:43.000000000","message":"From the comment above this block:\n\n```\n# PLEASE DO NOT ADD NEW VERSION CHECK TESTS. The entire point of the sanity\n# check system is to avoid doing version number checking since different\n# check system is to avoid doing version number checking since different\n# distros may backport fixes/features to different versioned releases and they\n# distros may backport fixes/features to different versioned releases and they\n# are not a reliable way to test for features/bugs. Please write sanity checks\n# are not a reliable way to test for features/bugs. Please write sanity checks\n# that actually test for the usage of the feature in combination with it being\n# that actually test for the usage of the feature in combination with it being\n# configured to use on the running system (e.g. via a config option).\n# configured to use on the running system (e.g. via a config option).\n```\n\nIf we even need to do this check, we should do it by connecting to the the configured DB and testing if the `Chassis_Private` table exists.","commit_id":"a579e504e982980427c3b3cc2958f6b673f2cbc0"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"3c9363fc076519e4c1e1d113f42cd787edd187dd","unresolved":false,"context_lines":[{"line_number":64,"context_line":"OVN_NB_DB_SCHEMA_STATELESS_NAT \u003d \u00275.17.0\u0027"},{"line_number":65,"context_line":"OVN_SB_DB_SCHEMA_VIRTUAL_PORT \u003d \u00272.5.0\u0027"},{"line_number":66,"context_line":"OVN_LOCALNET_LEARN_FDB \u003d \u002722.09.0\u0027"},{"line_number":67,"context_line":"OVN_SB_DB_SCHEMA_CHASSIS_PRIVATE \u003d \u00272.9.0\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"class OVNCheckType(enum.Enum):"}],"source_content_type":"text/x-python","patch_set":10,"id":"a6f033b2_e814b513","line":67,"in_reply_to":"b210254f_1c89ec5b","updated":"2023-12-18 17:14:14.000000000","message":"I give up trying to successfully copy/paste. :p","commit_id":"a579e504e982980427c3b3cc2958f6b673f2cbc0"}],"neutron/cmd/sanity_check.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f7e21b28269cf2a53ca4fab5f8119d14840937b1","unresolved":true,"context_lines":[{"line_number":451,"context_line":"                    help\u003d_(\u0027Check OVN supports localnet_learn_fdb option\u0027),"},{"line_number":452,"context_line":"                    default\u003dFalse),"},{"line_number":453,"context_line":"    BoolOptCallback(\u0027ovn_sb_db_schema_chassis_private_support\u0027,"},{"line_number":454,"context_line":"                    check_ovn_sb_db_schema_virtual_port,"},{"line_number":455,"context_line":"                    help\u003d_(\u0027Check OVN SB DB schema support chassis private\u0027),"},{"line_number":456,"context_line":"                    default\u003dFalse),"},{"line_number":457,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":7,"id":"77614e28_bef78c80","line":454,"updated":"2023-10-13 17:51:18.000000000","message":"check_ovn_sb_db_schema_chassis_private","commit_id":"b36f6118a4bd6f5e550f2e1f642c08b29cbb04fd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"992ac5326f8f2132c17b0705e0ef8be25f8664ba","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                    help\u003d_(\u0027Check OVN supports localnet_learn_fdb option\u0027),"},{"line_number":452,"context_line":"                    default\u003dFalse),"},{"line_number":453,"context_line":"    BoolOptCallback(\u0027ovn_sb_db_schema_chassis_private_support\u0027,"},{"line_number":454,"context_line":"                    check_ovn_sb_db_schema_virtual_port,"},{"line_number":455,"context_line":"                    help\u003d_(\u0027Check OVN SB DB schema support chassis private\u0027),"},{"line_number":456,"context_line":"                    default\u003dFalse),"},{"line_number":457,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":7,"id":"301d920a_1e2c8e7f","line":454,"in_reply_to":"77614e28_bef78c80","updated":"2023-10-17 11:39:04.000000000","message":"Done","commit_id":"b36f6118a4bd6f5e550f2e1f642c08b29cbb04fd"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f7e21b28269cf2a53ca4fab5f8119d14840937b1","unresolved":true,"context_lines":[{"line_number":452,"context_line":"                    default\u003dFalse),"},{"line_number":453,"context_line":"    BoolOptCallback(\u0027ovn_sb_db_schema_chassis_private_support\u0027,"},{"line_number":454,"context_line":"                    check_ovn_sb_db_schema_virtual_port,"},{"line_number":455,"context_line":"                    help\u003d_(\u0027Check OVN SB DB schema support chassis private\u0027),"},{"line_number":456,"context_line":"                    default\u003dFalse),"},{"line_number":457,"context_line":"]"},{"line_number":458,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"4c68451d_2fef4929","line":455,"range":{"start_line":455,"start_character":59,"end_line":455,"end_character":74},"updated":"2023-10-13 17:51:18.000000000","message":"nit: supports Chassis_Private (to match name in schema file)","commit_id":"b36f6118a4bd6f5e550f2e1f642c08b29cbb04fd"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"992ac5326f8f2132c17b0705e0ef8be25f8664ba","unresolved":false,"context_lines":[{"line_number":452,"context_line":"                    default\u003dFalse),"},{"line_number":453,"context_line":"    BoolOptCallback(\u0027ovn_sb_db_schema_chassis_private_support\u0027,"},{"line_number":454,"context_line":"                    check_ovn_sb_db_schema_virtual_port,"},{"line_number":455,"context_line":"                    help\u003d_(\u0027Check OVN SB DB schema support chassis private\u0027),"},{"line_number":456,"context_line":"                    default\u003dFalse),"},{"line_number":457,"context_line":"]"},{"line_number":458,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"748c5260_c3f59646","line":455,"range":{"start_line":455,"start_character":59,"end_line":455,"end_character":74},"in_reply_to":"4c68451d_2fef4929","updated":"2023-10-17 11:39:04.000000000","message":"Done","commit_id":"b36f6118a4bd6f5e550f2e1f642c08b29cbb04fd"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"fee75a76aab1420534829ab2a37aef4116dfb663","unresolved":true,"context_lines":[{"line_number":711,"context_line":"            self.driver.hash_ring_group)"},{"line_number":712,"context_line":"        self._last_touch \u003d None"},{"line_number":713,"context_line":"        # This is a map of tables that may be new after OVN database is updated"},{"line_number":714,"context_line":"        self._tables_to_register \u003d {"},{"line_number":715,"context_line":"            \u0027OVN_Southbound\u0027: [\u0027Chassis_Private\u0027],"},{"line_number":716,"context_line":"        }"},{"line_number":717,"context_line":""},{"line_number":718,"context_line":"    def handle_db_schema_changes(self, event, row):"},{"line_number":719,"context_line":"        if (event \u003d\u003d row_event.RowEvent.ROW_CREATE and"}],"source_content_type":"text/x-python","patch_set":10,"id":"6c198b91_56810649","line":716,"range":{"start_line":714,"start_character":0,"end_line":716,"end_character":9},"updated":"2024-01-13 17:48:09.000000000","message":"This can also be changed to `\u003d {}` if we still want to keep `handle_db_schema_changes()` since we no longer need to consider adding this table at upgrade.","commit_id":"a579e504e982980427c3b3cc2958f6b673f2cbc0"}]}
