-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Description
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.