Make it possible to configure the number of retries the random director
will make at getting a backend connection.  By default it tries as many
times as it has members.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3138 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add runtime check that parsing of timestamps actually work.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3137 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fixed up init script for varnishlog

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3136 d4fa192b-c00b-0410-8231-f00ffab90ce4

Varnish 2.0 bonus feature #1:  backend->max_connections

        backend default {
                .host = "127.0.0.1";
                .port = "9080";
                .max_connections = 100;
        }

will limit the simultaneous TCP connections to this backend to 100.

Stats counter backend_busy == "Backend connections too many" tells
how often we have hit this limit for a backend.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3135 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make the random and round-robin directors avoid backends which are
polled as unhealty.

This technically makes Varnish 2.0 "feature complete", but some further
polishing is to be expected.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3134 d4fa192b-c00b-0410-8231-f00ffab90ce4

make check now runs without races

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3133 d4fa192b-c00b-0410-8231-f00ffab90ce4

Tada!

First step in activating backend polling:  Do not attempt to open
connections to a unhealthy backend.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3132 d4fa192b-c00b-0410-8231-f00ffab90ce4

Mark all nonprobed backends healty

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3131 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fixed an error in the argument passing from the init script. Added some more info text to the stop action.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3130 d4fa192b-c00b-0410-8231-f00ffab90ce4

Give sessions a backend connection instead of a backend and
have the directors provide that.

Entirely a no-op, apart from a bugfix that just would screw
you even more (than "totally") if socket(2) should ever fail.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3129 d4fa192b-c00b-0410-8231-f00ffab90ce4

Correctly quote backend data in default vcl example.

Submitted by: ingvar
Fixes: #298

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3128 d4fa192b-c00b-0410-8231-f00ffab90ce4

Use TCP_close() to properly ignore the "canonical" TCP legal errno values
consistently.

Fixes #297

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3127 d4fa192b-c00b-0410-8231-f00ffab90ce4

Move the tcp.c file to libvarnish, we need the primitives in varnishtest
also.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3126 d4fa192b-c00b-0410-8231-f00ffab90ce4

Added a missing changelog item from Fedora

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3125 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fixing up init script according to newer Fedora standards

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3124 d4fa192b-c00b-0410-8231-f00ffab90ce4

correct pidfile option to daemon and killproc functions

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3123 d4fa192b-c00b-0410-8231-f00ffab90ce4

Patched up Red Hat init script to comply with newer Fedora standards

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3122 d4fa192b-c00b-0410-8231-f00ffab90ce4

A typo, and newline at the end

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3121 d4fa192b-c00b-0410-8231-f00ffab90ce4

Give the directors another method which returns not the backend to
connect to, but a connection to a backend, this makes it possible
for the directors to choose another backend, if connection to the
first backend fails.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3120 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make the director opaque outside the backend code.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3119 d4fa192b-c00b-0410-8231-f00ffab90ce4

Move selection of backend and creation of default Host: header until
we actually need to get a filedescriptor to the backend.

This also makes it evident for vcl_pass{}, vcl_pipe{} and vcl_miss{}
if the client sent a Host: header or not.  Previously these functions
saw the default Host: header.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3118 d4fa192b-c00b-0410-8231-f00ffab90ce4

Polish

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3117 d4fa192b-c00b-0410-8231-f00ffab90ce4

Use invalid address in binding test

IANA has reserved the 198.18.0.0/24 network for Network Interconnect Device
Benchmark Testing which vaguely fits what we are doing.  Use that
instead of 127.0.0.2 since the latter works fine to bind to on Linux
(at least).

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3116 d4fa192b-c00b-0410-8231-f00ffab90ce4

Be explicit about connecting to :9080

When testing synthetic error page caching, make sure we connect to
:9080 rather than the default :80 since the machine might well have an
httpd running already.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3115 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make make check run test suite

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3114 d4fa192b-c00b-0410-8231-f00ffab90ce4

Integrate Solaris patches, with a bit of polishing.

Don't look for umem on FreeBSD, which have dummy stub version for
dtrace or ZFS compatibility.

Generally restrict probes for Solaris specific stuff to when we
run on solaris, no need to make configure slower than it is.

Don't pretend we can get anywhere without poll(2) we use it for
lots of things:  Remove #if conditionals around poll-acceptor.

Sort a couple of lists, before they get too unsorted.

Submitted by: Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3113 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add a test-case that exercises the backend-polling a bit.  It does not
really check anything yet, because we lack the stats counters to test.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3112 d4fa192b-c00b-0410-8231-f00ffab90ce4

Don't call a zero byte response a "Good Recv".

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3111 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix ticket 292:  Make sure the "deleted by Connection:" flag sticks
with its header around.

Found and nailed by: Adrian Otto <aotto@mosso.com>

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3110 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add a regression test for ticket 292

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3109 d4fa192b-c00b-0410-8231-f00ffab90ce4

Silence FlexeLint

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3108 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make error strings static

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3107 d4fa192b-c00b-0410-8231-f00ffab90ce4

Don't attempt to update the Health information based on how regular
traffic fares.  Problems seen there could be caused by all sorts
of special circumstances (bad PHP, weird headers from client etc)
and is not indicative of the backends health.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3106 d4fa192b-c00b-0410-8231-f00ffab90ce4

Check that the HTTP response is 200 before we declare the probe happy.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3105 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add -u for unbuffered output.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3104 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add exponential average of responsetime to output

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3103 d4fa192b-c00b-0410-8231-f00ffab90ce4

Use the default (or specified) Host: header for the backend in the
probe requests.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3102 d4fa192b-c00b-0410-8231-f00ffab90ce4

Started the upgrade race for 2.0

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3101 d4fa192b-c00b-0410-8231-f00ffab90ce4

Update the backend->healty state based on the window/threshold
paramters.

Log each polls result in the SHMlog

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3100 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make up my mind: defaults go in varnishd

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3099 d4fa192b-c00b-0410-8231-f00ffab90ce4

Implement parsing of .window and .threshold in backend polling specifications.

.window is how many of the latest polls we examine.
.threshold is how many must have succeeded for the backend to be healthy.

.window = 40;
.threshold = 30;
// Thirty of the last fourty polls must have succeed.

.threshold = 4;
// The last four polls must have succeeded.

Default values:
.window = 8;
.threshold = 3;

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3098 d4fa192b-c00b-0410-8231-f00ffab90ce4

When an -expect fails in varnishtest, loop for up to a second since stats sometimes takes a little while to update

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3097 d4fa192b-c00b-0410-8231-f00ffab90ce4

Don't use a thread from the pool, backend polling is far too long-lived for
that to make sense.

We do not need a mutex for our list, all manipulation is from CLI thread.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3096 d4fa192b-c00b-0410-8231-f00ffab90ce4

Accept EINTR from waitpid()

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3095 d4fa192b-c00b-0410-8231-f00ffab90ce4

Changes in the specfile:
- default.vcl has moved
- Added groff to build requirements

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3094 d4fa192b-c00b-0410-8231-f00ffab90ce4

Be more careful about concurrency.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3093 d4fa192b-c00b-0410-8231-f00ffab90ce4

Install default.vcl to ${sysconfdir}/varnish, not ${datadir}

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3092 d4fa192b-c00b-0410-8231-f00ffab90ce4

Remove If-Modified-Since headers on included objects to get consistent
behaviour.

Submitted by:  sky (ticket 267)

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3091 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix build failure when srcdir != builddir

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3090 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix build failure, cache_backend_poll.h was not in the distributed tarball

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3089 d4fa192b-c00b-0410-8231-f00ffab90ce4

Stop hiding command invocations when building docs

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3088 d4fa192b-c00b-0410-8231-f00ffab90ce4

Added some more output suppression

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3087 d4fa192b-c00b-0410-8231-f00ffab90ce4

New file with the bitmap definitons for for backend polling.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3086 d4fa192b-c00b-0410-8231-f00ffab90ce4

Added suppresion of output when generating the default.vcl. Fixes #287

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3085 d4fa192b-c00b-0410-8231-f00ffab90ce4

Snapshot the backend polling code without the advanced math.

Presently we only record th state of the last 64 tries to poll the
backend in a shift register, which can be show with the interrim
CLI command "debug.health":

Health stats for backend b0
Oldest __________________________________________________ Newest
4444444444444444444444444444444444444444444444444444444444444444 Good IPv4
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX Good Xmit
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS Good Shut
RR-RR-R-RRRR---RR-RR-R-R-R-R-R-RRRR-RR-----RR--RRRR--RRR-R-----R Good Recv

More to follow.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3084 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make a central TCP_close() function that accepts the two errno's which
indicate that our partner gave up on us.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3083 d4fa192b-c00b-0410-8231-f00ffab90ce4

nits

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3082 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make the compile shared object be named $mumble.so to cater for
Solaris tool-chains.

Simplified slightly from submitted patch.

Submitted by: Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3081 d4fa192b-c00b-0410-8231-f00ffab90ce4

It now always flushes the stream, even for stdout. See ticket #182.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3080 d4fa192b-c00b-0410-8231-f00ffab90ce4

More whitespace changes

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3079 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix up style

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3078 d4fa192b-c00b-0410-8231-f00ffab90ce4

Be much more explicit about the string->time conversion disaster
visited upon us by the brilliant minds behind POSIX.

Parts from: Solaris patch from Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3077 d4fa192b-c00b-0410-8231-f00ffab90ce4

Eliminate <err.h> usage.

Inspired by: Solaris patch from Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3076 d4fa192b-c00b-0410-8231-f00ffab90ce4

Remove an spurious <err.h> include.

Fix and extend regression tests while here.

Inspired by: solaris patch from Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3075 d4fa192b-c00b-0410-8231-f00ffab90ce4

Cleaned up some usage statements

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3074 d4fa192b-c00b-0410-8231-f00ffab90ce4

Move the C-compiler invocation determination to autoconf.

(Fixed the apple & gnu compiler strings relative to submitted patch)

Submitted by: Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3073 d4fa192b-c00b-0410-8231-f00ffab90ce4

Pull in all the #include <config.h> from the Solaris patch

Submitted by: Theo Schlossnagle

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3072 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix a printf type complaint.

Also fix style.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3071 d4fa192b-c00b-0410-8231-f00ffab90ce4

Try to fix Automake spec to also work on FreeBSD.

XXX: Isn't something about removing the temporary files missing ?

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3070 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix up Makefiles so make distcheck works again (out of tree builds and remove generated files on distclean)

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3069 d4fa192b-c00b-0410-8231-f00ffab90ce4

An outline and some text for a "Getting started guide" for Varnish.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3068 d4fa192b-c00b-0410-8231-f00ffab90ce4

Support out-of-tree builds, and make sure to clean up after ourselves.  Both aka "make make distcheck work"

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3067 d4fa192b-c00b-0410-8231-f00ffab90ce4

Forgot a depedency to bin/varnishd/default.vcl

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3066 d4fa192b-c00b-0410-8231-f00ffab90ce4

Updated the man page for vcl to include the default bin/varnishd/default.vcl so it is easier kept in sync. Same goes for etc/default.vcl, which now is generated from the same bin/varnishd/default.vcl.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3065 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add script to generate the request-processing state diagram cheat-sheet.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3064 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add an .eps version

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3063 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fixed makefile depedency. default_vcl.h is now generated as a depdency of mgt_vcc.c

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3062 d4fa192b-c00b-0410-8231-f00ffab90ce4

Move the default VCL code into its own text file.

Generate "default_vcl.h" from this using sed(1) in the Makefile.

Include default_vcl.h in mgt_vcc.c

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3061 d4fa192b-c00b-0410-8231-f00ffab90ce4

The compiler should not care about field order in director decls.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3060 d4fa192b-c00b-0410-8231-f00ffab90ce4

Use WRONG instead of INCOMPL

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3059 d4fa192b-c00b-0410-8231-f00ffab90ce4

Make explicit what is missing and what is asserts in the state-engine.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3058 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add WRONG(expl) macro for panicing from places we just shouldn't get to.

Dump the handling value from the last VCL method on panic

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3057 d4fa192b-c00b-0410-8231-f00ffab90ce4

Longer delay for panic to complete.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3056 d4fa192b-c00b-0410-8231-f00ffab90ce4

Actually wait for the varnish child to stop in "varnish -stop".

Update test-case

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3055 d4fa192b-c00b-0410-8231-f00ffab90ce4

Report back to CLI if we cannot start the child due to non-bindable sockets

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3054 d4fa192b-c00b-0410-8231-f00ffab90ce4

Be much more explicit about localhost.

Force varnish instance into /tmp/__${name}

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3053 d4fa192b-c00b-0410-8231-f00ffab90ce4

Go over the b* series of tests:
add expectes for stats counters where necessary
generally polish.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3052 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add
-cliok <cli_cmd>
and
-clierr <expected_status> <cli_cmd>

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3051 d4fa192b-c00b-0410-8231-f00ffab90ce4

note when the fd gets closed

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3050 d4fa192b-c00b-0410-8231-f00ffab90ce4

Fix pass statistics counter

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3049 d4fa192b-c00b-0410-8231-f00ffab90ce4

Rename the vcl_fetch{} action "insert" to "deliver" as we may not
insert (for instance on pass) but we certainly will deliver (unless
error or restart actions are used instead).

Update the dotgraph to 2.0 intent

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3048 d4fa192b-c00b-0410-8231-f00ffab90ce4

That test rightfully is a v* test

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3047 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add a test-case for VCL purging

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3046 d4fa192b-c00b-0410-8231-f00ffab90ce4

Explanation

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3045 d4fa192b-c00b-0410-8231-f00ffab90ce4

Complete the separation of synthetic from received objects.

Any object, also errors, we receive from the backend will
go to vcl_fetch(), and can be cached if desired.

Any object we create, for instance if we cannot contact the
backend, will go to vcl_error() and will not be cached.

Technical details:

Implement VCL variable setting for booleans like obj.cacheable.

Remove obj.valid, only valid objects go to vcl_fetch now.

On fetch failure, drop the object and go to STP_ERROR with 503

Update testcase b00015.vtc accordingly.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3044 d4fa192b-c00b-0410-8231-f00ffab90ce4

Ok, 3 retries then...

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3043 d4fa192b-c00b-0410-8231-f00ffab90ce4

Retry client connects once.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3042 d4fa192b-c00b-0410-8231-f00ffab90ce4

Add uptime to varnishstat -1 output

Make varnishstat -1 also output the uptime.

Closes #68

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3041 d4fa192b-c00b-0410-8231-f00ffab90ce4

An assert to convince FlexeLint that we have a variable.

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3040 d4fa192b-c00b-0410-8231-f00ffab90ce4

Don't write to read-only structure members in the shared object
definition structure.

Fixes ticket 285

git-svn-id: svn+ssh://projects.linpro.no/svn/varnish/trunk@3039 d4fa192b-c00b-0410-8231-f00ffab90ce4