)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6f61e23f347905172a886d03b53d166f828dadfd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"97a06646_69889e19","updated":"2023-04-05 18:52:23.000000000","message":"One minor comment here","commit_id":"78ef74155bc0451770500340adea42eef6fe3198"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"869ba96588823dd549176d167bd0c57afdf10649","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0589c432_e255b44a","updated":"2023-04-13 18:03:04.000000000","message":"LGTM\nThanks for the update","commit_id":"324c3528ec7726fb6c69f70473b75b14357ff14a"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"7eed8150ba3f630c857268fda426eafa88107a42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f80fea39_e2ba52f7","updated":"2023-04-18 13:04:53.000000000","message":"recheck controller | W: Failed to fetch https://mirror.ca-ymq-1.vexxhost.opendev.org/ubuntu/dists/focal/InRelease","commit_id":"324c3528ec7726fb6c69f70473b75b14357ff14a"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"6adbab7a303ff18d0837f402da28244177b534a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0f052c48_51c4af4b","updated":"2023-04-18 15:47:09.000000000","message":"recheck octavia-v2-dsvm-scenario timeout","commit_id":"324c3528ec7726fb6c69f70473b75b14357ff14a"}],"octavia/amphorae/drivers/haproxy/rest_api_driver.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6f61e23f347905172a886d03b53d166f828dadfd","unresolved":true,"context_lines":[{"line_number":357,"context_line":"        return net_info"},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"    def post_vip_plug(self, amphora, load_balancer, amphorae_network_config,"},{"line_number":360,"context_line":"                      vrrp_port\u003dNone, vip_subnet\u003dNone,"},{"line_number":361,"context_line":"                      additional_vip_data\u003dNone):"},{"line_number":362,"context_line":"        if amphora.status !\u003d consts.DELETED:"},{"line_number":363,"context_line":"            self._populate_amphora_api_version(amphora)"}],"source_content_type":"text/x-python","patch_set":4,"id":"36b2959a_f08dca8a","line":360,"range":{"start_line":360,"start_character":21,"end_line":360,"end_character":53},"updated":"2023-04-05 18:52:23.000000000","message":"This change makes vrrp_port mandatory on line 364, so we should probably remove the None defaults on these. vip_subnet is also now required on line 370.","commit_id":"78ef74155bc0451770500340adea42eef6fe3198"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"93d2a07d3084e0d6abfc0ed0cce7cf716eee77c7","unresolved":false,"context_lines":[{"line_number":357,"context_line":"        return net_info"},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"    def post_vip_plug(self, amphora, load_balancer, amphorae_network_config,"},{"line_number":360,"context_line":"                      vrrp_port\u003dNone, vip_subnet\u003dNone,"},{"line_number":361,"context_line":"                      additional_vip_data\u003dNone):"},{"line_number":362,"context_line":"        if amphora.status !\u003d consts.DELETED:"},{"line_number":363,"context_line":"            self._populate_amphora_api_version(amphora)"}],"source_content_type":"text/x-python","patch_set":4,"id":"574051c3_e10fd221","line":360,"range":{"start_line":360,"start_character":21,"end_line":360,"end_character":53},"in_reply_to":"36b2959a_f08dca8a","updated":"2023-04-07 08:29:13.000000000","message":"done, vrrp_port and vip_subnet are now required in the interface","commit_id":"78ef74155bc0451770500340adea42eef6fe3198"}],"octavia/amphorae/drivers/keepalived/jinja/jinja_cfg.py":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"23411930c66bc34d1c1a6c1b538b15ea591b8858","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        :param loadbalancer: A loadbalancer object"},{"line_number":62,"context_line":"        :param amphora: An amphora object"},{"line_number":63,"context_line":"        :param amp_net_config: The amphora network config, a dict"},{"line_number":64,"context_line":"        \"\"\""},{"line_number":65,"context_line":"        # Note on keepalived configuration: The current base configuration"},{"line_number":66,"context_line":"        # enforced Master election whenever a high priority VRRP instance"}],"source_content_type":"text/x-python","patch_set":5,"id":"3298dd13_fcfee9ce","line":63,"range":{"start_line":63,"start_character":57,"end_line":63,"end_character":65},"updated":"2023-04-12 17:45:57.000000000","message":"This is the wrong place fore type info.","commit_id":"324c3528ec7726fb6c69f70473b75b14357ff14a"}]}
