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

Add more debugging for file settings edge case #100313

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Oct 5, 2023

In order to continue debugging #98391, this commit adds more debug logging to the test, to determine if the error metadata is not being placed in the cluster state correctly.

In order to continue debugging elastic#98391, this commit adds more debug
logging to the test, to determine if the error metadata is not being
placed in the cluster state correctly.
@elasticsearchmachine elasticsearchmachine added v8.12.0 needs:triage Requires assignment of a team area label labels Oct 5, 2023
@rjernst rjernst added >test Issues or PRs that are addressing/adding tests :Core/Infra/Settings Settings infrastructure and APIs labels Oct 5, 2023
@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team and removed needs:triage Requires assignment of a team area label labels Oct 5, 2023
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@williamrandolph williamrandolph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rjernst rjernst merged commit 06f838d into elastic:main Oct 5, 2023
@rjernst rjernst deleted the file_settings/test_debug branch October 5, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Settings Settings infrastructure and APIs Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants