)]}'
{"tempest/api/compute/v3/admin/test_services.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"4c4d20a414c06ca6270271d0d5baba389df30b0b","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_disable_service_with_no_reason(self):"},{"line_number":89,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":90,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":91,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2F3FU%3D","line":88,"updated":"2014-03-13 10:11:00.000000000","message":"Disabling a random service in parallel test run is unhealthy.\n\nIf we really would like to test these we need to create single threaded gate job.","commit_id":"ecfea7cfe163c2fbc686a4ffda6b77acacae5382"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"bb8126426cec40e2c5356921b4bea647d81f992f","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_disable_service_with_no_reason(self):"},{"line_number":89,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":90,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":91,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2Fu34%3D","line":88,"in_reply_to":"AAAAWH%2F%2F3Ag%3D","updated":"2014-03-14 06:06:50.000000000","message":"Now new negative tests(except Neutron tests) should be dropped.\n\nhttp://lists.openstack.org/pipermail/openstack-dev/2013-November/019077.html\n\nAnd we should not disable a service because the other test classes run in parallel at the gate as afazekas said.","commit_id":"ecfea7cfe163c2fbc686a4ffda6b77acacae5382"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"3ba0275190211a0afc21c742405baa6b1713d972","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_disable_service_with_no_reason(self):"},{"line_number":89,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":90,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":91,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2F3Ag%3D","line":88,"in_reply_to":"AAAAWH%2F%2F3FU%3D","updated":"2014-03-13 10:20:30.000000000","message":"So what about just doing enable_services test and the negative test?","commit_id":"ecfea7cfe163c2fbc686a4ffda6b77acacae5382"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"8ce44fb2d913c4ba458844ac39de9257144f40b1","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":88,"context_line":"    def test_disable_service_with_no_reason(self):"},{"line_number":89,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":90,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":91,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2Fsyc%3D","line":88,"in_reply_to":"AAAAWH%2F%2Fu34%3D","updated":"2014-03-14 12:07:50.000000000","message":"Ok, I will add the test_enable_service only.","commit_id":"ecfea7cfe163c2fbc686a4ffda6b77acacae5382"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5ced639e047b6f49a04165c843f21c3864a66ef4","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":81,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":82,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"},{"line_number":83,"context_line":"        resp, body \u003d self.client.enable_service(host_name, binary_name)"},{"line_number":84,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2Bz6k%3D","line":83,"updated":"2014-03-24 05:51:38.000000000","message":"This test seems a little dangerous because this test would enable a service what an operator has disabled.\n\nHow about adding configuration item which is \u0027not test\u0027 as the default value?","commit_id":"0d295084a436fa9b21ab54ba0eaa65f4e3eb0544"},{"author":{"_account_id":6348,"name":"xu-haiwei","email":"hai-xu@xr.jp.nec.com","username":"xu-haiwei"},"change_message_id":"f00b4e598ef55f6bad522569bd13e5eb51107813","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":81,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":82,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"},{"line_number":83,"context_line":"        resp, body \u003d self.client.enable_service(host_name, binary_name)"},{"line_number":84,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2BE0g%3D","line":83,"in_reply_to":"AAAAWH%2F%2BE2s%3D","updated":"2014-03-31 03:03:25.000000000","message":"So I think we should abandon this test.","commit_id":"0d295084a436fa9b21ab54ba0eaa65f4e3eb0544"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0cd3babbcb0a684c3ca51891b132bf0232ff0dab","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        resp, services \u003d self.client.list_services()"},{"line_number":81,"context_line":"        host_name \u003d services[0][\u0027host\u0027]"},{"line_number":82,"context_line":"        binary_name \u003d services[0][\u0027binary\u0027]"},{"line_number":83,"context_line":"        resp, body \u003d self.client.enable_service(host_name, binary_name)"},{"line_number":84,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":85,"context_line":"        self.assertEqual(\u0027enabled\u0027, body[\u0027status\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2BE2s%3D","line":83,"in_reply_to":"AAAAWH%2F%2Bz6k%3D","updated":"2014-03-31 02:49:16.000000000","message":"This kind of test has been removed once. Please see Ib681b393944b37c720e7a245e8b19b5254d116fd .","commit_id":"0d295084a436fa9b21ab54ba0eaa65f4e3eb0544"}]}