Skip to content

ESQL: Allow some error in test #126865

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

Merged
merged 1 commit into from
Apr 15, 2025
Merged

ESQL: Allow some error in test #126865

merged 1 commit into from
Apr 15, 2025

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Apr 15, 2025

Without #123290 we're not going to get accurate status reports. They are indicative, but not accurate.

Closes #126808

Without elastic#123290 we're not going to get accurate status reports. They are
indicative, but not accurate.

Closes elastic#126808
@nik9000 nik9000 added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v8.18.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Apr 15, 2025
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.18.1 labels Apr 15, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@nik9000
Copy link
Member Author

nik9000 commented Apr 15, 2025

The 8.16.7 BWC test is failing to build the old version but this seems ok. I don't usually ignore failures, but this is to silence a test that shouldn't be failing already and not being able to build an old version is pretty unrelated to that.

@nik9000 nik9000 merged commit c9e7e5a into elastic:8.18 Apr 15, 2025
13 of 16 checks passed
@nik9000 nik9000 deleted the esql_status_ok branch April 15, 2025 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.18.0 v8.18.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants