)]}'
{"cinder/volume/api.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"a081870bc2a96989ff105ddce2fab5eb05470b21","unresolved":false,"context_lines":[{"line_number":153,"context_line":"        # set or set incorrectly, in which case get_default_volume_type()"},{"line_number":154,"context_line":"        # returns None."},{"line_number":155,"context_line":"        if volume_type:"},{"line_number":156,"context_line":"            volume_type_id \u003d volume_type.get(\u0027id\u0027)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        options \u003d {"},{"line_number":159,"context_line":"            \u0027size\u0027: size,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALn%2F%2F8dU%3D","line":156,"updated":"2012-10-25 17:38:33.000000000","message":"This is the branch that isn\u0027t covered (it wasn\u0027t covered before either)","commit_id":"38789ec1d17a4f822e55cebc3c65f7a16d49c0cc"}],"cinder/volume/volume_types.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"a081870bc2a96989ff105ddce2fab5eb05470b21","unresolved":false,"context_lines":[{"line_number":114,"context_line":"def get_default_volume_type():"},{"line_number":115,"context_line":"    \"\"\"Get the default volume type.\"\"\""},{"line_number":116,"context_line":"    name \u003d FLAGS.default_volume_type"},{"line_number":117,"context_line":"    vol_type \u003d None"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    if name is not None:"},{"line_number":120,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALn%2F%2F8ds%3D","line":117,"updated":"2012-10-25 17:38:33.000000000","message":"This is more of a tactical comment, but I think it be kinda sexy if you used an empty dict as a sentinel, so when you build up the options dict for create you could just say:\nvolume_type_id \u003d volume_type.get(\u0027id\u0027)\nand let the db record continue to insert None/null.","commit_id":"38789ec1d17a4f822e55cebc3c65f7a16d49c0cc"}]}