[master] b1f812182 gc duplicate call to VBP_Update_Backend() when creating backends
Nils Goroll
nils.goroll at uplex.de
Thu Dec 10 18:43:06 UTC 2020
commit b1f812182bb03cb8ffbb5451e1220a729f8afc57
Author: Nils Goroll <nils.goroll at uplex.de>
Date: Thu Dec 10 19:08:28 2020 +0100
gc duplicate call to VBP_Update_Backend() when creating backends
VRT_new_backend_clustered() is the only caller of VBP_Insert().
There, VBP_Update_Backend() needs to be called after the director is
created in order to update the director status for a cold VCL.
For a warm VCL, VBP_Update_Backend() is called via
VRT_AddDirector() -> VDI_Event() -> vbe_dir_event() -> VBP_Control()
Thus, the additonal call before the director is added to the backend,
does not do anything but update the poll bits.
Somehow related to #3362
diff --git a/bin/varnishd/cache/cache_backend_probe.c b/bin/varnishd/cache/cache_backend_probe.c
index 8432c8582..8d7a9c9b0 100644
--- a/bin/varnishd/cache/cache_backend_probe.c
+++ b/bin/varnishd/cache/cache_backend_probe.c
@@ -695,7 +695,6 @@ VBP_Insert(struct backend *b, const struct vrt_backend_probe *vp,
vbp_build_req(vt, vp, b);
vbp_reset(vt);
- VBP_Update_Backend(vt);
}
void
More information about the varnish-commit
mailing list