)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"02f0f2e0a8d97aab378e9e0f07717cfe5f007702","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, if the file defined by api_paste_config is not found under"},{"line_number":10,"context_line":"working directory, it will use etc/ceilometer/api_paste.ini instead,"},{"line_number":11,"context_line":"this is because at that time, devstack doesn\u0027t support the new config"},{"line_number":12,"context_line":"file. After patch I57165d588da35f13b2d22629b48dc087502598a7 is merged,"},{"line_number":13,"context_line":"this workaround is no longer needed, and we should use advanced"},{"line_number":14,"context_line":"find_file() provided by oslo.config to search config file."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baada198_e7b2c592","line":12,"updated":"2014-07-07 20:57:13.000000000","message":"I\u0027m confused, I57165d588da35f13b2d22629b48dc087502598a7 was merged before you ever pushed up this ceilometer change.","commit_id":"8077b71f943af5379047cf614a5046c459bbdf8c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b6bb3e81905195cb47a81fdf8b6680086d866ac2","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, if the file defined by api_paste_config is not found under"},{"line_number":10,"context_line":"working directory, it will use etc/ceilometer/api_paste.ini instead,"},{"line_number":11,"context_line":"this is because at that time, devstack doesn\u0027t support the new config"},{"line_number":12,"context_line":"file. After patch I57165d588da35f13b2d22629b48dc087502598a7 is merged,"},{"line_number":13,"context_line":"this workaround is no longer needed, and we should use advanced"},{"line_number":14,"context_line":"find_file() provided by oslo.config to search config file."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baada198_e7600503","line":12,"in_reply_to":"baada198_e7b2c592","updated":"2014-07-07 20:58:43.000000000","message":"Oh nevermind, I was misunderstanding the explanation here, to me it sounded like something else needed to happen in ceilometer after both changes were merged, but it means with the merged devstack change you can remove the workaround in the ceilometer code, which is what this patch does.","commit_id":"8077b71f943af5379047cf614a5046c459bbdf8c"}],"ceilometer/api/app.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"047e1649aa9cc88e8bc60fa0d3630c9b56862158","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            raise cfg.ConfigFilesNotFoundError([cfg.CONF.api_paste_config])"},{"line_number":138,"context_line":"    else:"},{"line_number":139,"context_line":"        cfg_file \u003d os.path.abspath(cfg_file)"},{"line_number":140,"context_line":"    LOG.info(_(\"Full WSGI config used: %s\"), cfg_file)"},{"line_number":141,"context_line":"    return deploy.loadapp(\"config:\" + cfg_file)"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_4700d101","line":140,"updated":"2014-07-07 21:01:08.000000000","message":"This is an unrelated change to the bug fix, and you should be importing and using _LI from gettextutils for the translation domain.","commit_id":"8077b71f943af5379047cf614a5046c459bbdf8c"}]}