)]}'
{"ceilometer/agent.py":[{"author":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"change_message_id":"ba562c9bf499c8b5942bece4492ac37a462b5078","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                cfg.CONF.counter_source,"},{"line_number":49,"context_line":"            )"},{"line_number":50,"context_line":"            with publisher as p:"},{"line_number":51,"context_line":"                LOG.info(\u0027Polling and publishing %s\u0027, ext.name)"},{"line_number":52,"context_line":"                p(ext.obj.get_counters(manager, *args, **kwargs))"},{"line_number":53,"context_line":"        except Exception as err:"},{"line_number":54,"context_line":"            LOG.warning(\u0027Continuing after error from %s: %s\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAM3%2F%2Bvr4%3D","line":51,"updated":"2013-02-20 02:36:45.000000000","message":"Any reason the log level has gone from debug to info?\nit will probably fill up the log quickly.","commit_id":"f2bc72c2bddbc12b02e079e041a75adf92d72a4d"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"8fca0d88fc68eb5f4384f788485e276f5248590d","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                cfg.CONF.counter_source,"},{"line_number":49,"context_line":"            )"},{"line_number":50,"context_line":"            with publisher as p:"},{"line_number":51,"context_line":"                LOG.info(\u0027Polling and publishing %s\u0027, ext.name)"},{"line_number":52,"context_line":"                p(ext.obj.get_counters(manager, *args, **kwargs))"},{"line_number":53,"context_line":"        except Exception as err:"},{"line_number":54,"context_line":"            LOG.warning(\u0027Continuing after error from %s: %s\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAM3%2F%2BvGE%3D","line":51,"in_reply_to":"AAAAM3%2F%2Bvr4%3D","updated":"2013-02-20 10:17:39.000000000","message":"No, I mixed both the 2 lines (1 info and 1 debug), but I\u0027ll change.","commit_id":"f2bc72c2bddbc12b02e079e041a75adf92d72a4d"}],"ceilometer/pipeline.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"14cea6eea8967fe16a13d052b193cb050b797eb1","unresolved":false,"context_lines":[{"line_number":74,"context_line":"    def __enter__(self):"},{"line_number":75,"context_line":"        def p(counters):"},{"line_number":76,"context_line":"            for p in self.pipelines:"},{"line_number":77,"context_line":"                p.publish_counters(self.context,"},{"line_number":78,"context_line":"                                   counters,"},{"line_number":79,"context_line":"                                   self.source)"},{"line_number":80,"context_line":"        return p"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BzSc%3D","line":77,"updated":"2013-02-18 23:02:50.000000000","message":"Does publish_counters() catch all exceptions?","commit_id":"c356343dd2229b8f48005acc493b24d7380bdb70"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"879cdb36bd50721841ade04691a17670f52e4d2a","unresolved":false,"context_lines":[{"line_number":74,"context_line":"    def __enter__(self):"},{"line_number":75,"context_line":"        def p(counters):"},{"line_number":76,"context_line":"            for p in self.pipelines:"},{"line_number":77,"context_line":"                p.publish_counters(self.context,"},{"line_number":78,"context_line":"                                   counters,"},{"line_number":79,"context_line":"                                   self.source)"},{"line_number":80,"context_line":"        return p"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2ByS4%3D","line":77,"in_reply_to":"AAAAM3%2F%2BzSc%3D","updated":"2013-02-19 12:43:43.000000000","message":"Yes in _publish_counters_to_one_publisher it\u0027s catched","commit_id":"c356343dd2229b8f48005acc493b24d7380bdb70"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"14cea6eea8967fe16a13d052b193cb050b797eb1","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def __exit__(self, exc_type, exc_value, traceback):"},{"line_number":83,"context_line":"        for p in self.pipelines:"},{"line_number":84,"context_line":"            p.flush(self.context, self.source)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"class Pipeline(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BzSY%3D","line":84,"updated":"2013-02-18 23:02:50.000000000","message":"Does flush() catch all exceptions?","commit_id":"c356343dd2229b8f48005acc493b24d7380bdb70"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"879cdb36bd50721841ade04691a17670f52e4d2a","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def __exit__(self, exc_type, exc_value, traceback):"},{"line_number":83,"context_line":"        for p in self.pipelines:"},{"line_number":84,"context_line":"            p.flush(self.context, self.source)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"class Pipeline(object):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BySg%3D","line":84,"in_reply_to":"AAAAM3%2F%2BzSY%3D","updated":"2013-02-19 12:43:43.000000000","message":"Done","commit_id":"c356343dd2229b8f48005acc493b24d7380bdb70"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"aadd4c6bd47b8514719f378ff192d1a5bf0521ba","unresolved":false,"context_lines":[{"line_number":74,"context_line":"    def __enter__(self):"},{"line_number":75,"context_line":"        def p(counters):"},{"line_number":76,"context_line":"            for p in self.pipelines:"},{"line_number":77,"context_line":"                p.publish_counters(self.context,"},{"line_number":78,"context_line":"                                   counters,"},{"line_number":79,"context_line":"                                   self.source)"},{"line_number":80,"context_line":"        return p"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2ByEQ%3D","line":77,"updated":"2013-02-19 12:05:28.000000000","message":"Do we need to have any exception handling here?","commit_id":"a841b65ab7cdebc441ee8710d9de24478279f75f"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"aadd4c6bd47b8514719f378ff192d1a5bf0521ba","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def __exit__(self, exc_type, exc_value, traceback):"},{"line_number":83,"context_line":"        for p in self.pipelines:"},{"line_number":84,"context_line":"            p.flush(self.context, self.source)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"class Pipeline(object):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2ByEM%3D","line":84,"updated":"2013-02-19 12:05:28.000000000","message":"Do we need to have any exception handling here?","commit_id":"a841b65ab7cdebc441ee8710d9de24478279f75f"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"aadd4c6bd47b8514719f378ff192d1a5bf0521ba","unresolved":false,"context_lines":[{"line_number":280,"context_line":"        LOG.audit(\"Flush pipeline %s\", self)"},{"line_number":281,"context_line":"        for (i, transformer) in enumerate(self.transformers):"},{"line_number":282,"context_line":"            try:"},{"line_number":283,"context_line":"                self._publish_counters(i + 1, ctxt,"},{"line_number":284,"context_line":"                                       list(transformer.flush(ctxt, source)),"},{"line_number":285,"context_line":"                                       source)"},{"line_number":286,"context_line":"            except Exception as err:"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2ByEU%3D","line":283,"updated":"2013-02-19 12:05:28.000000000","message":"If you change the call to enumerate() above to add a second argument 1, it will start counting from 1 and you don\u0027t have to do math here.","commit_id":"a841b65ab7cdebc441ee8710d9de24478279f75f"}]}