On Fri, Jun 24, 2011 at 8:22 AM, Lucas Meneghel Rodrigues <[email protected]>wrote:
> On Fri, 2011-06-24 at 17:11 +0800, Shrirang Phansalkar wrote: > > Hi, > > > > I am trying to use fsdev_disks.py for getting the disks list. I was > trying it on one of my scripts > > I am doing following step > > > > disks = fsdev_disks.get_disk_list() > > > > When I execute the script I am getting following error > > > > Unhandled AttributeError: 'FsdevManager' object has no attribute > 'check_mount_point'. > > > > Is there something wrong with 'FsdevManager' or I am performing some > wrong steps? > > Please can anyone help me about this. > > Yes, there is something wrong with this class... fsdev_disks uses > fsdev_mgr, but this class is only a skeleton, and needs to be > complemented with a site extension to fill up the gaps... > > (Reference: client/bin/fsdev_mgr.py, class BaseFsdevManager) > > Considering we are shipping code that makes use of this non-implemented > methods, we need to have at least a basic implementation of them on > BaseFsdevManager. I found the origin of this code to be in: > > http://autotest.kernel.org/changeset/2156 > > Trying to copy the original author. I think those missing methods are > implemented on an internal extension kept at a google repo. Also copying > Greg and John. Guys, any idea of where this is and if it's possible to > fill up this base class with some reasonable implementation for upstream > users, as a reference and to avoid running broken code? If we can't find > this, will have to figure out some methods to implement this fsdev API. > > Well Shrirang, meanwhile you can either start looking at implementing > those missing methods or wait a little bit until we figure this out. > Sorry about that, opened a bug about this issue: > > http://autotest.kernel.org/ticket/55 > > Thank you guys, > > Lucas > > It looks like we're missing implementations of client.bin.fsdev_mgr.BaseFsdevManager use_partition() and check_mount_point() methods that we only had internal site implementations of. I'll mail a patch adding them shortly. -gps
_______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
