)]}'
{"functions":[{"author":{"_account_id":7203,"name":"yuuichi fujioka","email":"fujioka.yuuichi@gmail.com","username":"FujiokaYuuichi"},"change_message_id":"a861b4d7a78453410202b0fd29afbbbab4f67960","unresolved":false,"context_lines":[{"line_number":1620,"context_line":"    local do_syslog\u003d$4"},{"line_number":1621,"context_line":"    if [ \"$log_color\" \u003d\u003d \"True\" ] \u0026\u0026 [ \"$do_syslog\" \u003d\u003d \"False\" ]; then"},{"line_number":1622,"context_line":"        # Add color to logging output"},{"line_number":1623,"context_line":"        iniset $conf_file $conf_section logging_context_format_string \"%(asctime)s.%(msecs)03d %(color)s%(levelname)s %(name)s [\u001b[01;36m%(request_id)s \u001b[00;36m%(user_name)s %(project_name)s%(color)s] \u001b[01;35m%(instance)s%(color)s%(message)s\u001b[00m\""},{"line_number":1624,"context_line":"        iniset $conf_file $conf_section logging_default_format_string \"%(asctime)s.%(msecs)03d %(color)s%(levelname)s %(name)s [\u001b[00;36m-%(color)s] \u001b[01;35m%(instance)s%(color)s%(message)s\u001b[00m\""},{"line_number":1625,"context_line":"        iniset $conf_file $conf_section logging_debug_format_suffix \"\u001b[00;33mfrom (pid\u003d%(process)d) %(funcName)s %(pathname)s:%(lineno)d\u001b[00m\""},{"line_number":1626,"context_line":"        iniset $conf_file $conf_section logging_exception_prefix \"%(color)s%(asctime)s.%(msecs)03d TRACE %(name)s \u001b[01;35m%(instance)s\u001b[00m\""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAASn%2F%2FVt4%3D","line":1623,"updated":"2013-09-06 01:58:02.000000000","message":"cinder labels user_name as user_id and project_name as project_id.\nYou might want to add user_name and project_name to parameter.","commit_id":"1853777943ddf82cd28e5b511a1b2a80c8860dbf"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"3244070390f648406fe8fffc2962dd4a0a4b4608","unresolved":false,"context_lines":[{"line_number":1626,"context_line":"        iniset $conf_file $conf_section logging_exception_prefix \"%(color)s%(asctime)s.%(msecs)03d TRACE %(name)s \u001b[01;35m%(instance)s\u001b[00m\""},{"line_number":1627,"context_line":"    else"},{"line_number":1628,"context_line":"        # Show user_name and project_name instead of user_id and project_id"},{"line_number":1629,"context_line":"        iniset $conf_file $conf_section logging_context_format_string \"%(asctime)s.%(msecs)03d %(levelname)s %(name)s [%(request_id)s %(user_name)s %(project_name)s] %(instance)s%(message)s\""},{"line_number":1630,"context_line":"    fi"},{"line_number":1631,"context_line":"}"},{"line_number":1632,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAASX%2F%2F7cU%3D","line":1629,"updated":"2013-08-23 16:06:03.000000000","message":"I think you need to verify that this \u0027else\u0027 config is valid for all of the project you\u0027re using it in...have a look at the bottom of http://logs.openstack.org/06/43006/1/check/gate-tempest-devstack-vm-full/834cce3/logs/screen-c-api.txt.gz for an example.  This is from an otherwise-marked-successful jenkins run on this review.","commit_id":"1853777943ddf82cd28e5b511a1b2a80c8860dbf"}]}