)]}'
{"nova/virt/baremetal/virtual_power_driver.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"e8ad449b12ca1f2257418bd93712f8e4e6aec865","unresolved":false,"context_lines":[{"line_number":219,"context_line":"            LOG.debug(\u0027Result for run_command: %s\u0027 % result)"},{"line_number":220,"context_line":"        except exception.ProcessExecutionError:"},{"line_number":221,"context_line":"            result \u003d []"},{"line_number":222,"context_line":"            LOG.error(\"Error running command: %s\" % cmd)"},{"line_number":223,"context_line":"        return result"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2B7do%3D","line":222,"updated":"2013-02-14 02:41:38.000000000","message":"Perhaps LOG.exception(), should report the error that happened?","commit_id":"65a3b478a14bcbcddc4f8031a18c9dedb7415c1e"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"6cff0e120bccce463106fae455eb4bfd5272c184","unresolved":false,"context_lines":[{"line_number":219,"context_line":"            LOG.debug(\u0027Result for run_command: %s\u0027 % result)"},{"line_number":220,"context_line":"        except exception.ProcessExecutionError:"},{"line_number":221,"context_line":"            result \u003d []"},{"line_number":222,"context_line":"            LOG.error(\"Error running command: %s\" % cmd)"},{"line_number":223,"context_line":"        return result"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2B6yo%3D","line":222,"in_reply_to":"AAAAM3%2F%2B7do%3D","updated":"2013-02-14 15:54:11.000000000","message":"@lifeless, could you explain why you feel exception is better than error here? I chose to do the latter because I don\u0027t want an exception bubbling out of this context","commit_id":"65a3b478a14bcbcddc4f8031a18c9dedb7415c1e"}]}