)]}'
{"nova/tests/virt/libvirt/test_imagebackend.py":[{"author":{"_account_id":5371,"name":"Mathew Odden","email":"locke105@gmail.com","username":"locke105"},"change_message_id":"5b529cecd80cf7b0ed042f93d3b43a598db4835a","unresolved":false,"context_lines":[{"line_number":331,"context_line":"        os.path.exists(self.TEMPLATE_PATH).AndReturn(False)"},{"line_number":332,"context_line":"        os.path.exists(self.PATH).AndReturn(True)"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        imagebackend.libvirt_utils.get_disk_backing_file(self.PATH)\\"},{"line_number":335,"context_line":"            .AndReturn(None)"},{"line_number":336,"context_line":"        imagebackend.disk.get_disk_size(self.TEMPLATE_PATH"},{"line_number":337,"context_line":"                                       ).AndReturn(self.SIZE)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FcnY%3D","line":334,"updated":"2013-07-01 19:14:14.000000000","message":"nit: Two different line continuation styles between this and line 336. It would be good to be consistent within the same test.","commit_id":"cf3cf3c8e5137e775eec9939e88aa1df339ba1bf"},{"author":{"_account_id":1525,"name":"Rafi Khardalian","email":"rafi@metacloud.com","username":"rkhardalian"},"change_message_id":"57d309e93189f420eb0331955a483fbc99f056fc","unresolved":false,"context_lines":[{"line_number":332,"context_line":"        os.path.exists(self.PATH).AndReturn(True)"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        imagebackend.libvirt_utils.get_disk_backing_file(self.PATH)\\"},{"line_number":335,"context_line":"            .AndReturn(None)"},{"line_number":336,"context_line":"        imagebackend.disk.get_disk_size(self.TEMPLATE_PATH"},{"line_number":337,"context_line":"                                       ).AndReturn(self.SIZE)"},{"line_number":338,"context_line":"        os.path.exists(self.PATH).AndReturn(True)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2Fz8A%3D","line":335,"updated":"2013-06-20 16:47:41.000000000","message":"Backing file returned as None.","commit_id":"cf3cf3c8e5137e775eec9939e88aa1df339ba1bf"}],"nova/virt/libvirt/imagebackend.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"472a1b15acccdb809b5c02cbb6ad73be1553bd39","unresolved":false,"context_lines":[{"line_number":262,"context_line":"        # backing file it expects to be using."},{"line_number":263,"context_line":"        if os.path.exists(self.path):"},{"line_number":264,"context_line":"            backing_path \u003d libvirt_utils.get_disk_backing_file(self.path)"},{"line_number":265,"context_line":"            if backing_path is not None:"},{"line_number":266,"context_line":"                backing_file \u003d os.path.basename(backing_path)"},{"line_number":267,"context_line":"                backing_parts \u003d backing_file.rpartition(\u0027_\u0027)"},{"line_number":268,"context_line":"                if backing_file !\u003d backing_parts[-1] and \\"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FQAY%3D","line":265,"updated":"2013-07-06 06:14:59.000000000","message":"Question - why not:\nif backing_path:","commit_id":"cf3cf3c8e5137e775eec9939e88aa1df339ba1bf"}]}