Test suite for dynamic backends

Dridi Boukelmoune dridi at varni.sh
Wed Jul 8 11:44:24 CEST 2015


On Wed, Jul 8, 2015 at 12:55 AM, Poul-Henning Kamp <phk at phk.freebsd.dk> wrote:
>> A VRT_CTX may not be available in all situations for backends deletion.
>
> They will have to be.  If need be, the caller must construct one.

I didn't think it was OK to make my own context, thanks.

>>Subject: [PATCH 2/5] Keep track of backends references and connections
>
> I've done this entirely differently.
>
> Fixes #1755

Please revisit this one, my patch also fixes two panics I just got while
rebasing against the current trunk. See attached test logs.

>> [PATCH 3/5] Add a debug.sleep(DURATION) synchronization helper
> This should use VTIM_sleep().
>
> Commit it when fixed & tested.

Ack

>> Subject: [PATCH 4/5] Add a test suite for dynamic backends
>
> I've run out of steam, will look at this tomorrow.

Thanks

>> Subject: [PATCH 5/5] Add more information to a backend panic message
>
> This doesn't work with the way I did #2/5 above.

Do I send a patch or simply commit?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: d00008.log
Type: text/x-log
Size: 18887 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20150708/a5f3ca8d/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: d00009.log
Type: text/x-log
Size: 13475 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20150708/a5f3ca8d/attachment-0003.bin>


More information about the varnish-dev mailing list