)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"90ac699dd598045565c3a51c3e86eb0e39a2285a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Max Lamprecht \u003cmax.lamprecht@stackit.cloud\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-18 17:27:27 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix: add NB LRP name index"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This improves LRP lookup speeds in the neutron-api at scale."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bb4b447a_c9527ffe","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"updated":"2024-12-19 11:01:19.000000000","message":"There is no need to add \"fix\", just the description.","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4bc9343a4b6771cde5ea6a056a6962b078881399","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Max Lamprecht \u003cmax.lamprecht@stackit.cloud\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-18 17:27:27 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix: add NB LRP name index"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This improves LRP lookup speeds in the neutron-api at scale."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c0139ce1_421e8668","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"03e121f8_8e98a942","updated":"2024-12-23 10:57:45.000000000","message":"Done","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"e20542f5ac4ff37605cacc073896620b9c4c038c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Max Lamprecht \u003cmax.lamprecht@stackit.cloud\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-12-18 17:27:27 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix: add NB LRP name index"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This improves LRP lookup speeds in the neutron-api at scale."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"03e121f8_8e98a942","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"bb4b447a_c9527ffe","updated":"2024-12-19 12:11:51.000000000","message":"adjusted the commit message","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"90ac699dd598045565c3a51c3e86eb0e39a2285a","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix: add NB LRP name index"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This improves LRP lookup speeds in the neutron-api at scale."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #2092133"},{"line_number":12,"context_line":"Change-Id: I9cc8460244c030f83194736cbb71c3cf449cbf76"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"08e6dfcd_f483fc93","line":9,"range":{"start_line":9,"start_character":39,"end_line":9,"end_character":50},"updated":"2024-12-19 11:01:19.000000000","message":"No need to refer to a specific process using it.","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"e20542f5ac4ff37605cacc073896620b9c4c038c","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix: add NB LRP name index"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This improves LRP lookup speeds in the neutron-api at scale."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #2092133"},{"line_number":12,"context_line":"Change-Id: I9cc8460244c030f83194736cbb71c3cf449cbf76"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"395027fa_efd5a116","line":9,"range":{"start_line":9,"start_character":39,"end_line":9,"end_character":50},"in_reply_to":"08e6dfcd_f483fc93","updated":"2024-12-19 12:11:51.000000000","message":"ok, removed it 😊","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4bc9343a4b6771cde5ea6a056a6962b078881399","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fix: add NB LRP name index"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This improves LRP lookup speeds in the neutron-api at scale."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #2092133"},{"line_number":12,"context_line":"Change-Id: I9cc8460244c030f83194736cbb71c3cf449cbf76"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e2918395_d58d251c","line":9,"range":{"start_line":9,"start_character":39,"end_line":9,"end_character":50},"in_reply_to":"395027fa_efd5a116","updated":"2024-12-23 10:57:45.000000000","message":"Done","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"90ac699dd598045565c3a51c3e86eb0e39a2285a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e91005fe_cc21e588","updated":"2024-12-19 11:01:19.000000000","message":"I was going to ask for tests but we are actually testing it: https://github.com/openstack/ovsdbapp/blob/8b7ce6dbe5baee273159d648304250d0097d3703/ovsdbapp/tests/functional/schema/ovn_northbound/test_impl_idl.py#L56-L60","commit_id":"d8fc83a7937efbcea5ec6cefb09fea99edea9e2f"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"292d0653f6d9c5e5a4bd0670b9aeb761d21f8a7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b6bf8c59_9a8cb8a2","updated":"2025-01-06 14:57:28.000000000","message":"This shouldn\u0027t be necessary as it is already indexed by default via autocreate_indicies() because the schema is already marking LRP.name as indexed.","commit_id":"1cbe61683a0bf40a7894efaa4d069b96e8862344"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"706d2a842d5b50dc06143206767bc73090e8026b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bbd684a0_fbe72e17","updated":"2025-01-02 10:01:48.000000000","message":"thanks","commit_id":"1cbe61683a0bf40a7894efaa4d069b96e8862344"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"225719dc22974b62c2a0fe921689c162f882ad4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6456d28d_ce11f01a","in_reply_to":"b6bf8c59_9a8cb8a2","updated":"2025-01-13 16:57:23.000000000","message":"Right, it is in the NB definition [1] and in the ovsdbapp code [2]\n\n[1]https://github.com/ovn-org/ovn/blob/c9361e6037ae20f714d49d98d016e33ca15763af/ovn-nb.ovsschema#L493\n[2]https://github.com/openstack/ovsdbapp/blob/master/ovsdbapp/backend/ovs_idl/__init__.py#L67-L69","commit_id":"1cbe61683a0bf40a7894efaa4d069b96e8862344"}]}