Skip to content

[CI] FileSettingsServiceIT testErrorSaved failing #124279

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 Mar 6, 2025 · 3 comments · Fixed by #124960
Closed

[CI] FileSettingsServiceIT testErrorSaved failing #124279

elasticsearchmachine opened this issue Mar 6, 2025 · 3 comments · Fixed by #124960
Labels
:Core/Infra/Core Core issues without another label medium-risk An open issue or test failure that is a medium risk to future releases Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

Build Scans:

Reproduction Line:

./gradlew ":server:internalClusterTest" --tests "org.elasticsearch.reservedstate.service.FileSettingsServiceIT.testErrorSaved" -Dtests.seed=7829BBA35F7FC4F7 -Dtests.locale=fy -Dtests.timezone=Australia/Hobart -Druntime.java=24

Applicable branches:
9.0

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: null

Issue Reasons:

  • [9.0] 2 failures in test testErrorSaved (0.5% fail rate in 372 executions)
  • [9.0] 2 failures in pipeline elasticsearch-periodic-platform-support (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 Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >test-failure Triaged test failures from CI needs:risk Requires assignment of a risk label (low, medium, blocker) Team:Distributed Indexing Meta label for Distributed Indexing team labels Mar 6, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@fcofdez fcofdez added :Core/Infra/Core Core issues without another label Team:Core/Infra Meta label for core/infra team and removed Team:Distributed Indexing Meta label for Distributed Indexing team :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. labels Mar 11, 2025
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-core-infra (Team:Core/Infra)

@mosche mosche added medium-risk An open issue or test failure that is a medium risk to future releases and removed needs:risk Requires assignment of a risk label (low, medium, blocker) labels Mar 13, 2025
@mosche
Copy link
Contributor

mosche commented Mar 14, 2025

another one test failure in the same suite #124847
all failures are on arm so far, might be hard to reproduce but likely more towards low risk in this case

rjernst added a commit to rjernst/elasticsearch that referenced this issue Mar 16, 2025
@rjernst
Guard checking master node watching for file settings
da6555f 
In tests for file settings the master node is started async. The
assertion that the master node's file setting service is watching for
changes must then wait until the async thread has started. Some tests
already do that. This PR adjusts the remaining tests to similarly wait
for the master node's watcher thread to start.

closes elastic#124279
@rjernst rjernst mentioned this issue Mar 16, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Core/Infra/Core Core issues without another label medium-risk An open issue or test failure that is a medium risk to future releases Team:Core/Infra Meta label for core/infra team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Guard checking master node watching for file settings rjernst/elasticsearch
3 participants
@fcofdez @mosche @elasticsearchmachine