)]}'
{"ceilometer/pipeline.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"66ab1d595b58d0fa2a75f5f66325b31f322fc340","unresolved":false,"context_lines":[{"line_number":625,"context_line":"    def __init__(self, cfg_info):"},{"line_number":626,"context_line":"        \"\"\"Accept both file location or dictionary of pipeline.\"\"\""},{"line_number":627,"context_line":"        LOG.info(\"Config file: %s\", cfg_info)"},{"line_number":628,"context_line":"        if isinstance(cfg_info, dict):"},{"line_number":629,"context_line":"            self.cfg_file \u003d None"},{"line_number":630,"context_line":"        else:"},{"line_number":631,"context_line":"            if not os.path.exists(cfg_info):"},{"line_number":632,"context_line":"                cfg_info \u003d cfg.CONF.find_file(cfg_info)"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_4362693f","line":629,"range":{"start_line":628,"start_character":8,"end_line":629,"end_character":32},"updated":"2016-08-29 13:50:35.000000000","message":"This looks so weird. Like you don\u0027t care about cfg_info?","commit_id":"b1d623f6dc575b2d793ecba12396eb053d65deb6"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"4f989b348cddeef68de401f1e89e4d1b6bd0b53e","unresolved":false,"context_lines":[{"line_number":625,"context_line":"    def __init__(self, cfg_info):"},{"line_number":626,"context_line":"        \"\"\"Accept both file location or dictionary of pipeline.\"\"\""},{"line_number":627,"context_line":"        LOG.info(\"Config file: %s\", cfg_info)"},{"line_number":628,"context_line":"        if isinstance(cfg_info, dict):"},{"line_number":629,"context_line":"            self.cfg_file \u003d None"},{"line_number":630,"context_line":"        else:"},{"line_number":631,"context_line":"            if not os.path.exists(cfg_info):"},{"line_number":632,"context_line":"                cfg_info \u003d cfg.CONF.find_file(cfg_info)"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_f1fb39bf","line":629,"range":{"start_line":628,"start_character":8,"end_line":629,"end_character":32},"in_reply_to":"fa7ab95a_4362693f","updated":"2016-08-29 17:01:17.000000000","message":"i\u0027ll change it to cfg_loc. basically, we accept either a config file or a dict of configurations. if the former, we support reloading","commit_id":"b1d623f6dc575b2d793ecba12396eb053d65deb6"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"66ab1d595b58d0fa2a75f5f66325b31f322fc340","unresolved":false,"context_lines":[{"line_number":634,"context_line":""},{"line_number":635,"context_line":"    def load_config(self):"},{"line_number":636,"context_line":"        \"\"\"Load a configuration file and set its refresh values.\"\"\""},{"line_number":637,"context_line":"        LOG.info(\"Config file: %s\", self.cfg_file)"},{"line_number":638,"context_line":"        with open(self.cfg_file) as fap:"},{"line_number":639,"context_line":"            data \u003d fap.read()"},{"line_number":640,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_c356795b","line":637,"updated":"2016-08-29 13:50:35.000000000","message":"This is the same LOG as line 627 so likely confusing","commit_id":"b1d623f6dc575b2d793ecba12396eb053d65deb6"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"4f989b348cddeef68de401f1e89e4d1b6bd0b53e","unresolved":false,"context_lines":[{"line_number":634,"context_line":""},{"line_number":635,"context_line":"    def load_config(self):"},{"line_number":636,"context_line":"        \"\"\"Load a configuration file and set its refresh values.\"\"\""},{"line_number":637,"context_line":"        LOG.info(\"Config file: %s\", self.cfg_file)"},{"line_number":638,"context_line":"        with open(self.cfg_file) as fap:"},{"line_number":639,"context_line":"            data \u003d fap.read()"},{"line_number":640,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_11f97dc7","line":637,"in_reply_to":"fa7ab95a_c356795b","updated":"2016-08-29 17:01:17.000000000","message":"Done","commit_id":"b1d623f6dc575b2d793ecba12396eb053d65deb6"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"66ab1d595b58d0fa2a75f5f66325b31f322fc340","unresolved":false,"context_lines":[{"line_number":643,"context_line":"        self.pipeline_hash \u003d self.get_pipeline_hash()"},{"line_number":644,"context_line":"        return conf"},{"line_number":645,"context_line":""},{"line_number":646,"context_line":"    def get_pipeline_mtime(self):"},{"line_number":647,"context_line":"        \"\"\"Return modification time of cfg file\"\"\""},{"line_number":648,"context_line":"        return os.path.getmtime(self.cfg_file) if self.cfg_file else None"},{"line_number":649,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_03e4e1a9","line":646,"range":{"start_line":646,"start_character":8,"end_line":646,"end_character":26},"updated":"2016-08-29 13:50:35.000000000","message":"Why this is talking about pipeline whereas  this class does not talk about pipeline but just a a config file that is reloaded some time.\n\nMake it truly generic! :)","commit_id":"b1d623f6dc575b2d793ecba12396eb053d65deb6"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"4f989b348cddeef68de401f1e89e4d1b6bd0b53e","unresolved":false,"context_lines":[{"line_number":643,"context_line":"        self.pipeline_hash \u003d self.get_pipeline_hash()"},{"line_number":644,"context_line":"        return conf"},{"line_number":645,"context_line":""},{"line_number":646,"context_line":"    def get_pipeline_mtime(self):"},{"line_number":647,"context_line":"        \"\"\"Return modification time of cfg file\"\"\""},{"line_number":648,"context_line":"        return os.path.getmtime(self.cfg_file) if self.cfg_file else None"},{"line_number":649,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_310201b5","line":646,"range":{"start_line":646,"start_character":8,"end_line":646,"end_character":26},"in_reply_to":"fa7ab95a_03e4e1a9","updated":"2016-08-29 17:01:17.000000000","message":"Done","commit_id":"b1d623f6dc575b2d793ecba12396eb053d65deb6"}]}