)]}'
{"cinder/volume/manager.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"64268c40e275e9d2172732e82f0a76dff2cbfcf4","unresolved":false,"context_lines":[{"line_number":344,"context_line":"            # check the volume status before attaching"},{"line_number":345,"context_line":"            volume \u003d self.db.volume_get(context, volume_id)"},{"line_number":346,"context_line":"            if volume[\u0027status\u0027] \u003d\u003d \u0027attaching\u0027:"},{"line_number":347,"context_line":"                if volume[\u0027instance_uuid\u0027] and \\"},{"line_number":348,"context_line":"                    volume[\u0027instance_uuid\u0027] !\u003d instance_uuid:"},{"line_number":349,"context_line":"                    msg \u003d _(\"being attached by another instance\")"},{"line_number":350,"context_line":"                    raise exception.InvalidVolume(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F22I%3D","line":347,"updated":"2012-12-31 17:12:58.000000000","message":"Minor nit, rather than use the \"and \\\" I\u0027d prefer wrapping this in parens.  Also, you should indent the next line 8 spaces in order to distinguish it from the beginning of the function body.","commit_id":"945f22aec4a8aba6a8dd03896d5994490035cbc0"}]}