Skip to content

[CI] DriverTests testProfileAndStatusFinishInOneRound failing #126808

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.

Sign up for GitHub

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

Jump to bottom
Closed
elasticsearchmachine opened this issue Apr 14, 2025 · 3 comments
Closed

[CI] DriverTests testProfileAndStatusFinishInOneRound failing #126808

elasticsearchmachine opened this issue Apr 14, 2025 · 3 comments
Labels
:Analytics/ES|QL AKA ESQL low-risk An open issue or test failure that is a low risk to future releases Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

Build Scans:

Reproduction Line:

./gradlew ":x-pack:plugin:esql:compute:test" --tests "org.elasticsearch.compute.operator.DriverTests.testProfileAndStatusFinishInOneRound" -Dtests.seed=E8DB241CEC4BBFB6 -Dtests.locale=id -Dtests.timezone=Atlantic/St_Helena -Druntime.java=24

Applicable branches:
8.18

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: 
Expected: <80192L>
     but: was <681632L>

Issue Reasons:

  • [8.18] 2 failures in test testProfileAndStatusFinishInOneRound (0.8% fail rate in 254 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Analytics/ES|QL AKA ESQL >test-failure Triaged test failures from CI Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) needs:risk Requires assignment of a risk label (low, medium, blocker) labels Apr 14, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@bpintea bpintea added low-risk An open issue or test failure that is a low risk to future releases and removed needs:risk Requires assignment of a risk label (low, medium, blocker) labels Apr 15, 2025
@nik9000
Copy link
Member

nik9000 commented Apr 15, 2025

8.18 is missing #123290. Let's just make the test more permissive.

nik9000 added a commit to nik9000/elasticsearch that referenced this issue Apr 15, 2025
@nik9000
ESQL: Allow some error in test
486d0c1 
Without elastic#123290 we're not going to get accurate status reports. They are
indicative, but not accurate.

Closes elastic#126808
@nik9000 nik9000 mentioned this issue Apr 15, 2025
Merged
nik9000 added a commit that referenced this issue Apr 15, 2025
@nik9000
ESQL: Allow some error in test (#126865)
c9e7e5a 
Without #123290 we're not going to get accurate status reports. They are
indicative, but not accurate.

Closes #126808
@alex-spies
Copy link
Contributor

This was supposed to be closed by #126865, but the bot somehow didn't close this; so I'm gonna do the bot's job :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Analytics/ES|QL AKA ESQL low-risk An open issue or test failure that is a low risk to future releases Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nik9000 @bpintea @elasticsearchmachine @alex-spies