)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"11451a8fe8fca85241d4b59bf681eca89dcb7c52","unresolved":false,"context_lines":[{"line_number":17,"context_line":"it clutters the logs."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"This adds a \u0027client_log_level\u0027 config option to the \u0027ironic\u0027 config"},{"line_number":20,"context_line":"section in nova. If it\u0027s unset (default), nova\u0027s defaults are used."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Example usage:"},{"line_number":23,"context_line":"[ironic]"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1ae5cdf2_c81f39b7","line":20,"updated":"2014-06-17 15:57:11.000000000","message":"A couple of questions here:\n- if it is unset, does it use Nova\u0027s defaults, or Nova\u0027s log level setting?\n- What if oslo\u0027s \u0027default_log_levels\u0027 conf variable was used too? Will this client_log_level take precedence?","commit_id":"d0495b9bf06cb2634cb9a4cd9a24b7955990b918"}],"ironic/nova/virt/ironic/driver.py":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"11451a8fe8fca85241d4b59bf681eca89dcb7c52","unresolved":false,"context_lines":[{"line_number":59,"context_line":"    cfg.StrOpt(\u0027admin_url\u0027,"},{"line_number":60,"context_line":"               help\u003d\u0027Keystone public API endpoint.\u0027),"},{"line_number":61,"context_line":"    cfg.StrOpt(\u0027client_log_level\u0027,"},{"line_number":62,"context_line":"               help\u003d\u0027Log level override for ironicclient\u0027),"},{"line_number":63,"context_line":"    cfg.StrOpt(\u0027pxe_bootfile_name\u0027,"},{"line_number":64,"context_line":"               help\u003d\u0027This gets passed to Neutron as the bootfile dhcp \u0027"},{"line_number":65,"context_line":"               \u0027parameter when the dhcp_options_enabled is set.\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_e8c23533","line":62,"updated":"2014-06-17 15:57:11.000000000","message":"I\u0027d like a bit more help here, if this isn\u0027t specified. ;)\n\nEg, \u0027If not set (default), Nova\u0027s log level setting is used.\u0027  Or \u0027If not set, it defaults to using Nova\u0027s log level.\u0027","commit_id":"d0495b9bf06cb2634cb9a4cd9a24b7955990b918"},{"author":{"_account_id":11702,"name":"Ellen Hui","email":"ellenfkh@gmail.com","username":"ellenfkh"},"change_message_id":"a7fcb033f44cd94c24612bfe08de0914369fa227","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        self.extra_specs \u003d extra_specs"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"        icli_log_level \u003d CONF.ironic.client_log_level"},{"line_number":183,"context_line":"        if icli_log_level:"},{"line_number":184,"context_line":"            level \u003d py_logging.getLevelName(icli_log_level)"},{"line_number":185,"context_line":"            logger \u003d py_logging.getLogger(\u0027ironicclient\u0027)"},{"line_number":186,"context_line":"            logger.setLevel(level)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_84f45188","line":183,"updated":"2014-06-09 18:39:52.000000000","message":"Does this default automatically to something false?  The corresponding option doesn\u0027t have a default, so this looks like it might end up a weird value.","commit_id":"d0495b9bf06cb2634cb9a4cd9a24b7955990b918"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"5e134f46abefa5760b9d9be1b4d8c51e98fc620b","unresolved":false,"context_lines":[{"line_number":180,"context_line":"        self.extra_specs \u003d extra_specs"},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"        icli_log_level \u003d CONF.ironic.client_log_level"},{"line_number":183,"context_line":"        if icli_log_level:"},{"line_number":184,"context_line":"            level \u003d py_logging.getLevelName(icli_log_level)"},{"line_number":185,"context_line":"            logger \u003d py_logging.getLogger(\u0027ironicclient\u0027)"},{"line_number":186,"context_line":"            logger.setLevel(level)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_faebd3a1","line":183,"in_reply_to":"1ae5cdf2_84f45188","updated":"2014-06-17 17:35:59.000000000","message":"conf options without defaults are None. They aren\u0027t random values. :)","commit_id":"d0495b9bf06cb2634cb9a4cd9a24b7955990b918"}]}
