Skip to content
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

[CI] GoogleCloudStorageBlobContainerRetriesTests testReadBlobWithReadTimeouts failing #115172

Closed
elasticsearchmachine opened this issue Oct 20, 2024 · 4 comments · Fixed by #115197
Assignees
Labels
:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs low-risk An open issue or test failure that is a low risk to future releases Team:Distributed Meta label for distributed team (obsolete) >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Oct 20, 2024

Build Scans:

Reproduction Line:

./gradlew ':plugins:repository-gcs:test' --tests "org.elasticsearch.repositories.gcs.GoogleCloudStorageBlobContainerRetriesTests.testReadBlobWithReadTimeouts" -Dtests.seed=D91004A714FAA6C5 -Dtests.locale=lij-Latn-IT -Dtests.timezone=Pacific/Marquesas -Druntime.java=23

Applicable branches:
7.17

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: 
Expected: <2>
     but: was <0>

Issue Reasons:

  • [7.17] 2 failures in test testReadBlobWithReadTimeouts (0.6% fail rate in 331 executions)
  • [7.17] 2 failures in step openjdk23_java-matrix (40.0% fail rate in 5 executions)
  • [7.17] 2 failures in pipeline elasticsearch-periodic (16.7% fail rate in 12 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.

@elasticsearchmachine elasticsearchmachine added :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test-failure Triaged test failures from CI Team:Distributed Meta label for distributed team (obsolete) needs:risk Requires assignment of a risk label (low, medium, blocker) labels Oct 20, 2024
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-distributed (Team:Distributed)

@pxsalehi
Copy link
Member

pxsalehi commented Oct 21, 2024

This doesn't seem to be as clearly the same issue as #115145 (I don't see the HTTP server assertion), but seems to only fail for JDK23, which makes me thing it is a similar issue. It also fails on main.

@pxsalehi pxsalehi 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 Oct 21, 2024
@pxsalehi pxsalehi self-assigned this Oct 21, 2024
@ywangd
Copy link
Member

ywangd commented Oct 21, 2024

Maybe we should just remove the randomBoolean in sendIncompleteContent so it always flush. I thought about it in #115177. But was not sure whether it is too aggressive. Maybe it is the right move for jdk 23.

@pxsalehi
Copy link
Member

ok. same time as I opened the PR. It seems this should be only temporary, so maybe not always always.

elasticsearchmachine pushed a commit that referenced this issue Oct 21, 2024
pxsalehi added a commit to pxsalehi/elasticsearch that referenced this issue Oct 21, 2024
pxsalehi added a commit to pxsalehi/elasticsearch that referenced this issue Oct 21, 2024
elasticsearchmachine pushed a commit that referenced this issue Oct 21, 2024
mark-vieira pushed a commit to mark-vieira/elasticsearch that referenced this issue Oct 23, 2024
elasticsearchmachine pushed a commit that referenced this issue Oct 23, 2024
…ndIncompleteContent with JDK23 (#115197) (#115441)

Resolves #115172

Co-authored-by: Pooya Salehi <[email protected]>
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this issue Oct 25, 2024
mark-vieira pushed a commit to mark-vieira/elasticsearch that referenced this issue Oct 25, 2024
elasticsearchmachine pushed a commit that referenced this issue Oct 25, 2024
…ndIncompleteContent with JDK23 (#115197) (#115680)

Resolves #115172

Co-authored-by: Pooya Salehi <[email protected]>
jfreden pushed a commit to jfreden/elasticsearch that referenced this issue Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs low-risk An open issue or test failure that is a low risk to future releases Team:Distributed Meta label for distributed team (obsolete) >test-failure Triaged test failures from CI
Projects
None yet
3 participants