Re: pynfs: cleanup and turn off some tests by default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/15/2012 05:28 PM, J. Bruce Fields wrote:
> On Wed, Feb 15, 2012 at 04:28:01PM -0800, Boaz Harrosh wrote:
>> On 02/15/2012 02:45 PM, J. Bruce Fields wrote:
>>> I've committed the following to my pynfs repo.
>>>
>>> Fix a couple minor problems found while working on a bug, then turn off
>>> utf8 and pnfs tests by default (both can still be requested by adding
>>> "utf8" or "pnfs", respectively, on the commandline), and remove one
>>> incorrect test.
>>>
>>> I'd like to get to the point where the pynfs tests all pass against a
>>> reasonable server.  That means fixing the server where necessary, of
>>> course, but also not complaining about unimplemented optional features
>>> or about stuff we don't plan to fix.
>>>
>>
>> Can't we "auto detect" the pnfs option. Request a pnfs access if failed
>> proceed to the next set of tests, if successful continue to test "pnfs"
> 
> Good idea!  I think you'd still want "pnfs" and "nopnfs" options: "pnfs"
> for "I expect this server to support pnfs, and if it doesn't report that
> as a bug!", and "nopnfs" for: "this server claims to support  pnfs but
> please just don't even bother telling me how buggy it is, I already
> know".
> 

Sounds good

> But just turning it off by default was a minute's worth of
> search-and-replace for me, and I don't have the time for more right now.
> So that's a project looking for a volunteer.
> 

NP, thanks for the fixes so far. We'll probably add pynfs testing to our
Ganesha testing rig. When we do we'll look into it. That is if no one
bits us to it, (Hopefully ;-) )

> --b.

Thanks
Boaz
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux