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.

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

[CI] Ec2DiscoveryTests testFilterByTags failing #125090

Closed
elasticsearchmachine opened this issue Mar 18, 2025 · 4 comments · Fixed by #125109
Closed

[CI] Ec2DiscoveryTests testFilterByTags failing #125090

elasticsearchmachine opened this issue Mar 18, 2025 · 4 comments · Fixed by #125109
Labels
:Distributed Coordination/Discovery-Plugins Anything related to our integration plugins with EC2, GCP and Azure low-risk An open issue or test failure that is a low risk to future releases Team:Distributed Coordination Meta label for Distributed Coordination team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Mar 18, 2025 •
edited
Loading

Build Scans:

Reproduction Line:

gradlew ":plugins:discovery-ec2:test" --tests "org.elasticsearch.discovery.ec2.Ec2DiscoveryTests.testFilterByTags" -Dtests.seed=6270229B48F52580 -Dtests.locale=bn-Beng-BD -Dtests.timezone=America/Ciudad_Juarez -Druntime.java=24

Applicable branches:
8.x

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: 
Expected: a collection with size <5>
     but: collection size was <0>

Issue Reasons:

  • [8.x] 13 failures in test testFilterByTags (9.0% fail rate in 144 executions)
  • [8.x] 2 failures in step part1 (9.1% fail rate in 22 executions)
  • [8.x] 3 failures in step part-1 (10.7% fail rate in 28 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic-platform-support (50.0% fail rate in 4 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic (50.0% fail rate in 4 executions)
  • [8.x] 2 failures in pipeline elasticsearch-intake (9.1% fail rate in 22 executions)
  • [8.x] 3 failures in pipeline elasticsearch-pull-request (10.7% fail rate in 28 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Distributed Coordination/Discovery-Plugins Anything related to our integration plugins with EC2, GCP and Azure >test-failure Triaged test failures from CI labels Mar 18, 2025
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 2 failures in test testFilterByTags (1.0% fail rate in 207 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Mar 18, 2025
@elasticsearchmachine
Mute org.elasticsearch.discovery.ec2.Ec2DiscoveryTests testFilterByTags
Loading
Loading status checks…
8fc0120 
#125090
@elasticsearchmachine elasticsearchmachine added needs:risk Requires assignment of a risk label (low, medium, blocker) Team:Distributed Coordination Meta label for Distributed Coordination team labels Mar 18, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@DaveCTurner
Copy link
Contributor

Looks like a locale bug in the SDK itself, see aws/aws-sdk-java-v2#5968

@DaveCTurner DaveCTurner mentioned this issue Mar 18, 2025
Merged
@nicktindall nicktindall 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 Mar 18, 2025
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch 8.x

Mute Reasons:

  • [8.x] 9 failures in test testFilterByTags (6.6% fail rate in 137 executions)
  • [8.x] 2 failures in step part1 (9.1% fail rate in 22 executions)
  • [8.x] 3 failures in step part-1 (10.7% fail rate in 28 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic-platform-support (50.0% fail rate in 4 executions)
  • [8.x] 2 failures in pipeline elasticsearch-intake (9.1% fail rate in 22 executions)
  • [8.x] 3 failures in pipeline elasticsearch-pull-request (10.7% fail rate in 28 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Mar 19, 2025
@elasticsearchmachine
Mute org.elasticsearch.discovery.ec2.Ec2DiscoveryTests testFilterByTags
Loading
Loading status checks…
fc97d65 
#125090
DaveCTurner added a commit that referenced this issue Mar 19, 2025
@DaveCTurner
Work around testFilterByTags failure (#125109)

Verified

This commit was signed with the committer’s verified signature.
DaveCTurner David Turner
SSH Key Fingerprint: 2FbaAHs1T3MU2yhr4CIg0XwwZ6yF4pe+1K9H5/OOdcY
Verified
Learn about vigilant mode
Loading
Loading status checks…
1b22b0e 
A SDK bug[^1] means that these values sometimes don't parse as an
integer. We don't need them to do so in order to pass the test, so this
commit relaxes the test to accept any string.

Closes #125090 Closes #125166

[^1]: aws/aws-sdk-java-v2#5968
smalyshev pushed a commit to smalyshev/elasticsearch that referenced this issue Mar 21, 2025
@DaveCTurner @smalyshev
Work around testFilterByTags failure (elastic#125109)

Verified

This commit was signed with the committer’s verified signature.
smalyshev Stanislav Malyshev
GPG key ID: 88CFB1F42CFAEAF6
Verified
Learn about vigilant mode
8c4d9d1 
A SDK bug[^1] means that these values sometimes don't parse as an
integer. We don't need them to do so in order to pass the test, so this
commit relaxes the test to accept any string.

Closes elastic#125090 Closes elastic#125166

[^1]: aws/aws-sdk-java-v2#5968
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Distributed Coordination/Discovery-Plugins Anything related to our integration plugins with EC2, GCP and Azure low-risk An open issue or test failure that is a low risk to future releases Team:Distributed Coordination Meta label for Distributed Coordination team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Work around testFilterByTags failure DaveCTurner/elasticsearch
3 participants
@nicktindall @DaveCTurner @elasticsearchmachine