Skip to content

watcher - remove types from full cluster restart tests#48014

Merged
jakelandis merged 1 commit intoelastic:masterfrom
jakelandis:fix_watcher_full_cluster_no_types
Oct 14, 2019
Merged

watcher - remove types from full cluster restart tests#48014
jakelandis merged 1 commit intoelastic:masterfrom
jakelandis:fix_watcher_full_cluster_no_types

Conversation

@jakelandis
Copy link
Copy Markdown
Contributor

@jakelandis jakelandis commented Oct 14, 2019

Types are no longer supported and this commit removes
them from the full cluster restart tests.

Related : #47986


Note - also ran full suite successfully ./gradlew :x-pack:qa:full-cluster-restart:bwcTest -Dtests.class=org.elasticsearch.xpack.restart.FullClusterRestartIT -Dtests.timestamp=$(date +%S)

@jakelandis jakelandis added >test Issues or PRs that are addressing/adding tests :Distributed/Watcher v8.0.0 labels Oct 14, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Watcher)

Copy link
Copy Markdown
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM

@jakelandis
Copy link
Copy Markdown
Contributor Author

merging before packaging-sample tests are complete since this is only a test fix and master CI is currently broken (this fixes it). BWC here and full BWC restart tests have been run and passed.

@jakelandis jakelandis merged commit 84030fc into elastic:master Oct 14, 2019
@jakelandis jakelandis deleted the fix_watcher_full_cluster_no_types branch October 14, 2019 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/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.

3 participants