)]}'
{"tempest/tests/compute/admin/test_services.py":[{"author":{"_account_id":5586,"name":"Ivan-Zhu","email":"zhubo198511@gmail.com","username":"ivan-zhu"},"change_message_id":"efaa772a57b06cac5e1448041022f994e4ee98cf","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        # Change to disable the status"},{"line_number":50,"context_line":"        resp, service \u003d self.client.is_disable_service(binary, host)"},{"line_number":51,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":52,"context_line":"        self.addCleanup(self.client.is_enable_service, binary, host)"},{"line_number":53,"context_line":"        self.assertEqual(\u0027disabled\u0027, service[\u0027status\u0027])"},{"line_number":54,"context_line":"        self.assertEqual(binary, service[\u0027binary\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2F4go%3D","line":51,"updated":"2013-04-14 00:47:14.000000000","message":"If disable the service, other tests run in other process may be failed.","commit_id":"492e2fa5eb52570b4b80fc655174a6b52cfb7fb3"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"c78fe5d047a85c8a93f336f00fc7bb0490ce052e","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        host \u003d services[0][\u0027host\u0027]"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        # Change to enable the status of the service"},{"line_number":58,"context_line":"        resp, service \u003d self.client.enable_service(binary, host)"},{"line_number":59,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":60,"context_line":"        self.assertEqual(\u0027enabled\u0027, service[\u0027status\u0027])"},{"line_number":61,"context_line":"        self.assertEqual(binary, service[\u0027binary\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAOn%2F%2F1bE%3D","line":58,"updated":"2013-04-19 10:16:41.000000000","message":"If this service was already enabled then it isn\u0027t actually testing anything is it? Also if it was disabled then we should leave it disabled at the end of the test","commit_id":"e0c1299dec94eb146e023b0e22ea47e8bd4cdc3c"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"5b6425cb3380dd01521424b716810f5027a09a46","unresolved":false,"context_lines":[{"line_number":23,"context_line":"class ServicesAdminTestJSON(base.BaseComputeAdminTest):"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    \"\"\""},{"line_number":26,"context_line":"    Tests Services API. List and Enable/Disable require admin privileges."},{"line_number":27,"context_line":"    \"\"\""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    _interface \u003d \u0027json\u0027"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAOn%2F%2FYcM%3D","line":26,"updated":"2013-05-12 10:54:32.000000000","message":"Minor nit - perhaps update this comment?","commit_id":"b7fb86e90392e8cec9514198ba240b28b7e8ecba"}]}