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

Re-enable single node tests #38852

Merged
merged 4 commits into from
Feb 16, 2019
Merged

Re-enable single node tests #38852

merged 4 commits into from
Feb 16, 2019

Conversation

jdconrad
Copy link
Contributor

This will re-enable SingleNodeTests with a bit of extra information logged to help debug the CI issue (#36782) causing this test to fail.

@jdconrad jdconrad added >test Issues or PRs that are addressing/adding tests :Data Management/Watcher v8.0.0 labels Feb 13, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

Copy link
Contributor

@talevy talevy left a comment

Choose a reason for hiding this comment

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

may we never see these logs in CI 🤞

@jdconrad
Copy link
Contributor Author

@talevy Thanks for the review and suggestion for the annotation. Will commit as soon as CI completes successfully.

@jdconrad
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/1

@jdconrad jdconrad merged commit 1fc5958 into elastic:master Feb 16, 2019
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Feb 16, 2019
* elastic/master:
  Ensure global test seed is used for all random testing tasks (elastic#38991)
  re-mutes SmokeTestWatcherWithSecurityIT (elastic#38995)
  Rollup jobs should be cleaned up before indices are deleted (elastic#38930)
  relax ML Info Docs expected response (elastic#38993)
  Re-enable single node tests (elastic#38852)
  ClusterClientIT refactor (elastic#38872)
  Fix typo in Index API doc
  Edits to text & formatting in Term Suggester doc (elastic#38963) (elastic#38989)
  Migrate Streamable to Writeable for WatchStatus (elastic#37390)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Watcher >test Issues or PRs that are addressing/adding tests v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants