)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3f8daadca2523aff863e2f20a9b619b6a1f4acd6","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace the direct OVSDB lookups in ``BGPAgentExtension.chassis_name``"},{"line_number":10,"context_line":"and ``BGPAgentExtension.chassis_id`` with the ``chassis`` and"},{"line_number":11,"context_line":"``chassis_id`` properties already exposed by ``agent_api``."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e"},{"line_number":14,"context_line":"Change-Id: I190c38e139f441a2528d07444afed5f52230fff6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"11a13223_71032de3","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":12},"updated":"2026-07-27 12:58:46.000000000","message":"I need to review this","commit_id":"1cfb1dd928ea3d6bae034f149a06b6324e30f880"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6ec25d4499007cd4a554638a56f40ba0b937d05e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace the direct OVSDB lookups in ``BGPAgentExtension.chassis_name``"},{"line_number":10,"context_line":"and ``BGPAgentExtension.chassis_id`` with the ``chassis`` and"},{"line_number":11,"context_line":"``chassis_id`` properties already exposed by ``agent_api``."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e"},{"line_number":14,"context_line":"Change-Id: I190c38e139f441a2528d07444afed5f52230fff6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"128a3711_a6c78dc6","line":11,"range":{"start_line":11,"start_character":2,"end_line":11,"end_character":12},"in_reply_to":"11a13223_71032de3","updated":"2026-07-27 13:43:08.000000000","message":"Done","commit_id":"1cfb1dd928ea3d6bae034f149a06b6324e30f880"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"e6b2eb59806f33f71c81a74df253db90c5a8f4f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f0d03c27_bce8c154","updated":"2026-07-27 11:42:48.000000000","message":"check experimental","commit_id":"2c9b29583bf2c9a108188c308cbed73b6ca7ec1c"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"47978c99213778a8b8cfab60ef345e5b3c7dfff3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d8548827_f227b7d7","updated":"2026-07-27 12:48:52.000000000","message":"check experimental","commit_id":"01f02e43af2b514443103328c9828fd08a49f069"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7af3f801d42dbe9da90ff492dfbb7714b2835c42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d29db993_89933cf8","updated":"2026-07-27 12:57:21.000000000","message":"check experimental","commit_id":"1cfb1dd928ea3d6bae034f149a06b6324e30f880"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ee3f85584bb0023b33edb900d35865a14cabcbc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"07410dba_1c8e406c","updated":"2026-07-27 13:42:55.000000000","message":"check experimental","commit_id":"161b1dbc591610773eca228c8a9d0d7fecb3a5ec"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"068b8f0486e90abb6a7316058ac86e7dca546ee2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5ea24a43_9bddf47c","updated":"2026-07-31 06:38:55.000000000","message":"recheck neutron-functional","commit_id":"73e3f73ebcacc92369e816764f819271c43107fa"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e0c096e56ecc99d772799074cc98c37d743f8c2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"38319cac_c1f4d589","updated":"2026-07-31 09:29:32.000000000","message":"recheck neutron-tempest-plugin-ovn","commit_id":"73e3f73ebcacc92369e816764f819271c43107fa"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"16254a53e0a8eaaf0b14e29dcf72a35d8648ca33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d620d9c9_93967989","updated":"2026-07-30 13:05:21.000000000","message":"recheck neutron-tempest-plugin-ovn","commit_id":"73e3f73ebcacc92369e816764f819271c43107fa"}],"neutron/agent/ovn/extensions/bgp/__init__.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"31373410004db5b33ed5465c488a4cf159028e8e","unresolved":false,"context_lines":[{"line_number":17,"context_line":"from oslo_log import log"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron.agent.linux import ip_lib"},{"line_number":20,"context_line":"from neutron.agent.ovn.agent import ovsdb"},{"line_number":21,"context_line":"from neutron.agent.ovn.extensions.bgp import bridge"},{"line_number":22,"context_line":"from neutron.agent.ovn.extensions.bgp import commands"},{"line_number":23,"context_line":"from neutron.agent.ovn.extensions.bgp import events"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2ea1c4_d343ace7","line":20,"updated":"2026-07-27 12:12:02.000000000","message":"pep8: F401 \u0027neutron.agent.ovn.agent.ovsdb\u0027 imported but unused","commit_id":"2c9b29583bf2c9a108188c308cbed73b6ca7ec1c"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"d30c4d0630b77efd2fd237eb8348d13e2cbb44cd","unresolved":true,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @property"},{"line_number":88,"context_line":"    def chassis_uuid(self):"},{"line_number":89,"context_line":"        if self._chassis_uuid:"},{"line_number":90,"context_line":"            return self._chassis_uuid"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        chassis \u003d self.agent_api.sb_idl.db_find_rows("},{"line_number":93,"context_line":"            \u0027Chassis\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"2bd45f7e_e4f35004","line":90,"range":{"start_line":89,"start_character":0,"end_line":90,"end_character":37},"updated":"2026-07-27 14:02:25.000000000","message":"I\u0027d like to discuss caching an in-memory value and its benefits and tradeoffs. I\u0027d be more in favor of having the ovsdb value as the source of truth than risking a cache miss - in general.\n\nThe db lookup based on the name is performed on an indexed column, and is in-memory lookup. The lookup is super-fast and effective so I would argue we\u0027re gaining any gain here, even at scale of hundreds of chassis.","commit_id":"161b1dbc591610773eca228c8a9d0d7fecb3a5ec"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"23b552df4df13adcc655f789f5fd18a7dd1f9969","unresolved":true,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @property"},{"line_number":88,"context_line":"    def chassis_uuid(self):"},{"line_number":89,"context_line":"        if self._chassis_uuid:"},{"line_number":90,"context_line":"            return self._chassis_uuid"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        chassis \u003d self.agent_api.sb_idl.db_find_rows("},{"line_number":93,"context_line":"            \u0027Chassis\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"dadeb32e_b3bad3a7","line":90,"range":{"start_line":89,"start_character":0,"end_line":90,"end_character":37},"in_reply_to":"2bd45f7e_e4f35004","updated":"2026-07-27 14:20:34.000000000","message":"Yes, that\u0027s true. I\u0027ll revert this change.\n\nIn any case, it should not be possible to change the chassis UUID while running the agent. That would be a mayor change that the agent should not be able to handle. That should imply at least an error exception in the logs.","commit_id":"161b1dbc591610773eca228c8a9d0d7fecb3a5ec"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c93603f5ca5ee8260635b6f74715b3d268b87459","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @property"},{"line_number":88,"context_line":"    def chassis_uuid(self):"},{"line_number":89,"context_line":"        if self._chassis_uuid:"},{"line_number":90,"context_line":"            return self._chassis_uuid"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        chassis \u003d self.agent_api.sb_idl.db_find_rows("},{"line_number":93,"context_line":"            \u0027Chassis\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"94de567c_fd5c1f11","line":90,"range":{"start_line":89,"start_character":0,"end_line":90,"end_character":37},"in_reply_to":"dadeb32e_b3bad3a7","updated":"2026-07-30 10:58:37.000000000","message":"Done","commit_id":"161b1dbc591610773eca228c8a9d0d7fecb3a5ec"}]}