)]}'
{"nova/virt/vmwareapi/vm_util.py":[{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"9eaf43e2d6a998fe0c0c8ca7bb7a45ac930e26ba","unresolved":false,"context_lines":[{"line_number":570,"context_line":"                ds_free \u003d prop.val"},{"line_number":571,"context_line":"        # Local storage identifier"},{"line_number":572,"context_line":"        if ds_type \u003d\u003d \"VMFS\" or ds_type \u003d\u003d \"NFS\":"},{"line_number":573,"context_line":"            data_store_name \u003d ds_name"},{"line_number":574,"context_line":"            return elem.obj, data_store_name, ds_cap, ds_free"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"    if data_store_name is None:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FcyQ%3D","line":573,"updated":"2013-05-09 07:41:26.000000000","message":"lets just return ds_name here and remove line 556.","commit_id":"150b29fa1eb647ac933f26af97153a744eae76a4"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"9eaf43e2d6a998fe0c0c8ca7bb7a45ac930e26ba","unresolved":false,"context_lines":[{"line_number":572,"context_line":"        if ds_type \u003d\u003d \"VMFS\" or ds_type \u003d\u003d \"NFS\":"},{"line_number":573,"context_line":"            data_store_name \u003d ds_name"},{"line_number":574,"context_line":"            return elem.obj, data_store_name, ds_cap, ds_free"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"    if data_store_name is None:"},{"line_number":577,"context_line":"        raise exception.DatastoreNotFound()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FcyU%3D","line":575,"updated":"2013-05-09 07:41:26.000000000","message":"If we get to this line we will always raise an exception so the if statement isn\u0027t needed .","commit_id":"150b29fa1eb647ac933f26af97153a744eae76a4"}]}