)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"589d65e0768c67f91fdad32f8a63eb112a32cf8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"805cf3d3_48f4644e","updated":"2022-04-28 14:26:39.000000000","message":"The __str__() I\u0027ve implemented so far use the fields that the command line\nclient\u0027s respective \"list\" command shows. Tests will follow once we agree on\nthe implementation.","commit_id":"6fff9c7d13f59bbc57efda23f56fb207fb445b68"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"00d7eabefe70e9187336bfcd7cbcaa847f47b362","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3fa61ce7_53fbd4d6","updated":"2022-04-29 06:03:55.000000000","message":"recheck strange errors","commit_id":"6fff9c7d13f59bbc57efda23f56fb207fb445b68"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"be29145983dae66f677f11ced69b2bac926d3b8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f03b3c1b_d411907f","updated":"2022-06-24 20:39:14.000000000","message":"LGTM","commit_id":"ff6d0c5280fbdbe8f5361469fb88448fa650f365"}],"octavia/db/models.py":[{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"eb7e6be1180f58fb13263f75251b6a571ef177c3","unresolved":true,"context_lines":[{"line_number":650,"context_line":"                f\"{self.load_balancer_id!r}, status\u003d{self.status!r}, \""},{"line_number":651,"context_line":"                f\"role\u003d{self.role!r}, lb_network_ip\u003d{self.lb_network_ip!r}, \""},{"line_number":652,"context_line":"                f\"ha_ip\u003d{self.vrrp_ip!r})\")"},{"line_number":653,"context_line":""},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"class AmphoraHealth(base_models.BASE):"},{"line_number":656,"context_line":"    __data_model__ \u003d data_models.AmphoraHealth"}],"source_content_type":"text/x-python","patch_set":3,"id":"2d13a2af_dce26cef","line":653,"updated":"2022-05-04 10:21:47.000000000","message":"What about other objects like listeners and healthmonitors?","commit_id":"6fff9c7d13f59bbc57efda23f56fb207fb445b68"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"089de38df9c3f46a48f42bb27b19ee39b9f17511","unresolved":true,"context_lines":[{"line_number":650,"context_line":"                f\"{self.load_balancer_id!r}, status\u003d{self.status!r}, \""},{"line_number":651,"context_line":"                f\"role\u003d{self.role!r}, lb_network_ip\u003d{self.lb_network_ip!r}, \""},{"line_number":652,"context_line":"                f\"ha_ip\u003d{self.vrrp_ip!r})\")"},{"line_number":653,"context_line":""},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"class AmphoraHealth(base_models.BASE):"},{"line_number":656,"context_line":"    __data_model__ \u003d data_models.AmphoraHealth"}],"source_content_type":"text/x-python","patch_set":3,"id":"419def8b_ee7d1ab9","line":653,"in_reply_to":"2d13a2af_dce26cef","updated":"2022-05-04 12:08:14.000000000","message":"I was hoping for feedback about which classes should have a __str__() and about how others think about that idea in general. Listener and Healthmonitor would definitely benefit from it IMO.\n\nI will now move on and implement __str__() for more classes now and I will use https://github.com/openstack/python-octaviaclient/blob/master/octaviaclient/osc/v2/constants.py as an orientation for which fields should be included.","commit_id":"6fff9c7d13f59bbc57efda23f56fb207fb445b68"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"1700628a7f2090a3d1c5c0410d7f3390b1a04817","unresolved":false,"context_lines":[{"line_number":650,"context_line":"                f\"{self.load_balancer_id!r}, status\u003d{self.status!r}, \""},{"line_number":651,"context_line":"                f\"role\u003d{self.role!r}, lb_network_ip\u003d{self.lb_network_ip!r}, \""},{"line_number":652,"context_line":"                f\"ha_ip\u003d{self.vrrp_ip!r})\")"},{"line_number":653,"context_line":""},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"class AmphoraHealth(base_models.BASE):"},{"line_number":656,"context_line":"    __data_model__ \u003d data_models.AmphoraHealth"}],"source_content_type":"text/x-python","patch_set":3,"id":"68d8a4ca_9d773bdc","line":653,"in_reply_to":"419def8b_ee7d1ab9","updated":"2022-05-04 12:48:03.000000000","message":"Yes, I think this would be useful, thank you for proposing this!","commit_id":"6fff9c7d13f59bbc57efda23f56fb207fb445b68"}]}