Skip to content

Commit

Permalink
Make CCR QA tests build again (#33113)
Browse files Browse the repository at this point in the history
Welp, I broke this. I merged a change to auto-discover the CCR QA tests
by making :x-pack:plugin:ccr:check auto-discover the check tasks in the
qa sub-project. Yet, the check tasks for these sub-projects did not
depend on the necessary test tasks (as we were previously doing this
directly from the ccr build file. This commit fixes this!
  • Loading branch information
jasontedor authored Aug 24, 2018
1 parent 91a052b commit 7fa8a72
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,4 +36,5 @@ followClusterTestRunner {
finalizedBy 'leaderClusterTestCluster#stop'
}

check.dependsOn followClusterTest
test.enabled = false
Original file line number Diff line number Diff line change
Expand Up @@ -71,4 +71,5 @@ followClusterTestRunner {
finalizedBy 'leaderClusterTestCluster#stop'
}

check.dependsOn followClusterTest
test.enabled = false // no unit tests for multi-cluster-search, only the rest integration test
1 change: 1 addition & 0 deletions x-pack/plugin/ccr/qa/multi-cluster/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,5 @@ followClusterTestRunner {
finalizedBy 'leaderClusterTestCluster#stop'
}

check.dependsOn followClusterTest
test.enabled = false // no unit tests for multi-cluster-search, only the rest integration test

0 comments on commit 7fa8a72

Please sign in to comment.