)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"032718f9ef0362d5ccde5aa6056ecccb175ee8bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"514c0d27_317fe815","updated":"2024-12-04 16:24:54.000000000","message":"I guess this is where all the tests are","commit_id":"2f10a39283b22fc0da2f31e9bada40dce1e74dda"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"599407b397d697c3ff5f852c2b2e309a9941279c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"6cf19fd3_99d4bc20","updated":"2024-12-13 09:55:21.000000000","message":"rech","commit_id":"f89fa79ce089018b40fcada317b7cbaa52018bc3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"09d7827ef0ef5a412d94f6e489d35052a22d8615","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"92dbc754_c987d33d","updated":"2024-12-18 16:14:55.000000000","message":"Just one comment, otherwise looks good","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0f70170eea0eef3e1be09a8331ccb29500974c45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"5ec21c6b_b984d854","updated":"2024-12-20 14:49:14.000000000","message":"recheck neutron-functional","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"26354333ef7529431ebc9b55acc1ade7ca1f26ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"2d56d577_425009a2","updated":"2024-12-21 13:04:12.000000000","message":"recheck neutron-ovs-tempest-dvr-ha-multinode-full","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c799d049cb3b14589f52d63d7b1d73bbab25b921","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"8340f752_540d6f92","updated":"2024-12-20 11:31:22.000000000","message":"recheck neutron-tempest-plugin-ovn","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"}],"neutron/db/ovn_hash_ring_db.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"032718f9ef0362d5ccde5aa6056ecccb175ee8bd","unresolved":true,"context_lines":[{"line_number":103,"context_line":"def touch_node(context, node_uuid, updated_at\u003dNone):"},{"line_number":104,"context_line":"    # NOTE(ralonsoh): there are several mechanisms to update the node OVN hash"},{"line_number":105,"context_line":"    # ring register. This method does not retry the DB operation in case of"},{"line_number":106,"context_line":"    # failure but relays on later calls. That will prevent from blocking the"},{"line_number":107,"context_line":"    # DB needlessly."},{"line_number":108,"context_line":"    if updated_at is None:"},{"line_number":109,"context_line":"        updated_at \u003d timeutils.utcnow()"}],"source_content_type":"text/x-python","patch_set":6,"id":"069c892f_68e77083","line":106,"range":{"start_line":106,"start_character":18,"end_line":106,"end_character":24},"updated":"2024-12-04 16:24:54.000000000","message":"s/relies ? meaning a later call will fail?","commit_id":"2f10a39283b22fc0da2f31e9bada40dce1e74dda"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3ecdb6967b84282cbcaf3f2d27f17c397c107f4f","unresolved":false,"context_lines":[{"line_number":103,"context_line":"def touch_node(context, node_uuid, updated_at\u003dNone):"},{"line_number":104,"context_line":"    # NOTE(ralonsoh): there are several mechanisms to update the node OVN hash"},{"line_number":105,"context_line":"    # ring register. This method does not retry the DB operation in case of"},{"line_number":106,"context_line":"    # failure but relays on later calls. That will prevent from blocking the"},{"line_number":107,"context_line":"    # DB needlessly."},{"line_number":108,"context_line":"    if updated_at is None:"},{"line_number":109,"context_line":"        updated_at \u003d timeutils.utcnow()"}],"source_content_type":"text/x-python","patch_set":6,"id":"f41b75d9_967d82f7","line":106,"range":{"start_line":106,"start_character":18,"end_line":106,"end_character":24},"in_reply_to":"069c892f_68e77083","updated":"2024-12-06 12:39:05.000000000","message":"No, actually the current call can fail but the following ones should succeed.\n\nI\u0027ve update the text.","commit_id":"2f10a39283b22fc0da2f31e9bada40dce1e74dda"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"032718f9ef0362d5ccde5aa6056ecccb175ee8bd","unresolved":true,"context_lines":[{"line_number":371,"context_line":"            context, self.hash_ring_group, created_at\u003dcreated_at)"},{"line_number":372,"context_line":"        newer_nodes \u003d ovn_hash_ring_db.get_nodes("},{"line_number":373,"context_line":"            context, self.hash_ring_group, created_at\u003dcreated_at)"},{"line_number":374,"context_line":"        LOG.debug(\u0027Hash Ring setup, this worker has detected %s OVN hash\u0027"},{"line_number":375,"context_line":"                  \u0027ring registers in the database\u0027, len(newer_nodes))"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":"    def post_fork_initialize(self, resource, event, trigger, payload\u003dNone):"}],"source_content_type":"text/x-python","patch_set":6,"id":"41b922ce_254a2238","line":374,"updated":"2024-12-04 16:24:54.000000000","message":"nit: missing trailing space","commit_id":"2f10a39283b22fc0da2f31e9bada40dce1e74dda"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3ecdb6967b84282cbcaf3f2d27f17c397c107f4f","unresolved":false,"context_lines":[{"line_number":371,"context_line":"            context, self.hash_ring_group, created_at\u003dcreated_at)"},{"line_number":372,"context_line":"        newer_nodes \u003d ovn_hash_ring_db.get_nodes("},{"line_number":373,"context_line":"            context, self.hash_ring_group, created_at\u003dcreated_at)"},{"line_number":374,"context_line":"        LOG.debug(\u0027Hash Ring setup, this worker has detected %s OVN hash\u0027"},{"line_number":375,"context_line":"                  \u0027ring registers in the database\u0027, len(newer_nodes))"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":"    def post_fork_initialize(self, resource, event, trigger, payload\u003dNone):"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a8bbadd_5461527d","line":374,"in_reply_to":"41b922ce_254a2238","updated":"2024-12-06 12:39:05.000000000","message":"Done","commit_id":"2f10a39283b22fc0da2f31e9bada40dce1e74dda"}],"neutron/tests/unit/common/ovn/test_hash_ring_manager.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"09d7827ef0ef5a412d94f6e489d35052a22d8615","unresolved":true,"context_lines":[{"line_number":122,"context_line":"                                      \u0027node-2\u0027)"},{"line_number":123,"context_line":"        hash_dict_before \u003d {\u0027fake-uuid\u0027: node1,"},{"line_number":124,"context_line":"                            \u0027fake-uuid-0\u0027: node2,"},{"line_number":125,"context_line":"                            \u0027fake-uuid-ABCDE\u0027: node_other}"},{"line_number":126,"context_line":"        self._verify_hashes(hash_dict_before)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch.object(hash_ring_manager.LOG, \u0027debug\u0027)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5e3ad1a1_a9a75272","line":125,"updated":"2024-12-18 16:14:55.000000000","message":"Is this necessary? The dict was already created above.","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c152e928e1ed5ae710264626e8605b37859d5382","unresolved":false,"context_lines":[{"line_number":122,"context_line":"                                      \u0027node-2\u0027)"},{"line_number":123,"context_line":"        hash_dict_before \u003d {\u0027fake-uuid\u0027: node1,"},{"line_number":124,"context_line":"                            \u0027fake-uuid-0\u0027: node2,"},{"line_number":125,"context_line":"                            \u0027fake-uuid-ABCDE\u0027: node_other}"},{"line_number":126,"context_line":"        self._verify_hashes(hash_dict_before)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch.object(hash_ring_manager.LOG, \u0027debug\u0027)"}],"source_content_type":"text/x-python","patch_set":13,"id":"2a195b64_58673ee5","line":125,"in_reply_to":"47630cff_a5f6fa91","updated":"2024-12-19 20:24:08.000000000","message":"Ah, timestamp has changed. The comment threw me off.","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9b82bf9d92a2449d416e25a452a3f0a0cb11cede","unresolved":false,"context_lines":[{"line_number":122,"context_line":"                                      \u0027node-2\u0027)"},{"line_number":123,"context_line":"        hash_dict_before \u003d {\u0027fake-uuid\u0027: node1,"},{"line_number":124,"context_line":"                            \u0027fake-uuid-0\u0027: node2,"},{"line_number":125,"context_line":"                            \u0027fake-uuid-ABCDE\u0027: node_other}"},{"line_number":126,"context_line":"        self._verify_hashes(hash_dict_before)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @mock.patch.object(hash_ring_manager.LOG, \u0027debug\u0027)"}],"source_content_type":"text/x-python","patch_set":13,"id":"47630cff_a5f6fa91","line":125,"in_reply_to":"5e3ad1a1_a9a75272","updated":"2024-12-19 12:41:23.000000000","message":"Yes, node1 and node2 changed so we need to create again this dictionary","commit_id":"ca41c99f0b1013fdac61d067f6a62468c0516c89"}]}