)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"fb548e097fc1b4311afc46e794b25e25e5e88bfc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"087598b6_e1d7a337","updated":"2023-08-06 22:51:22.000000000","message":"Unfortunately, we might not be able to make this test work ...","commit_id":"b76833ce52dd3df5c4a80a3b14523653af677a62"}],"cinder/tests/functional/api_sample_tests/test_services.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"fb548e097fc1b4311afc46e794b25e25e5e88bfc","unresolved":true,"context_lines":[{"line_number":96,"context_line":"    def test_service_failover(self):"},{"line_number":97,"context_line":"        response \u003d self._do_get(\u0027os-services\u0027)"},{"line_number":98,"context_line":"        service \u003d jsonutils.loads(response.content)[\u0027services\u0027][0]"},{"line_number":99,"context_line":"        service[\"replication_status\"] \u003d \"enabled\""},{"line_number":100,"context_line":"        sub \u003d {"},{"line_number":101,"context_line":"            \"host\": service[\"host\"],"},{"line_number":102,"context_line":"            \"backend_id\": \"backend\""}],"source_content_type":"text/x-python","patch_set":1,"id":"3e15e14a_8ac8b5c6","line":99,"range":{"start_line":99,"start_character":8,"end_line":99,"end_character":49},"updated":"2023-08-06 22:51:22.000000000","message":"we need this but setting it this way doesn\u0027t work because we are passing it anywhere but fetching the service in the API code\n\nwe need the service replication_status to be \"enabled\" or \"failed_over\"\n\n2023-08-04 12:17:41,225 ERROR [cinder.volume.api] replication_status in [\u002785699154380546b9b729a977536e2c99\u0027] must be [\u0027enabled\u0027, \u0027failed-over\u0027] and must be up to failover.","commit_id":"b76833ce52dd3df5c4a80a3b14523653af677a62"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"fb548e097fc1b4311afc46e794b25e25e5e88bfc","unresolved":true,"context_lines":[{"line_number":101,"context_line":"            \"host\": service[\"host\"],"},{"line_number":102,"context_line":"            \"backend_id\": \"backend\""},{"line_number":103,"context_line":"        }"},{"line_number":104,"context_line":"        res \u003d self._do_put(\u0027os-services/failover\u0027,"},{"line_number":105,"context_line":"                           \u0027services-failover-host-request\u0027, sub)"},{"line_number":106,"context_line":"        self.assertEqual(202, res.status_code)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4274de85_7909642a","line":105,"range":{"start_line":104,"start_character":0,"end_line":105,"end_character":65},"updated":"2023-08-06 22:51:22.000000000","message":"this fails because of the DB update error\n\nhttps://github.com/openstack/cinder/blob/ff4b1c910e65274efcbc0fd052f1f9bc5a643603/cinder/volume/api.py#L2191-L2192","commit_id":"b76833ce52dd3df5c4a80a3b14523653af677a62"}]}