)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"7052e1368e52237269017b5a47a0a85aefa98fab","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Matt Riedemann \u003cmriedem@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-04 00:32:37 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add the ability to configure glanceclient debug logging"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It\u0027s currently not possible to configure debug logging"},{"line_number":10,"context_line":"for glanceclient, it\u0027s hard-coded to warning level. It"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bacf61ea_d56cfed1","line":7,"updated":"2016-08-05 15:34:33.000000000","message":"I wonder if using a separate logging config file would be a more general solution to this problem (so that we do not have to have new config options per client / library used).","commit_id":"eb5dc52407757d7dc16c10f836fd1520540d09e6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"74651337ba25b68bf283ad937b990441d0e1b621","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Matt Riedemann \u003cmriedem@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-04 00:32:37 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add the ability to configure glanceclient debug logging"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It\u0027s currently not possible to configure debug logging"},{"line_number":10,"context_line":"for glanceclient, it\u0027s hard-coded to warning level. It"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bacf61ea_069e6462","line":7,"in_reply_to":"bacf61ea_d56cfed1","updated":"2016-08-05 20:37:24.000000000","message":"Well, technically I think you could configure this via default_log_levels in oslo.log, but the default levels for those are hard-coded in code, so using that in actual config sucks in case the defaults in the code change. This just seemed much easier.\n\nhttps://github.com/openstack/oslo.log/blob/master/oslo_log/_options.py#L19","commit_id":"eb5dc52407757d7dc16c10f836fd1520540d09e6"}],"nova/config.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"1f28d0da45c5b9cdc2ed0cd31a43ba5455441789","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    # We use the oslo.log default log levels which includes suds\u003dINFO"},{"line_number":34,"context_line":"    # and add only the extra levels that Nova needs"},{"line_number":35,"context_line":"    if CONF.glance.debug:"},{"line_number":36,"context_line":"        _EXTRA_DEFAULT_LOG_LEVELS \u003d [\u0027glanceclient\u003dDEBUG\u0027]"},{"line_number":37,"context_line":"    else:"},{"line_number":38,"context_line":"        _EXTRA_DEFAULT_LOG_LEVELS \u003d [\u0027glanceclient\u003dWARN\u0027]"},{"line_number":39,"context_line":"    log.set_defaults(default_log_levels\u003dlog.get_default_log_levels() +"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_5f2448ae","line":36,"range":{"start_line":36,"start_character":8,"end_line":36,"end_character":33},"updated":"2016-08-03 05:33:54.000000000","message":"as it is moved from the module scope to a local scope I think  it should be renamed to all lower case. Also the leading underscore seems unnecessary. e.g. extra_default_log_levels","commit_id":"b7501790ffb032d10d33e49c912331503015ac48"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b92ecfc3d78f72e6f69ec43e9ecf2b36345d1181","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    # We use the oslo.log default log levels which includes suds\u003dINFO"},{"line_number":34,"context_line":"    # and add only the extra levels that Nova needs"},{"line_number":35,"context_line":"    if CONF.glance.debug:"},{"line_number":36,"context_line":"        _EXTRA_DEFAULT_LOG_LEVELS \u003d [\u0027glanceclient\u003dDEBUG\u0027]"},{"line_number":37,"context_line":"    else:"},{"line_number":38,"context_line":"        _EXTRA_DEFAULT_LOG_LEVELS \u003d [\u0027glanceclient\u003dWARN\u0027]"},{"line_number":39,"context_line":"    log.set_defaults(default_log_levels\u003dlog.get_default_log_levels() +"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_32735417","line":36,"range":{"start_line":36,"start_character":8,"end_line":36,"end_character":33},"in_reply_to":"bacf61ea_5f2448ae","updated":"2016-08-04 00:33:06.000000000","message":"Done","commit_id":"b7501790ffb032d10d33e49c912331503015ac48"}]}