- 05 Mar, 2019 5 commits
-
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
Add another column for the probe result Ref #2896
-
Nils Goroll authored
backend.list now shows "probe" only if a probe is present. Ref #2896
-
Nils Goroll authored
The sick state of the director and the healthy callback are the same thing coming from different directions: Either we query the status dynamically or we already have it. For layering directors, the health state is determined by their backends, so having a director-layer sick state does not make much sense and duplicates logic. Also, the sick field duplicates admin_health to some extend (see cache_director.c do_set_health). This is also relevant in the context of streamlining the backend.list output: admin_health "probe" only makes sense if backends actually do have a probe (= some dynamically determined health state). It appears streaight forward that the presense of a vdi_healthy_f callback is the signal for dynamically determined health state. So we move the sick field into VBE and retire VRT_SetHealth(). We also remove the ctx argument from VRT_SetChanged() because I previously overlooked that it is required, for example, in probe code where we got no ctx. Ref #2896
-
- 04 Mar, 2019 12 commits
-
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
highest open fd by reading /proc/`getpid`/fd directory. If it fails we fall back to sysconf(_SC_OPEN_MAX) as usual.
-
Poul-Henning Kamp authored
as soon as a RST_STREAM has been received. Testcase by: xcir Fixes: #2923
-
Nils Goroll authored
Implicitly fixes the same issue for VDI_GetHdr
-
Nils Goroll authored
Ref: #2896
-
Nils Goroll authored
by example of the round-robin director Also changes the backend probe state good->healthy bad->sick for consistency
-
Nils Goroll authored
Ref #2899
-
Nils Goroll authored
Ref #2859 #1799
-
Nils Goroll authored
Ref #2859
-
Nils Goroll authored
Ref #1799
-
Nils Goroll authored
In order to avoid the cross product problem with conversion functions (from x to y types would require x*y functions), we add flexibility regarding the input parameters to conversion functions: Each convertion function named after the destination type now takes all sensible arguments by name. int is named integer because of the reserved symbol name "int" in C. All functions should be fully backwards compatible (existing vtcs continue to work), but compile time checks are now effectively removed. The conversion functions now trigger vcl errors if used incorrectly or if conversion errors occur and no failback is provided. However, if a failback is provided, vcl errors are only raised for usage errors. For consistency, the conversion functions now only ever truncate if necessary. std.round() is added for explicit rounding where required. Existing functions which are now obsolete are marked deprecated. Ref #2899 Ref https://github.com/varnishcache/varnish-cache/wiki/VIP12:-vmod-polymorphism-(for-type-conversions)
-
Poul-Henning Kamp authored
I have reimplemented this based on Nils's #2858, because I found it too complex and intrusive. (In particular we try to avoid unions in Varnish). Testcase m00051 by: Nils Goroll Closes: #2858
-
- 03 Mar, 2019 9 commits
-
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
It may have sounded like it's been added for the first time.
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
Geoff Simmons authored
-
- 01 Mar, 2019 1 commit
-
-
Federico G. Schwindt authored
-
- 27 Feb, 2019 4 commits
-
-
Nils Goroll authored
-
Nils Goroll authored
the previous code would round 9007199254740991 to 9007199254740992 Tested on linux and the four vtest SunOS variants
-
Nils Goroll authored
vtest would segfault due to a null pointer access on empty headers
-
Poul-Henning Kamp authored
Closes #2905
-
- 26 Feb, 2019 9 commits
-
-
Nils Goroll authored
we missed to change these when we changed the VCL_INT typedef from long to int64_t Should we add VCL_INT_MAX / VCL_INT_MIN (and for the other integer types, respectively)?
-
Nils Goroll authored
-
Nils Goroll authored
Ref: doc/README.WRITING_RST.rst
-
Nils Goroll authored
in particular: - remove defintion lists (Description/Example) - examples as code blocks:: - argument names as *emphasis* - ``literals`` Ref: doc/README.WRITING_RST.rst
-
Nils Goroll authored
Spotted by @lkarsten
-
Lasse Karstensen authored
This commit attempts to make it easier to read build output, and hopefully makes configure.ac easier as well. -Wall has already been added around line 645, no need to add it twice. On GCC -Wreturn-type is a part of -Wall, which we enable elsewhere. https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html On Clang -Wreturn-type is enabled by default: https://clang.llvm.org/docs/DiagnosticsReference.html#wreturn-type Same with -Wswitch as -Wreturn-type, so that can be removed as well. Finally, use spaces instead of tabs to make cc output shorter and more readable.
-
Nils Goroll authored
-
Nils Goroll authored
I did consider generalizing VCL_IterDirector which handles backend iterations for the CLI, but found the complications not worth the savings for such a trivial function.
-
Dridi Boukelmoune authored
-