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

Check valid cluster service state transitions #21538

Conversation

ywelsch
Copy link
Contributor

@ywelsch ywelsch commented Nov 14, 2016

This commit adds assertions to check whether the cluster service state transitions in a way that we expect it to.

Relates to #21379 (comment)

This commit adds assertions to check whether the cluster service state transitions in a way that we expect it to.
@s1monw
Copy link
Contributor

s1monw commented Nov 14, 2016

LGTM

@ywelsch ywelsch merged commit d42f7ee into elastic:master Nov 14, 2016
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Nov 14, 2016
* master: (82 commits)
  Remove `IndexTemplateAlreadyExistsException` and `IndexShardAlreadyExistsException` (elastic#21539)
  Fix line length in TCPTransportTests
  Check valid cluster service state transitions (elastic#21538)
  [TEST] Add test to ensure `transport.tcp.compress` works
  remove forbidden API
  Enable 5.x to 6.x BWC tests
  Set LoggedExec back
  Rebasing master
  Add Vagrant Gradle plugin
  [docs] clients need to add jackson-databind (elastic#21527)
  Update significantterms-aggregation.asciidoc
  Upgrade to lucene-6.3.0. (elastic#21464)
  Hack around cluster service and logging race
  Do not prematurely shutdown Log4j
  Support decimal constants with trailing [dD] in painless (elastic#21412)
  In painless suggest a long constant if int won't do (elastic#21415)
  Account for different paths for sysctl utilities
  [TEST] testRebalancePossible() may not have an assigned node id
  Tests: Disable merge in SearchCancellationTests
  Tests: clean search scroll at the end of SearchCancellationIT
  ...
bleskes pushed a commit that referenced this pull request Dec 19, 2016
This commit adds assertions to check whether the cluster service state transitions in a way that we expect it to.

Relates to #21379.
@bleskes bleskes added the v5.2.0 label Dec 19, 2016
@bleskes
Copy link
Contributor

bleskes commented Dec 19, 2016

This is backported to 5.x as well. I adapted the labels

@clintongormley clintongormley added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. and removed :Cluster labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >non-issue v5.2.0 v6.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants