)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"70f060b2dbc5a49aec106850c48cc559fb0f6129","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7aafe2b9_6fb02f5f","updated":"2021-12-20 14:33:18.000000000","message":"Great patch! I think it would be good to cover this with unittests.","commit_id":"c72a5baf0cdb7656d71ef37b13384f3c6175aa99"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e423fb70e2482a09cd6ff871e34d1d0074dbbb31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c30a96c0_daace452","in_reply_to":"7aafe2b9_6fb02f5f","updated":"2021-12-22 10:00:35.000000000","message":"Done","commit_id":"c72a5baf0cdb7656d71ef37b13384f3c6175aa99"}],"neutron/agent/ovn/metadata/agent.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"c58e4082556513778b72f920ad3a7d7140760fe7","unresolved":true,"context_lines":[{"line_number":209,"context_line":"            self.chassis_id \u003d uuid.UUID(self.chassis)"},{"line_number":210,"context_line":"        except ValueError:"},{"line_number":211,"context_line":"            # OVS system-id could be a non UUID formatted string."},{"line_number":212,"context_line":"            self.chassis_id \u003d uuid.uuid5(OVN_METADATA_UUID_NAMESPACE,"},{"line_number":213,"context_line":"                                         self.chassis)"},{"line_number":214,"context_line":"        self.ovn_bridge \u003d self._get_ovn_bridge()"},{"line_number":215,"context_line":"        LOG.debug(\"Loaded chassis name %s (UUID: %s) and ovn bridge %s.\","}],"source_content_type":"text/x-python","patch_set":4,"id":"838a0507_64a109c0","line":212,"updated":"2021-12-21 21:38:21.000000000","message":"Just a question: maybe we should add some warning that we are changing chassis id so it will not be surprise for operator?","commit_id":"79037c951637dc06d47b6d354776d116a1d2a9ad"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e423fb70e2482a09cd6ff871e34d1d0074dbbb31","unresolved":true,"context_lines":[{"line_number":209,"context_line":"            self.chassis_id \u003d uuid.UUID(self.chassis)"},{"line_number":210,"context_line":"        except ValueError:"},{"line_number":211,"context_line":"            # OVS system-id could be a non UUID formatted string."},{"line_number":212,"context_line":"            self.chassis_id \u003d uuid.uuid5(OVN_METADATA_UUID_NAMESPACE,"},{"line_number":213,"context_line":"                                         self.chassis)"},{"line_number":214,"context_line":"        self.ovn_bridge \u003d self._get_ovn_bridge()"},{"line_number":215,"context_line":"        LOG.debug(\"Loaded chassis name %s (UUID: %s) and ovn bridge %s.\","}],"source_content_type":"text/x-python","patch_set":4,"id":"c25cb36b_8e88bb62","line":212,"in_reply_to":"838a0507_64a109c0","updated":"2021-12-22 10:00:35.000000000","message":"That\u0027s the point: we can have both UUID and non-UUID strings and both are accepted. This is why in L215-216 I print both the chassis name and the UUID.","commit_id":"79037c951637dc06d47b6d354776d116a1d2a9ad"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9de060eace845be1cc2242d9ca5ed42122f929a3","unresolved":true,"context_lines":[{"line_number":209,"context_line":"            self.chassis_id \u003d uuid.UUID(self.chassis)"},{"line_number":210,"context_line":"        except ValueError:"},{"line_number":211,"context_line":"            # OVS system-id could be a non UUID formatted string."},{"line_number":212,"context_line":"            self.chassis_id \u003d uuid.uuid5(OVN_METADATA_UUID_NAMESPACE,"},{"line_number":213,"context_line":"                                         self.chassis)"},{"line_number":214,"context_line":"        self.ovn_bridge \u003d self._get_ovn_bridge()"},{"line_number":215,"context_line":"        LOG.debug(\"Loaded chassis name %s (UUID: %s) and ovn bridge %s.\","}],"source_content_type":"text/x-python","patch_set":4,"id":"c4ed73e0_29c98f07","line":212,"in_reply_to":"c25cb36b_8e88bb62","updated":"2021-12-22 10:39:46.000000000","message":"Ok, maybe my question wasn\u0027t clear. Maybe we should increase level of that message in L215-216 in case when those 2 things aren\u0027t the same.","commit_id":"79037c951637dc06d47b6d354776d116a1d2a9ad"}]}