On 08/08/2012 07:20 PM, Steve French wrote: > On Wed, Aug 8, 2012 at 7:41 AM, Jeff Layton <jlayton@xxxxxxxxxx> wrote: >> On Wed, 08 Aug 2012 17:31:09 +0530 >> Suresh Jayaraman <sjayaraman@xxxxxxxx> wrote: >>> >>> The primary intent of the tests is to provide some basic infrastructure >>> upon which tests can be added easily in the future and these tests are >>> by no means comprehensive. I have tried to avoid duplicating tests >>> already done by Connectathon and other tests but there could still be a >>> few duplicates in there. The tests are only lightly tested. >>> >>> Currently cifstests is hosted here: >>> >>> https://github.com/sureshjayaram/cifstests >>> >> >> Sounds like a very worthwhile endeavor. The key to any testing >> infrastructure is to make it very easy to run the tests. Any hassle in >> setting it up is a reason not to do so, so you want to make sure there >> are no such barriers. >> >> You also want to make sure that you have the ability to drill down into >> a single test failure without needing to run a bunch of goop around it. >> The cthon suite is good for this since most of the tests are written in >> C. That makes it easy to strace them to track down problems. I assume >> we'll be able to write tests in C and just have the python framework >> call them? > > I like the idea of something expandable that we can add tests to (e.g. > for cifs specific mount options, or to add tests to cover recently > reported bugs). Yeah, one of the reasons to keep these tests separate and not adding to xfstests or other is it, it will give us more flexibility to add CIFS specific tests without doing a lot of testing on other filesystems. > I have patches to run xfstests over cifs, but I mostly run > connectathon and dbench and fsx. Ideally running regression tests > could be triggered by commits to a staging tree (or branch of > cifs-2.6.git) on git.samba.org although not certain how easy this While that would be ideal, I think if we could run such tests atleast twice during a release cycle (let's say once during -rc1 and once during final rc), it would still help us to catch regressions. Thanks Suresh -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
