)]}'
{"tempest/scenario/test_baremetal_basic_ops.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"d1b0c318aa75316fd10c89da59034f50ac258451","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                status\u003d\u0027ACTIVE\u0027)"},{"line_number":67,"context_line":"        except exceptions.TimeoutException as e:"},{"line_number":68,"context_line":"            node \u003d self.get_node(node_id\u003dnode[\u0027uuid\u0027])"},{"line_number":69,"context_line":"            LOG.debug(node)"},{"line_number":70,"context_line":"            raise e"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def verify_partition(self, client, label, mount, gib_size):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a710552_1c644e47","line":69,"updated":"2015-12-16 01:24:54.000000000","message":"So we definitely need to add context to this log line. Right now it will just be a bare dict (and the other basic log context) not a lot to go off of. We need to also log what the dict is and why its there.\n\nAlso is debug the right level? I\u0027m really not sure because this is only in an error path, if you could check the logging guidelines spec to confirm that would be great.","commit_id":"1192df6158bf01dd7e3d0fc3195c4cae516559f8"}]}