varnish-cache icon indicating copy to clipboard operation
varnish-cache copied to clipboard

WIP more WS_Overflowed() checks

Open nigoroll opened this issue 1 year ago • 2 comments

Re #4232

Adding earlier overflow checks is pretty easy if we use VCL failure, but so far we had centralized the overflow handling on the client side in the vtr_deliver_f, sending downstream a 500 error and not a 503. So my main question is: If we want consistency here, can we settle on simply handling workspace overflows as VCL failures? Or do we really want the separate error code still? If we do, improving the overflow detection on the backend side might become a bit more involved, because right now the backend side error just turns into a 503 on the client side.

Note that this PR is really for discussing the design questions, it does not pass the test suite because of the changed error code.

nigoroll avatar Nov 25 '24 18:11 nigoroll

To make progress here, I'd suggest the following:

  • turn all VCL and out-of-workspace related errors into 500 because this is really what they are. 503 makes specific reference to a temporary overload or scheduled maintenance, but if we run into a VCL failure or out-of-workspace, that is likely to persist for the next request.
  • differentiate other error conditions. I think we have also more tickets in this area...

nigoroll avatar Dec 02 '24 15:12 nigoroll

bugwash: turn it all into 500, update PR, write doc

nigoroll avatar Dec 09 '24 14:12 nigoroll