)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"490810c38b72017acb94ff68df1314a01e0556a8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update the default log levels"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Add the \u0027futurist\u003dINFO\u0027 to the default log levels to disable the priodic"},{"line_number":10,"context_line":"  debug log messages emitted by futurist."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- Clean the stevedore and keystoneclient(keystoneauth1) log levels hooking"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_38c81d22","line":9,"range":{"start_line":9,"start_character":20,"end_line":9,"end_character":25},"updated":"2016-04-26 09:00:05.000000000","message":"or warn?","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"d23bef111536ce76aed645dacb32f74ac0ef5b2b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update the default log levels"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Add the \u0027futurist\u003dINFO\u0027 to the default log levels to disable the priodic"},{"line_number":10,"context_line":"  debug log messages emitted by futurist."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- Clean the stevedore and keystoneclient(keystoneauth1) log levels hooking"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_aca19296","line":9,"range":{"start_line":9,"start_character":20,"end_line":9,"end_character":25},"in_reply_to":"1a122d0e_38c81d22","updated":"2016-04-26 09:29:42.000000000","message":"Hmmm, I cannot find any noise after I set it to INFO","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"c1d4544560e03519f385a9952a0a9c0a52c767db","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update the default log levels"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Add the \u0027futurist\u003dINFO\u0027 to the default log levels to disable the priodic"},{"line_number":10,"context_line":"  debug log messages emitted by futurist."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- Clean the stevedore and keystoneclient(keystoneauth1) log levels hooking"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_ec00aa04","line":9,"range":{"start_line":9,"start_character":20,"end_line":9,"end_character":25},"in_reply_to":"1a122d0e_aca19296","updated":"2016-04-26 09:39:07.000000000","message":"good news","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"490810c38b72017acb94ff68df1314a01e0556a8","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  because they have been included in oslo.log config options[1]."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[1]"},{"line_number":16,"context_line":"https://github.com/openstack/oslo.log/blob/master/oslo_log/_options.py#L19-L30"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I8024fdc202be88fb25a82bd68dc3548b141acd70"},{"line_number":19,"context_line":"Closes-Bug: #1575035"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_78ce152e","line":16,"range":{"start_line":16,"start_character":42,"end_line":16,"end_character":50},"updated":"2016-04-26 09:00:05.000000000","message":"we do not use the master code, the lowest 1.2.0 doesn\u0027t have this, bump the version as well, or leave it untouched","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"d23bef111536ce76aed645dacb32f74ac0ef5b2b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  because they have been included in oslo.log config options[1]."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[1]"},{"line_number":16,"context_line":"https://github.com/openstack/oslo.log/blob/master/oslo_log/_options.py#L19-L30"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I8024fdc202be88fb25a82bd68dc3548b141acd70"},{"line_number":19,"context_line":"Closes-Bug: #1575035"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_2c958232","line":16,"range":{"start_line":16,"start_character":42,"end_line":16,"end_character":50},"in_reply_to":"1a122d0e_78ce152e","updated":"2016-04-26 09:29:42.000000000","message":"yeah, I prefer to bump the version of oslo.log","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"9b1bb846140491d28f80b423308e4bccf09dc0b7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"- Add the \u0027futurist\u003dINFO\u0027 to the default log levels to disable the priodic"},{"line_number":10,"context_line":"  debug log messages emitted by futurist."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- Clean the stevedore because it has been included in oslo.log config options[1]"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- Bump the version of oslo.log to 1.14.0 for above change."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"[1]"},{"line_number":17,"context_line":"https://github.com/openstack/oslo.log/blob/1.14.0/oslo_log/_options.py#L110-L118"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a122d0e_207583f9","line":14,"range":{"start_line":12,"start_character":2,"end_line":14,"end_character":58},"updated":"2016-04-26 12:01:37.000000000","message":"not true: https://github.com/openstack/oslo.log/blob/1.2.0/oslo_log/_options.py#L97","commit_id":"70c752c634121c85112fec7ca30137926b916414"}],"aodh/service.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"490810c38b72017acb94ff68df1314a01e0556a8","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    oslo_i18n.enable_lazy()"},{"line_number":51,"context_line":"    log.register_options(conf)"},{"line_number":52,"context_line":"    log_levels \u003d (conf.default_log_levels +"},{"line_number":53,"context_line":"                  [\u0027stevedore\u003dINFO\u0027, \u0027keystoneclient\u003dINFO\u0027])"},{"line_number":54,"context_line":"    log.set_defaults(default_log_levels\u003dlog_levels)"},{"line_number":55,"context_line":"    defaults.set_cors_middleware_defaults()"},{"line_number":56,"context_line":"    db_options.set_defaults(conf)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_f874e5fe","side":"PARENT","line":53,"range":{"start_line":53,"start_character":37,"end_line":53,"end_character":58},"updated":"2016-04-26 09:00:05.000000000","message":"why remove keystoneclient?","commit_id":"0eb45a195700fa1e0d29b1d6d2a4d7eb82828292"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"490810c38b72017acb94ff68df1314a01e0556a8","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    conf \u003d cfg.ConfigOpts()"},{"line_number":50,"context_line":"    oslo_i18n.enable_lazy()"},{"line_number":51,"context_line":"    log.register_options(conf)"},{"line_number":52,"context_line":"    log_levels \u003d (conf.default_log_levels + [\u0027futurist\u003dINFO\u0027])"},{"line_number":53,"context_line":"    log.set_defaults(default_log_levels\u003dlog_levels)"},{"line_number":54,"context_line":"    defaults.set_cors_middleware_defaults()"},{"line_number":55,"context_line":"    db_options.set_defaults(conf)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_d871290e","line":52,"range":{"start_line":52,"start_character":55,"end_line":52,"end_character":59},"updated":"2016-04-26 09:00:05.000000000","message":"INFO is ok? may be warn?","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"696166d2aef54c23cddde56ec0b1ce8b4317b5e1","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    conf \u003d cfg.ConfigOpts()"},{"line_number":50,"context_line":"    oslo_i18n.enable_lazy()"},{"line_number":51,"context_line":"    log.register_options(conf)"},{"line_number":52,"context_line":"    log_levels \u003d (conf.default_log_levels + [\u0027futurist\u003dINFO\u0027])"},{"line_number":53,"context_line":"    log.set_defaults(default_log_levels\u003dlog_levels)"},{"line_number":54,"context_line":"    defaults.set_cors_middleware_defaults()"},{"line_number":55,"context_line":"    db_options.set_defaults(conf)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_cf55e050","line":52,"range":{"start_line":52,"start_character":55,"end_line":52,"end_character":59},"in_reply_to":"1a122d0e_8c0f7638","updated":"2016-04-26 11:09:27.000000000","message":"agree :)","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"d23bef111536ce76aed645dacb32f74ac0ef5b2b","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    conf \u003d cfg.ConfigOpts()"},{"line_number":50,"context_line":"    oslo_i18n.enable_lazy()"},{"line_number":51,"context_line":"    log.register_options(conf)"},{"line_number":52,"context_line":"    log_levels \u003d (conf.default_log_levels + [\u0027futurist\u003dINFO\u0027])"},{"line_number":53,"context_line":"    log.set_defaults(default_log_levels\u003dlog_levels)"},{"line_number":54,"context_line":"    defaults.set_cors_middleware_defaults()"},{"line_number":55,"context_line":"    db_options.set_defaults(conf)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_ec71ca0d","line":52,"range":{"start_line":52,"start_character":55,"end_line":52,"end_character":59},"in_reply_to":"1a122d0e_d871290e","updated":"2016-04-26 09:29:42.000000000","message":"For now, we use ksytoneauth1 as auth client, I guess the reason why keystoneclient is here is because the frequently authorization cause api requests log noises. for aodh, the keystoneclient may be used to create/delete trust, so does this still needed ? not very sure.","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"c1d4544560e03519f385a9952a0a9c0a52c767db","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    conf \u003d cfg.ConfigOpts()"},{"line_number":50,"context_line":"    oslo_i18n.enable_lazy()"},{"line_number":51,"context_line":"    log.register_options(conf)"},{"line_number":52,"context_line":"    log_levels \u003d (conf.default_log_levels + [\u0027futurist\u003dINFO\u0027])"},{"line_number":53,"context_line":"    log.set_defaults(default_log_levels\u003dlog_levels)"},{"line_number":54,"context_line":"    defaults.set_cors_middleware_defaults()"},{"line_number":55,"context_line":"    db_options.set_defaults(conf)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_8c0f7638","line":52,"range":{"start_line":52,"start_character":55,"end_line":52,"end_character":59},"in_reply_to":"1a122d0e_ec71ca0d","updated":"2016-04-26 09:39:07.000000000","message":"we are using its code, not worth to take too much time just to figure out when it will log or not inside it, or what it will be in next release, just leave it there, when we remove that module, then we remove it","commit_id":"4ae729be0d48a548d7e210a5f25b1520dd3f71df"}],"requirements.txt":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"46cd50e97c3f83b6f00c38a5edf078b327a0ccac","unresolved":false,"context_lines":[{"line_number":13,"context_line":"oslo.db\u003e\u003d1.12.0 # Apache-2.0"},{"line_number":14,"context_line":"oslo.config\u003e\u003d2.6.0 # Apache-2.0"},{"line_number":15,"context_line":"oslo.i18n\u003e\u003d1.5.0 # Apache-2.0"},{"line_number":16,"context_line":"oslo.log\u003e\u003d1.14.0 # Apache-2.0"},{"line_number":17,"context_line":"oslo.policy\u003e\u003d0.5.0 # Apache-2.0"},{"line_number":18,"context_line":"oslo.service\u003e\u003d0.1.0 # Apache-2.0"},{"line_number":19,"context_line":"PasteDeploy\u003e\u003d1.5.0"}],"source_content_type":"text/plain","patch_set":2,"id":"1a122d0e_40187fd3","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":29},"updated":"2016-04-26 12:00:31.000000000","message":"I think it is unrelated now, we can leave it there, until something breaks it or must bumps it","commit_id":"70c752c634121c85112fec7ca30137926b916414"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"78ac4e9d08f9965fe4f503b32e5b70eb77b66081","unresolved":false,"context_lines":[{"line_number":13,"context_line":"oslo.db\u003e\u003d1.12.0 # Apache-2.0"},{"line_number":14,"context_line":"oslo.config\u003e\u003d2.6.0 # Apache-2.0"},{"line_number":15,"context_line":"oslo.i18n\u003e\u003d1.5.0 # Apache-2.0"},{"line_number":16,"context_line":"oslo.log\u003e\u003d1.14.0 # Apache-2.0"},{"line_number":17,"context_line":"oslo.policy\u003e\u003d0.5.0 # Apache-2.0"},{"line_number":18,"context_line":"oslo.service\u003e\u003d0.1.0 # Apache-2.0"},{"line_number":19,"context_line":"PasteDeploy\u003e\u003d1.5.0"}],"source_content_type":"text/plain","patch_set":2,"id":"1a122d0e_8dde3e0f","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":29},"in_reply_to":"1a122d0e_40187fd3","updated":"2016-04-26 22:06:42.000000000","message":"yeah that does not seem right","commit_id":"70c752c634121c85112fec7ca30137926b916414"}]}