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 possible message to flaky test check #8989

Merged
merged 2 commits into from
Nov 8, 2018

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Nov 8, 2018

TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circustances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

Fixes #8412

@jsoriano jsoriano added module review Metricbeat Metricbeat needs_backport PR is waiting to be backported to other branches. flaky-test Unstable or unreliable test cases. labels Nov 8, 2018
@jsoriano jsoriano force-pushed the timeout-test-modules-test-error branch from 1ee92e0 to 5a2eb4d Compare November 8, 2018 11:58
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circustances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.
@jsoriano jsoriano force-pushed the timeout-test-modules-test-error branch from 5a2eb4d to 115f75e Compare November 8, 2018 11:59
@jsoriano jsoriano merged commit 4fbcec0 into elastic:master Nov 8, 2018
@jsoriano jsoriano deleted the timeout-test-modules-test-error branch November 8, 2018 14:58
@jsoriano jsoriano added v6.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Nov 8, 2018
jsoriano added a commit to jsoriano/beats that referenced this pull request Nov 8, 2018
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 4fbcec0)
@jsoriano jsoriano added the v6.4.4 label Nov 8, 2018
jsoriano added a commit to jsoriano/beats that referenced this pull request Nov 8, 2018
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 4fbcec0)
jsoriano added a commit to jsoriano/beats that referenced this pull request Nov 8, 2018
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 4fbcec0)
@jsoriano jsoriano added the v6.6.0 label Nov 8, 2018
jsoriano added a commit that referenced this pull request Nov 9, 2018
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 4fbcec0)
jsoriano added a commit that referenced this pull request Nov 9, 2018
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 4fbcec0)
jsoriano added a commit that referenced this pull request Nov 9, 2018
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 4fbcec0)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 5a38ccc)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
TestCommands.test_modules_test_error relays on the presence of an error
message, this error message seems to be different under some
circumstances, as seen in CI builds.

Add the timeout message already seen, and print logs in case of failure
so it is more easy to fix if happens in the future.

(cherry picked from commit 5a38ccc)
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.

[Metricbeat] Flaky test_cmd.TestCommands.test_modules_test_error
2 participants