-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[8.x](backport #4012) [Integration testing] Remove AltErr and only expect 400 from ES bulk API #4030
Conversation
Cherry-pick of 1ae9ab6 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
buildkite test it |
This pull request has not been merged yet. Could you please review and merge it @ycombinator? 🙏 |
Fix mergify mess
f5ddc88
to
0850e31
Compare
buildkite test this |
b67faee
to
7b1d0d9
Compare
Quality Gate passedIssues Measures |
What is the problem this PR solves?
This PR expects only a 400 HTTP response code from the Elasticsearch bulk API in integration tests when invalid index name is specified. Concretely, it removes the
AltErr
field temporarily introduced in #4001 which was allowing the tests to accept an alternate error from Elasticsearch.Related: elastic/elasticsearch#114869
This is an automatic backport of pull request #4012 done by Mergify.