Hi all, Thanks a lot for the help. This has made my work easier.
Thanks and regards, Shrirang ________________________________________ From: Gregory P. Smith [[email protected]] Sent: Friday, June 24, 2011 11:32 PM To: Lucas Meneghel Rodrigues Cc: Shrirang Phansalkar; [email protected]; [email protected]; John Admanski Subject: Re: [Autotest] 'FsdevManager' object has no attribute 'check_mount_point' On Fri, Jun 24, 2011 at 8:22 AM, Lucas Meneghel Rodrigues <[email protected]<mailto:[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 PROPRIETARY-CONFIDENTIAL INFORMATION INCLUDED This electronic transmission, and any documents attached hereto, may contain confidential, proprietary and/or legally privileged information. The information is intended only for use by the recipient named above. If you received this electronic message in error, please notify the sender and delete the electronic message. Any disclosure, copying, distribution, or use of the contents of information received in error is strictly prohibited, and violators will be pursued legally. _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
