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 filesystem name to test error message #4272

Merged

Conversation

Projects
None yet
2 participants
@andrewkroh
Copy link
Member

commented May 9, 2017

Errors that are logged by the system/filesystem test case don’t have enough context to debug them. This adds the filesystem that caused the error to the message.

Add filesystem name to test error message
Errors that are logged by the system/filesystem test case don’t have enough context to debug them. This adds the filesystem that caused the error to the message.
@ruflin

ruflin approved these changes May 10, 2017

Copy link
Collaborator

left a comment

WFG

@ruflin ruflin merged commit 9c9b4ac into elastic:master May 10, 2017

6 checks passed

CLA Commit author has signed the CLA
Details
codecov/patch Coverage not affected when comparing e5dbf0f...b73564f
Details
codecov/project 63% (+0.02%) compared to e5dbf0f
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
default Build finished.
Details

@andrewkroh andrewkroh deleted the andrewkroh:bugfix/system-filesystem-error-debug branch Jul 5, 2017

ramon-garcia added a commit to ramon-garcia/beats that referenced this pull request Dec 5, 2017

Add filesystem name to test error message (elastic#4272)
Errors that are logged by the system/filesystem test case don’t have enough context to debug them. This adds the filesystem that caused the error to the message.

athom added a commit to athom/beats that referenced this pull request Jan 25, 2018

Add filesystem name to test error message (elastic#4272)
Errors that are logged by the system/filesystem test case don’t have enough context to debug them. This adds the filesystem that caused the error to the message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.