)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"dc7182e86fc02656628d6308dfe39b23d1710b4a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We want a wsgi_script as the entry point for our placement API"},{"line_number":10,"context_line":"actually getting run, this is wrapping in the smooth and mellow pbr"},{"line_number":11,"context_line":"patterns that make it sensible to also run just by starting it on the"},{"line_number":12,"context_line":"command line."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This also actually initializes the logging subsystem for the placement"},{"line_number":15,"context_line":"API, and does the standard pattern of dumping the configuration if"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa7ab95a_324b5cb2","line":12,"updated":"2016-08-31 12:40:11.000000000","message":"Yeah, that was a pbr feature I wasn\u0027t knowing, that looks great.","commit_id":"d957bfb9f137610d6f28ec3f63d85a5584a2ba62"}],"nova/api/openstack/placement/wsgi.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"51b1b9e082558ac028f978f290a7ef75ced74de7","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    # Any dependent libraries that have unhelp debug levels should be"},{"line_number":31,"context_line":"    # pinned to a higher default."},{"line_number":32,"context_line":"    extra_log_level_defaults \u003d ["},{"line_number":33,"context_line":"        \u0027dogpile\u003dINFO\u0027,"},{"line_number":34,"context_line":"        \u0027routes\u003dINFO\u0027,"},{"line_number":35,"context_line":"    ]"},{"line_number":36,"context_line":"    logging.set_defaults(default_log_levels\u003dlogging.get_default_log_levels() +"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_f6c828ab","line":33,"updated":"2016-08-31 01:13:58.000000000","message":"I\u0027m guessing this was a carry over from keystone that we don\u0027t need since we don\u0027t use it.","commit_id":"3a07066345a9aeed423a692242c2096fe5a2682d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"51b1b9e082558ac028f978f290a7ef75ced74de7","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        env \u003d os.environ"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    dirname \u003d env.get(\u0027OS_PLACEMENT_CONFIG_DIR\u0027, \u0027/etc/nova\u0027).strip()"},{"line_number":47,"context_line":"    return os.path.join(dirname, \u0027nova.conf\u0027)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"def init_application():"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_b635c0a9","line":47,"range":{"start_line":47,"start_character":33,"end_line":47,"end_character":44},"updated":"2016-08-31 01:13:58.000000000","message":"Use your CONFIG_FILE constant here, otherwise it\u0027s unused and we can just delete it.","commit_id":"3a07066345a9aeed423a692242c2096fe5a2682d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"51b1b9e082558ac028f978f290a7ef75ced74de7","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    # dump conf if we\u0027re at debug"},{"line_number":59,"context_line":"    if conf.CONF.debug:"},{"line_number":60,"context_line":"        conf.CONF.log_opt_values(logging.getLogger(conf.CONF.prog), logging.DEBUG)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    return deploy.loadapp(conf.CONF)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_f63f4885","line":60,"range":{"start_line":60,"start_character":61,"end_line":60,"end_character":65},"updated":"2016-08-31 01:13:58.000000000","message":"What is this? Does it come out to be \u0027nova\u0027? Maybe we should just hardcode that?","commit_id":"3a07066345a9aeed423a692242c2096fe5a2682d"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"549c0627308dadc142448a796b66a0749b853f1e","unresolved":false,"context_lines":[{"line_number":27,"context_line":"CONFIG_FILE \u003d \u0027nova.conf\u0027"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"def setup_logging(config):"},{"line_number":31,"context_line":"    # Any dependent libraries that have unhelp debug levels should be"},{"line_number":32,"context_line":"    # pinned to a higher default."},{"line_number":33,"context_line":"    extra_log_level_defaults \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_fcdb901f","line":30,"updated":"2016-08-31 12:47:08.000000000","message":"Nit: should go for _setup_logging to match _get_config_file?","commit_id":"d957bfb9f137610d6f28ec3f63d85a5584a2ba62"}]}