[PATCH 3/7] Test basic quota enforcement

Jan Kara jack at suse.cz
Thu May 20 12:21:15 CDT 2010


On Thu 20-05-10 11:11:30, Christoph Hellwig wrote:
> The golden output contains "/local/test1/file5" which appaears
> to be in your test fs.  You need to do something like
> 
> 	sed -e "s,$SCRATCH_MNT,SCRATCH_MNT,g"
> 
> or
> 
> 	sed -e "s,$T“T_MNT,TEST_MNT,g"
> 
> to make the test portable.  This probably applies to the other tests
> as well.
  I have filter_scratch function for this in all tests. I just forgot to
filter output of touch(1) through it. It should be fixed now.
  BTW: What is the purpose of 'status' variable? I set it to 1 if I want
the test to fail because diff found differences between accounted and
real usage. Is this correct? Actually, diff will also print the differences
to stdout so setting status probably shouldn't be needed. So is it preferred
to set status to 1 or leave it 0?

								Honza
-- 
Jan Kara <jack at suse.cz>
SUSE Labs, CR




More information about the xfs mailing list