Follow up to #104886
In all my local testing, the error.reason was always "error-message", but in some circumstances in CI, the error.reason was "fail_processor_exception: error-message". As a workaround, on #104886 I switched the test to check for a regex match on error-message which'll work in both cases.
@masseyke suggested that the issue might be that if you test against a single node, then you'll see just "error-message" but that with multiple nodes (maybe some of which don't have the ingest role) you might be able to provoke this, and I think he's got the right idea.
I think it's reasonable to follow up on that to make sure that we're consistently serializing errors back to the client in all cases, but I didn't want to jump on that task immediately.
Follow up to #104886
In all my local testing, the
error.reasonwas always"error-message", but in some circumstances in CI, theerror.reasonwas"fail_processor_exception: error-message". As a workaround, on #104886 I switched the test to check for a regex match onerror-messagewhich'll work in both cases.@masseyke suggested that the issue might be that if you test against a single node, then you'll see just
"error-message"but that with multiple nodes (maybe some of which don't have the ingest role) you might be able to provoke this, and I think he's got the right idea.I think it's reasonable to follow up on that to make sure that we're consistently serializing errors back to the client in all cases, but I didn't want to jump on that task immediately.