)]}'
{"tests/api/v2/test_app.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"15d1bfaded0093f689599123550779c7acdc6a87","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        cfg.CONF.reset()"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_keystone_middleware_conf(self):"},{"line_number":42,"context_line":"        cfg.CONF.set_override(\"auth_protocol\", \"foottp\","},{"line_number":43,"context_line":"                              group\u003dacl.OPT_GROUP_NAME)"},{"line_number":44,"context_line":"        cfg.CONF.set_override(\"auth_version\", \"v2.0\", group\u003dacl.OPT_GROUP_NAME)"},{"line_number":45,"context_line":"        cfg.CONF.set_override(\"pipeline_cfg_file\","}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2Fbl4%3D","line":42,"updated":"2013-09-26 19:08:57.000000000","message":"so the point of the test is to verify that the keystoneclient conf opts set in our ceilometer conf are read properly.  we should set auth_uri here instead of auth_protocol to prove this.","commit_id":"4286ba5cce353de010bd98ed4a59b2fbb368f226"},{"author":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"change_message_id":"3fcf1ef55c42eeb6f2943d185590a2ccb795d379","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        cfg.CONF.reset()"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_keystone_middleware_conf(self):"},{"line_number":42,"context_line":"        cfg.CONF.set_override(\"auth_protocol\", \"foottp\","},{"line_number":43,"context_line":"                              group\u003dacl.OPT_GROUP_NAME)"},{"line_number":44,"context_line":"        cfg.CONF.set_override(\"auth_version\", \"v2.0\", group\u003dacl.OPT_GROUP_NAME)"},{"line_number":45,"context_line":"        cfg.CONF.set_override(\"pipeline_cfg_file\","}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2FbZ8%3D","line":42,"in_reply_to":"AAAAS3%2F%2Fbl4%3D","updated":"2013-09-26 20:22:30.000000000","message":"the point is to test the protocol is is used. at least that is the intent of the\noriginal test case. auth_uri gets set if it is none. otherwise, it uses the value  as is.","commit_id":"4286ba5cce353de010bd98ed4a59b2fbb368f226"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"15d1bfaded0093f689599123550779c7acdc6a87","unresolved":false,"context_lines":[{"line_number":55,"context_line":"            f.write(\"pipeline_cfg_file \u003d %s\\n\" %"},{"line_number":56,"context_line":"                    self.path_get(\"etc/ceilometer/pipeline.yaml\"))"},{"line_number":57,"context_line":"            f.write(\"[%s]\\n\" % acl.OPT_GROUP_NAME)"},{"line_number":58,"context_line":"            f.write(\"auth_protocol \u003d barttp\\n\")"},{"line_number":59,"context_line":"            f.write(\"auth_version \u003d v2.0\\n\")"},{"line_number":60,"context_line":"        service.prepare_service([\u0027ceilometer-api\u0027,"},{"line_number":61,"context_line":"                                 \u0027--config-file\u003d%s\u0027 % tmpfile])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2Fblw%3D","line":58,"updated":"2013-09-26 19:08:57.000000000","message":"same as above","commit_id":"4286ba5cce353de010bd98ed4a59b2fbb368f226"},{"author":{"_account_id":2860,"name":"litong01","email":"email4tong@yahoo.com","username":"litong01"},"change_message_id":"3fcf1ef55c42eeb6f2943d185590a2ccb795d379","unresolved":false,"context_lines":[{"line_number":55,"context_line":"            f.write(\"pipeline_cfg_file \u003d %s\\n\" %"},{"line_number":56,"context_line":"                    self.path_get(\"etc/ceilometer/pipeline.yaml\"))"},{"line_number":57,"context_line":"            f.write(\"[%s]\\n\" % acl.OPT_GROUP_NAME)"},{"line_number":58,"context_line":"            f.write(\"auth_protocol \u003d barttp\\n\")"},{"line_number":59,"context_line":"            f.write(\"auth_version \u003d v2.0\\n\")"},{"line_number":60,"context_line":"        service.prepare_service([\u0027ceilometer-api\u0027,"},{"line_number":61,"context_line":"                                 \u0027--config-file\u003d%s\u0027 % tmpfile])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2FbZ4%3D","line":58,"in_reply_to":"AAAAS3%2F%2Fblw%3D","updated":"2013-09-26 20:22:30.000000000","message":"The config is read from the temp file which contains no auth_uri, we should\nnot do it here.","commit_id":"4286ba5cce353de010bd98ed4a59b2fbb368f226"}]}