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

[ML] making test more determinate #40374

Merged

Conversation

benwtrent
Copy link
Member

@benwtrent benwtrent commented Mar 22, 2019

Initially, there was the semi-rare scenario where the inconsistentReplica would actually be the primary shard, and not cause a failure. In this change, I force the inconsistentReplica to only be a shard replica and never the primary.

closes #40368

@benwtrent benwtrent added >test Issues or PRs that are addressing/adding tests v8.0.0 v7.2.0 :ml/Transform Transform labels Mar 22, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

@benwtrent benwtrent requested a review from hendrikmuhs March 22, 2019 17:13
Copy link

@hendrikmuhs hendrikmuhs left a comment

Choose a reason for hiding this comment

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

LGTM

@benwtrent benwtrent merged commit 73df33d into elastic:master Mar 22, 2019
@benwtrent benwtrent deleted the test/data-frame-fix-failed-test-40368 branch March 22, 2019 20:22
benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Mar 22, 2019
* [ML] making test more determinate

* unmuting test
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Mar 22, 2019
* elastic/master:
  [DOCS] Adds notable highlights tags (elastic#40330)
  [ML] making test more determinate (elastic#40374)
  [ML] adds support for non-numeric mapped types (elastic#40220)
  Move outbound message handling to OutboundHandler (elastic#40336)
  Add implicit this for class binding in Painless (elastic#40285)
  Muting test testExtractIndexCheckpointsInconsistentGlobalCheckpoints (elastic#40371)
  DOC: polish client docs
  Fix building bwc versions (elastic#40361)
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Mar 22, 2019
…-stop-time

* elastic/master:
  [DOCS] Adds notable highlights tags (elastic#40330)
  [ML] making test more determinate (elastic#40374)
  [ML] adds support for non-numeric mapped types (elastic#40220)
  Move outbound message handling to OutboundHandler (elastic#40336)
  Add implicit this for class binding in Painless (elastic#40285)
benwtrent added a commit that referenced this pull request Mar 23, 2019
* [ML] making test more determinate

* unmuting test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml/Transform Transform >test Issues or PRs that are addressing/adding tests v7.2.0 v8.0.0-alpha1
Projects
None yet
4 participants