You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Rall <dl...@finemaltcoding.com> on 2005/05/19 18:20:33 UTC

[patch] RAM disk usage instructions and svntest

On Thu, 2005-05-19 at 15:34 +0200, Erik Huelsmann wrote:
>On 5/19/05, lundblad@tigris.org <lu...@tigris.org> wrote:
>> Author: lundblad
>> Date: Thu May 19 07:50:15 2005
>> New Revision: 14778
>> 
>> Modified:
>>   trunk/tools/test-scripts/svntest/README
>> 
>> Log:
>> * tools/test-scripts/svntest/README
>>  (mod_dav_svn_shared.conf): Added authentication parameters for
>>  lock tests over ra_dav
>>  (ramdisk): Changed the ramdisk requirement to 700 MB because
>>  tests failing due to lack of disk space
>
>Unless, ofcourse you run with CLEANUP=1 which significantly reduces
>tmpfs requirements: sussman, brane and I run the windows tests with no
>more than 50MB ramdisc space that way.

Erik, what do you think about the strategy of automatically setting the
cleanup flag when svntest is run with a RAM disk?

[[[
Improved RAM disk usage instructions for svntest, and general info.


* www/faq.html
  Added much more information about testing on a RAM disk.


* tools/test-scripts/svntest/svntest-run.sh
  When a RAM disk is in use, automatically flag Makefile test targets
  for cleanup to reduce overall RAM disk size.  The trade-off of
  increased I/O should be quite reasonable, considering that the I/O
  will be against memory, rather than physical disk.


* tools/test-scripts/svntest/README
  Added a quick blurb about what "svntest" actually is/does.  Altered
  RAM disk instructions to match up with new svntest-run.sh cleanup
  flag behavior, and updated pointer to FAQ URL.
]]]


Re: [patch] RAM disk usage instructions and svntest

Posted by Erik Huelsmann <eh...@gmail.com>.
On 5/19/05, Daniel Rall <dl...@finemaltcoding.com> wrote:
> On Thu, 2005-05-19 at 15:34 +0200, Erik Huelsmann wrote:
> >On 5/19/05, lundblad@tigris.org <lu...@tigris.org> wrote:
> >> Author: lundblad
> >> Date: Thu May 19 07:50:15 2005
> >> New Revision: 14778
> >>
> >> Modified:
> >>   trunk/tools/test-scripts/svntest/README
> >>
> >> Log:
> >> * tools/test-scripts/svntest/README
> >>  (mod_dav_svn_shared.conf): Added authentication parameters for
> >>  lock tests over ra_dav
> >>  (ramdisk): Changed the ramdisk requirement to 700 MB because
> >>  tests failing due to lack of disk space
> >
> >Unless, ofcourse you run with CLEANUP=1 which significantly reduces
> >tmpfs requirements: sussman, brane and I run the windows tests with no
> >more than 50MB ramdisc space that way.
> 
> Erik, what do you think about the strategy of automatically setting the
> cleanup flag when svntest is run with a RAM disk?

I think 700MB RAM disc size is not  reasonable, so, yes, I think that
bringing it back to around 50-60MB is a good idea.

Note that this does not destroy any data, since the RAM disc is
unmounted at the end of the script (thus eliminating any possibility
to check any test data left behind).

bye,


Erik.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org


Re: [patch] RAM disk usage instructions and svntest

Posted by Jani Averbach <ja...@jaa.iki.fi>.
On 2005-05-19 13:05-0700, Daniel Rall wrote:
>
> I committed the changes in r14781.  Feel free to suggest additional
> adjustments.

It looks perfectly good, thanks!

BR, Jani

-- 
Jani Averbach

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [patch] RAM disk usage instructions and svntest

Posted by Daniel Rall <dl...@finemaltcoding.com>.
On Thu, 2005-05-19 at 11:35 -0700, Daniel Rall wrote:
..
>Erik came up with a good patch along these lines.  I'm merging our
>patches together, and will re-post.  Thanks for the feedback, Jani.

I committed the changes in r14781.  Feel free to suggest additional
adjustments.

- Dan



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [patch] RAM disk usage instructions and svntest

Posted by Daniel Rall <dl...@finemaltcoding.com>.
On Thu, 2005-05-19 at 12:31 -0600, Jani Averbach wrote:
>On 2005-05-19 11:20-0700, Daniel Rall wrote:
>> 
>> Erik, what do you think about the strategy of automatically setting the
>> cleanup flag when svntest is run with a RAM disk?
>> 
>
>Please provide it as a flag. I run svntest framework time to time by
>hand or by external scripts, and I then definitely like to poke around
>with test repositories.
>
>> +
>> +    # Flag the tests to cleanup to avoid requiring hundreds of MBs of
>> +    # memory.
>> +    CHECK_ARGS="$CHECK_ARGS CLEANUP=1"
>>  fi
>
>Something like:
>
>+    CHECK_ARGS="$CHECK_ARGS $CHECK_CLEANUP"
>
>and svntest-config.sh:
>+ CHECK_CLEANUP="CLEANUP=1"

Erik came up with a good patch along these lines.  I'm merging our
patches together, and will re-post.  Thanks for the feedback, Jani.

- Dan



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [patch] RAM disk usage instructions and svntest

Posted by Jani Averbach <ja...@jaa.iki.fi>.
On 2005-05-19 11:20-0700, Daniel Rall wrote:
> 
> Erik, what do you think about the strategy of automatically setting the
> cleanup flag when svntest is run with a RAM disk?
> 

Please provide it as a flag. I run svntest framework time to time by
hand or by external scripts, and I then definitely like to poke around
with test repositories.

> +
> +    # Flag the tests to cleanup to avoid requiring hundreds of MBs of
> +    # memory.
> +    CHECK_ARGS="$CHECK_ARGS CLEANUP=1"
>  fi

Something like:

+    CHECK_ARGS="$CHECK_ARGS $CHECK_CLEANUP"

and svntest-config.sh:
+ CHECK_CLEANUP="CLEANUP=1"


Thanks,
Jani

-- 
Jani Averbach

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org