Fix rolling upgrade test dependency
The rolling upgrade nodes need a keystore for SSL configuration but there was no dependency on the task that copies the keystore into the output directory for the nodes to pick up as an extra configuration file. This commit addresses this by adding such a dependency. To do this, we need to break the dependency of the keystore copy task on the REST spec copy task; this is not an issue since the dependency was for convenience of ordering the task and not actually needed. Original commit: elastic/x-pack-elasticsearch@fddbc06e9f
This commit is contained in:
parent
386ac7345c
commit
284bf2512a
|
@ -57,6 +57,12 @@ task bwcTest {
|
||||||
group = 'verification'
|
group = 'verification'
|
||||||
}
|
}
|
||||||
|
|
||||||
|
task copyTestNodeKeystore(type: Copy) {
|
||||||
|
from project(':x-pack-elasticsearch:plugin')
|
||||||
|
.file('src/test/resources/org/elasticsearch/xpack/security/transport/ssl/certs/simple/testnode.jks')
|
||||||
|
into outputDir
|
||||||
|
}
|
||||||
|
|
||||||
for (Version version : wireCompatVersions) {
|
for (Version version : wireCompatVersions) {
|
||||||
String baseName = "v${version}"
|
String baseName = "v${version}"
|
||||||
|
|
||||||
|
@ -66,6 +72,7 @@ for (Version version : wireCompatVersions) {
|
||||||
|
|
||||||
Object extension = extensions.findByName("${baseName}#oldClusterTestCluster")
|
Object extension = extensions.findByName("${baseName}#oldClusterTestCluster")
|
||||||
configure(extensions.findByName("${baseName}#oldClusterTestCluster")) {
|
configure(extensions.findByName("${baseName}#oldClusterTestCluster")) {
|
||||||
|
dependsOn copyTestNodeKeystore
|
||||||
plugin ':x-pack-elasticsearch:plugin'
|
plugin ':x-pack-elasticsearch:plugin'
|
||||||
distribution = 'zip'
|
distribution = 'zip'
|
||||||
bwcVersion = version
|
bwcVersion = version
|
||||||
|
@ -168,15 +175,8 @@ task copyXPackRestSpec(type: Copy) {
|
||||||
into project.sourceSets.test.output.resourcesDir
|
into project.sourceSets.test.output.resourcesDir
|
||||||
}
|
}
|
||||||
|
|
||||||
task copyTestNodeKeystore(type: Copy) {
|
|
||||||
dependsOn(copyXPackRestSpec)
|
|
||||||
from project(':x-pack-elasticsearch:plugin')
|
|
||||||
.file('src/test/resources/org/elasticsearch/xpack/security/transport/ssl/certs/simple/testnode.jks')
|
|
||||||
into outputDir
|
|
||||||
}
|
|
||||||
|
|
||||||
task copyXPackPluginProps(type: Copy) {
|
task copyXPackPluginProps(type: Copy) {
|
||||||
dependsOn(copyTestNodeKeystore)
|
dependsOn(copyXPackRestSpec)
|
||||||
from project(':x-pack-elasticsearch:plugin').file('src/main/plugin-metadata')
|
from project(':x-pack-elasticsearch:plugin').file('src/main/plugin-metadata')
|
||||||
from project(':x-pack-elasticsearch:plugin').tasks.pluginProperties
|
from project(':x-pack-elasticsearch:plugin').tasks.pluginProperties
|
||||||
into outputDir
|
into outputDir
|
||||||
|
|
Loading…
Reference in New Issue