)]}'
{"ceilometer/publisher/__init__.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"51b8d117d77e719676589b68c2a08acf908875b7","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def __init__(self, namespace):"},{"line_number":27,"context_line":"        super(PublisherExtensionManager, self).__init__("},{"line_number":28,"context_line":"            namespace\u003d\u0027ceilometer.publisher\u0027,"},{"line_number":29,"context_line":"            check_func\u003dlambda x: True,"},{"line_number":30,"context_line":"            invoke_on_load\u003dTrue,"},{"line_number":31,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOX%2F%2F7Ho%3D","line":28,"updated":"2013-04-05 20:28:03.000000000","message":"should this be using namespace param?","commit_id":"2f8a291cef479af1ec224ef4a9d2b297a4d3491b"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"0570eed6b98f7fdb5e2f19b26f0d4f011f4da62e","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def __init__(self, namespace):"},{"line_number":27,"context_line":"        super(PublisherExtensionManager, self).__init__("},{"line_number":28,"context_line":"            namespace\u003d\u0027ceilometer.publisher\u0027,"},{"line_number":29,"context_line":"            check_func\u003dlambda x: True,"},{"line_number":30,"context_line":"            invoke_on_load\u003dTrue,"},{"line_number":31,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOX%2F%2F6g8%3D","line":28,"in_reply_to":"AAAAOX%2F%2F7Ho%3D","updated":"2013-04-07 03:06:43.000000000","message":"Agreed with gordon.","commit_id":"2f8a291cef479af1ec224ef4a9d2b297a4d3491b"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"aca9636557e918a0156d94468fc55d66d85ddc3b","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def __init__(self, namespace):"},{"line_number":27,"context_line":"        super(PublisherExtensionManager, self).__init__("},{"line_number":28,"context_line":"            namespace\u003d\u0027ceilometer.publisher\u0027,"},{"line_number":29,"context_line":"            check_func\u003dlambda x: True,"},{"line_number":30,"context_line":"            invoke_on_load\u003dTrue,"},{"line_number":31,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2F%2B2s%3D","line":28,"in_reply_to":"AAAAOX%2F%2F7Ho%3D","updated":"2013-04-09 08:29:31.000000000","message":"Done","commit_id":"2f8a291cef479af1ec224ef4a9d2b297a4d3491b"}],"tests/agentbase.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"300eda908e7639cfce1768929367ac3459fefc3b","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        self.publisher_manager \u003d publisher.PublisherExtensionManager("},{"line_number":102,"context_line":"            \u0027fake\u0027,"},{"line_number":103,"context_line":"        )"},{"line_number":104,"context_line":"        self.publisher_manager.extensions \u003d ["},{"line_number":105,"context_line":"            extension.Extension("},{"line_number":106,"context_line":"                \u0027test_pub\u0027,"},{"line_number":107,"context_line":"                None,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOX%2F%2F7Zk%3D","line":104,"updated":"2013-04-05 16:51:12.000000000","message":"This should really be replaced with a TestExtensionManager: http://stevedore.readthedocs.org/en/latest/managers.html#testextensionmanager","commit_id":"2f8a291cef479af1ec224ef4a9d2b297a4d3491b"}],"tests/test_pipeline.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"51b8d117d77e719676589b68c2a08acf908875b7","unresolved":false,"context_lines":[{"line_number":99,"context_line":"            \u0027fake\u0027,"},{"line_number":100,"context_line":"        )"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"        self.transformer_manager \u003d transformer.TransformerExtensionManager()"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        self.publisher \u003d self.PublisherClass()"},{"line_number":105,"context_line":"        self.new_publisher \u003d self.PublisherClass()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOX%2F%2F7Gw%3D","line":102,"updated":"2013-04-05 20:28:03.000000000","message":"i assume this is done to initialize transformer_manager at the same time as publisher_manager? not that familiar with dependency between the two managers but looks a bit strange doing this inside _create_publisher_manager.  maybe do this in setup() if its not needed here?","commit_id":"2f8a291cef479af1ec224ef4a9d2b297a4d3491b"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"aca9636557e918a0156d94468fc55d66d85ddc3b","unresolved":false,"context_lines":[{"line_number":99,"context_line":"            \u0027fake\u0027,"},{"line_number":100,"context_line":"        )"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"        self.transformer_manager \u003d transformer.TransformerExtensionManager()"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"        self.publisher \u003d self.PublisherClass()"},{"line_number":105,"context_line":"        self.new_publisher \u003d self.PublisherClass()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2F%2B2k%3D","line":102,"in_reply_to":"AAAAOX%2F%2F7Gw%3D","updated":"2013-04-09 08:29:31.000000000","message":"That\u0027s because PipelineManager used to get a publisher_manager as arg, and created TransformerManager itself. Now both have to be created before and passed as arg. :)\n\nI\u0027ve reorganized this code a bit, and remove the useless ext_name arg.","commit_id":"2f8a291cef479af1ec224ef4a9d2b297a4d3491b"}]}
