)]}'
{"tempest/api/compute/admin/test_services.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"836f26d617740ba8297a6b2baab448f57b1c4a57","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        resp, services \u003d self.client.list_services(params)"},{"line_number":150,"context_line":"        self.assertEqual(\u0027disabled\u0027, services[0][\u0027status\u0027])"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"        resp, service \u003d self.client.enable_service(host_name, binary_name)"},{"line_number":153,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":154,"context_line":"        resp, services \u003d self.client.list_services(params)"},{"line_number":155,"context_line":"        self.assertEqual(\u0027enabled\u0027, services[0][\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FZtc%3D","line":152,"updated":"2013-11-07 06:42:18.000000000","message":"It\u0027s better to use addCleanup to enable service.\nAnd I have a concern that if this testcases impact others as the service would be disabled.","commit_id":"c704b6c95a289bb2f4ab03af862c3384ba686914"},{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"e60c113864ff29f1e871a8881b5f896ddf776b0a","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        resp, services \u003d self.client.list_services(params)"},{"line_number":150,"context_line":"        self.assertEqual(\u0027disabled\u0027, services[0][\u0027status\u0027])"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"        resp, service \u003d self.client.enable_service(host_name, binary_name)"},{"line_number":153,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":154,"context_line":"        resp, services \u003d self.client.list_services(params)"},{"line_number":155,"context_line":"        self.assertEqual(\u0027enabled\u0027, services[0][\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FZp0%3D","line":152,"in_reply_to":"AAAATn%2F%2FZtc%3D","updated":"2013-11-07 07:03:18.000000000","message":"hi, thanks for your reivew. \nHere I want to check the disabled service has been enabled explicitly. Because I also  concern that if this testcase impact others as the service would be disabled.\n\nActually, I think you can help to answer this question :). Because there is the same problem with the above test: test_service_enable_disable , and it was added by you in this commit: d42c991c46a663bdda1260fec851600230a3c128","commit_id":"c704b6c95a289bb2f4ab03af862c3384ba686914"}]}