)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"11bf1a8120390ca47116e26db999019e7d2b4e68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"069a8371_e51aa03b","updated":"2024-08-21 23:46:58.000000000","message":"recheck\nPillow failed to build","commit_id":"c7ae72fe56d6660768ca479d6278879a4734f0b1"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"b5f27e97e97210ecc9952167c0ba35c7f98a3014","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cf24cfdf_e8c4efad","updated":"2024-08-22 22:35:09.000000000","message":"rebased, so re-applying the +w","commit_id":"a4317a80d61ce2d8e699ea7ca17f161ecc4db8ee"}],"octavia/tests/unit/controller/worker/v2/flows/test_load_balancer_flows.py":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"4979b9d12e1c2c94bc7338a4517525fa3c3731ff","unresolved":true,"context_lines":[{"line_number":344,"context_line":"        self.assertIn(constants.FLAVOR, failover_flow.requires)"},{"line_number":345,"context_line":"        self.assertIn(constants.LOADBALANCER, failover_flow.requires)"},{"line_number":346,"context_line":"        self.assertIn(constants.LOADBALANCER_ID, failover_flow.requires)"},{"line_number":347,"context_line":"        self.assertIn(constants.AMPHORAE_STATUS, failover_flow.requires)"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.assertIn(constants.UPDATED_PORTS, failover_flow.provides)"},{"line_number":350,"context_line":"        self.assertIn(constants.AMPHORA, failover_flow.provides)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e9575bd2_44a03c6e","side":"PARENT","line":347,"updated":"2024-06-05 16:54:19.000000000","message":"I don\u0027t understand where this comes from. This change only seems to add a new task.","commit_id":"b43620cfa8e509647f99835e6eb43416a7f214f8"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"97f4d6604b9fe6b0ad4962bcb986376bd4964757","unresolved":true,"context_lines":[{"line_number":344,"context_line":"        self.assertIn(constants.FLAVOR, failover_flow.requires)"},{"line_number":345,"context_line":"        self.assertIn(constants.LOADBALANCER, failover_flow.requires)"},{"line_number":346,"context_line":"        self.assertIn(constants.LOADBALANCER_ID, failover_flow.requires)"},{"line_number":347,"context_line":"        self.assertIn(constants.AMPHORAE_STATUS, failover_flow.requires)"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.assertIn(constants.UPDATED_PORTS, failover_flow.provides)"},{"line_number":350,"context_line":"        self.assertIn(constants.AMPHORA, failover_flow.provides)"}],"source_content_type":"text/x-python","patch_set":1,"id":"939982d0_b540f492","side":"PARENT","line":347,"in_reply_to":"4c817674_2528aed7","updated":"2024-06-11 07:36:34.000000000","message":"Thanks for the explanation. Yes, checking this is in provides now seems like a good idea then.","commit_id":"b43620cfa8e509647f99835e6eb43416a7f214f8"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"6c7cf78f0f93ccb2ed6dbc5b82723337dd36bf83","unresolved":true,"context_lines":[{"line_number":344,"context_line":"        self.assertIn(constants.FLAVOR, failover_flow.requires)"},{"line_number":345,"context_line":"        self.assertIn(constants.LOADBALANCER, failover_flow.requires)"},{"line_number":346,"context_line":"        self.assertIn(constants.LOADBALANCER_ID, failover_flow.requires)"},{"line_number":347,"context_line":"        self.assertIn(constants.AMPHORAE_STATUS, failover_flow.requires)"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self.assertIn(constants.UPDATED_PORTS, failover_flow.provides)"},{"line_number":350,"context_line":"        self.assertIn(constants.AMPHORA, failover_flow.provides)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4c817674_2528aed7","side":"PARENT","line":347,"in_reply_to":"e9575bd2_44a03c6e","updated":"2024-06-05 19:54:58.000000000","message":"amphorae_status in requires was incorrect (it means that the value must be set externally - by the caller)\nwith this fix, amphorae_status is provided by a task of the flow (AmphoraeGetConnectivityStatus)\nso it\u0027s ok to remove it from \u0027requires\u0027, but we could also check that it\u0027s included in \u0027provides\u0027","commit_id":"b43620cfa8e509647f99835e6eb43416a7f214f8"}]}