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 aditional logging for ILM history store tests #50624

Merged
merged 2 commits into from Jan 6, 2020

Conversation

dakrone
Copy link
Member

@dakrone dakrone commented Jan 3, 2020

These tests use the same index name, making it hard to read logs when
diagnosing the failures. Additionally more information about the current
state of the index could be retrieved when failing.

This changes these two things in the hope of capturing more data about
why this fails on some CI nodes but not others.

Relates to #50353

These tests use the same index name, making it hard to read logs when
diagnosing the failures. Additionally more information about the current
state of the index could be retrieved when failing.

This changes these two things in the hope of capturing more data about
why this fails on some CI nodes but not others.

Relates to elastic#50353
@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

(This PR is so that I can run CI over and over for these tests, it does not yet contain the fix)

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

14 similar comments
@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 3, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 4, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 5, 2020

@elasticmachine run elasticsearch-ci/2

@dakrone
Copy link
Member Author

dakrone commented Jan 5, 2020

@elasticmachine run elasticsearch-ci/2

Copy link
Member

@cbuescher cbuescher left a comment

Choose a reason for hiding this comment

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

Looks like its not failing that often now, I'd be okay to remove the mutes and get the additional logging into the real CI rotation even this isn't a fix (maybe just change the issue title then)...

@dakrone
Copy link
Member Author

dakrone commented Jan 6, 2020

@elasticmachine update branch

@dakrone dakrone added :Data Management/ILM+SLM Index and Snapshot lifecycle management >non-issue v8.0.0 labels Jan 6, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/ILM+SLM)

@dakrone dakrone changed the title Fix ILM history store tests Add aditional logging for ILM history store tests Jan 6, 2020
@dakrone dakrone added the v7.6.0 label Jan 6, 2020
@dakrone dakrone merged commit 43fb784 into elastic:master Jan 6, 2020
@dakrone dakrone deleted the ilm-fix-history-store-tests branch January 6, 2020 16:58
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Jan 6, 2020
These tests use the same index name, making it hard to read logs when
diagnosing the failures. Additionally more information about the current
state of the index could be retrieved when failing.

This changes these two things in the hope of capturing more data about
why this fails on some CI nodes but not others.

Relates to elastic#50353

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
dakrone added a commit that referenced this pull request Jan 6, 2020
* Add aditional logging for ILM history store tests (#50624)

These tests use the same index name, making it hard to read logs when
diagnosing the failures. Additionally more information about the current
state of the index could be retrieved when failing.

This changes these two things in the hope of capturing more data about
why this fails on some CI nodes but not others.

Relates to #50353
SivagurunathanV pushed a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
These tests use the same index name, making it hard to read logs when
diagnosing the failures. Additionally more information about the current
state of the index could be retrieved when failing.

This changes these two things in the hope of capturing more data about
why this fails on some CI nodes but not others.

Relates to elastic#50353

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants