Stille før stormen: varnish-1.0.4

[Dag-Erling Smørgrav]

[actually cc: the list this time]

Ingvar Hagelund <ingvar at linpro.no> writes:
> Denne postingen burde sikkert vært på engelsk, og postet til
> varnish-dist.

I'll bite :)

> Det er fint om du (des) ser over og kommenterer
>
> * varnish.sysconfig

The following variables:

 VARNISH_MIN_WORKER_THREADS=1
 VARNISH_MAX_WORKER_THREADS=1000
 VARNISH_WORKER_THREAD_TIMEOUT=10
 VARNISH_HASHOPTION=classic

should be removed; changing them is generally not a good idea.

There should be a variable for "extra flags".

The following variable:

 VARNISH_BACKEND_STORAGE_SIZE=10240000

could be written as

 VARNISH_BACKEND_STORAGE_SIZE=10m

but are you sure you want a 10 MB file by default?  The built-in default
is "50%", which means min(50% of available disk space, SIZE_MAX) rounded
down to the nearest page boundary.

The following variable:

 VARNISH_TTL=120

is not very useful (the default is already 120, and in most cases you
want to tweak the TTL on a per-object basis, which must be done in VCL).
The preferred way to change it these days is to set the corresponding
run-time parameter either with -p on the command line or through the
managment port.

The startup knobs which I consider critical are the following (taken
from the updated FreeBSD port):

varnishd_listen=":6081"
varnishd_config="%%PREFIX%%/etc/varnish/default.vcl"
varnishd_telnet="localhost:6082"
varnishd_storage="file,/tmp,50%"
varnishd_flags="-P ${pidfile} -a ${varnishd_listen} -f ${varnishd_config} -T ${varnishd_telnet} -s ${varnishd_storage}"

("varnishd_telnet" should probably be called something else)

That being said, I see no harm in offering additional options, as long
as they neither confuse the user nor encourage him to twiddle knobs he
shouldn't twiddle.

> * varnishlog.sysinit

404

> * varnish.initrc

Is the lock file necessary?  Isn't it better to check for the presence
of the PID file, and of the process it references?

> Disse ligger i svn, i redhat-katalogen. Videre må vi ha en fin
> default.vcl som erstatter vcl.conf med korrekt backend, og resten
> utkommentert (de er jo default-innstillinger, sant?)

Done.

> redhat/varnish.sysconfig og debian/varnish.default må sannsynligvis
> oppdateres, i alle fall med verdier for lytteadresse (tom verdi => ipv4
> +ipv6, sant?) og lytteport (80? 8080? 8000?) . Disse filene kunne
> forresten gjerne være samme fil, eller i hvert fall rimelig identiske.
> Stig? Lars?

Considering that Debian, Fedora and OpenSUSE packages already bind to
:6081 and localhost:6082, I think we should keep those defaults.

> For å få svn-utgaven til å bygge må jeg legge til en './autogen.sh' for
> å få generert configure. Jeg antar at denne blir overflødig i den
> endelige tarballen?

I use 'make dist' to create the tarball.  It automatically creates and
includes the required files, including a working configure script.

DES
-- 
Dag-Erling Smørgrav
Senior Software Developer
Linpro AS - www.linpro.no